On Tue, Jul 02, 2013 at 05:04:49PM +0530, Prasanna Santhanam wrote: > On Tue, Jul 02, 2013 at 03:33:39PM +0530, Girish Shilamkar wrote: > > Hello Prasanna, > > > > While adding rest of the affinity rules tests I had removed affinity rules > > test from smoke. But while > > going through various iterations on the review board this hunk to remove > > test from smoke was dropped. > > The same test is now included in component. > > > I think we should remove test_affnity_groups.py from smoke. > > We'll have one test suite per feature at least in the smoke set. This > includes some basic positive tests that the dev owner will include as > part of their feature merge to master. > > The component tests are in-depth coverage of the feature written by > QAs. > > -- BTW, most of the affinity_group tests in component suite are failing/erroring out: https://gist.github.com/vogxn/5908555
------------------------ Powered by BigRock.com
