Merge pull request #292 from kyrofa/backport/catkin_pep8_fixes

Backport to 1.x: Fix ordering for conditional negations in catkin tests
canonical · Feb 4, 2016 · b5f04b9 · b5f04b9
2 parents 0d46bfa + 4d33612
commit b5f04b9
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/snapcraft/tests/test_plugin_catkin.py b/snapcraft/tests/test_plugin_catkin.py
@@ -331,11 +331,11 @@ def __init__(self, test):
             def __eq__(self, args):
                 index = args.index('--pkg')
                 packages = args[index+1:index+3]
-                if not 'my_package' in packages:
+                if 'my_package' not in packages:
                     self.test.fail('Expected "my_package" to be installed '
                                    'within the same command as "package_2"')
 
-                if not 'package_2' in packages:
+                if 'package_2' not in packages:
                     self.test.fail('Expected "package_2" to be installed '
                                    'within the same command as "my_package"')