commit 5d55607e5becb89b2055e8f5730d288952444d31
author Zhongyue Luo <zhongyue.nah@intel.com> Fri Sep 21 02:00:47 2012 +0900
committer Jenkins <jenkins@review.openstack.org> Wed Sep 26 14:23:10 2012 +0000
tree 28571bdda92a5a55c75c4f61795fd538d8006409
parent 6ad5d5b1afd342bbdab6186d8c8c6677b721a459

Fix pep8 E127 violations

Updated pep8 version requirement to 1.3.3
Fixed E127 errors
All ignores are to be removed in the next sequence of patches

Change-Id: Ia9e922b8873686a0f905f2548cc43d534ee1c912
Reviewed-on: https://review.openstack.org/13642
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Zhongyue Luo <zhongyue.nah@intel.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins

tests/test_scheduler.py

diff --git a/tests/test_scheduler.py b/tests/test_scheduler.py
index 41848f6..3493c98 100644
--- a/tests/test_scheduler.py
+++ b/tests/test_scheduler.py
@@ -362,14 +362,17 @@
 
 class FakeJenkinsEvent(object):
     def __init__(self, name, number, parameters, phase, status=None):
-        data = {'build':
-                     {'full_url': 'https://server/job/%s/%s/' % (name, number),
-                      'number': number,
-                      'parameters': parameters,
-                      'phase': phase,
-                      'url': 'job/%s/%s/' % (name, number)},
-                     'name': name,
-                     'url': 'job/%s/' % name}
+        data = {
+            'build': {
+                'full_url': 'https://server/job/%s/%s/' % (name, number),
+                'number': number,
+                'parameters': parameters,
+                'phase': phase,
+                'url': 'job/%s/%s/' % (name, number),
+            },
+            'name': name,
+            'url': 'job/%s/' % name,
+        }
         if status:
             data['build']['status'] = status
         self.body = json.dumps(data)
@@ -815,7 +818,7 @@
     def test_independent_queues(self):
         "Test that changes end up in the right queues"
         self.fake_jenkins.hold_jobs_in_build = True
-        A = self.fake_gerrit.addFakeChange('org/project',  'master', 'A')
+        A = self.fake_gerrit.addFakeChange('org/project', 'master', 'A')
         B = self.fake_gerrit.addFakeChange('org/project1', 'master', 'B')
         C = self.fake_gerrit.addFakeChange('org/project2', 'master', 'C')
         A.addApproval('CRVW', 2)
@@ -1122,13 +1125,18 @@
 
     def test_post(self):
         "Test that post jobs run"
-        e = {"type": "ref-updated",
-             "submitter": {"name": "User Name"},
-             "refUpdate": {"oldRev":
-                               "90f173846e3af9154517b88543ffbd1691f31366",
-                           "newRev":
-                               "d479a0bfcb34da57a31adb2a595c0cf687812543",
-                           "refName": "master", "project": "org/project"}}
+        e = {
+            "type": "ref-updated",
+            "submitter": {
+                "name": "User Name",
+            },
+            "refUpdate": {
+                "oldRev": "90f173846e3af9154517b88543ffbd1691f31366",
+                "newRev": "d479a0bfcb34da57a31adb2a595c0cf687812543",
+                "refName": "master",
+                "project": "org/project",
+            }
+        }
         self.fake_gerrit.addEvent(e)
         self.waitUntilSettled()