mozreview: fix pep8 white space errors (bug 1232700); r?smacleod draft
authorMauro Doglio <mdoglio@mozilla.com>
Wed, 16 Dec 2015 12:25:13 +0000
changeset 7162 e4b99e14ccd3b5135d261651157fa280ad9d73ad
parent 7161 294192cde265f1dec88a22d07d3bbeb9acbc0b2e
child 7163 04273860bf11e2bd9dfb025299635617f9e7c3d4
push id603
push usermdoglio@mozilla.com
push dateTue, 09 Feb 2016 16:30:08 +0000
reviewerssmacleod
bugs1232700
mozreview: fix pep8 white space errors (bug 1232700); r?smacleod This fixes the following errors: E201 whitespace after ‘(‘ E225 missing whitespace around operator E231 missing whitespace after ‘,’, ‘;’, or ‘:’ E261 at least two spaces before inline comment MozReview-Commit-ID: AwuT768Dw87
pylib/mozreview/mozreview/autoland/resources.py
pylib/mozreview/mozreview/bugzilla/models.py
pylib/mozreview/mozreview/extension.py
pylib/mozreview/mozreview/tests/test-browser-cache.py
--- a/pylib/mozreview/mozreview/autoland/resources.py
+++ b/pylib/mozreview/mozreview/autoland/resources.py
@@ -478,17 +478,17 @@ class AutolandRequestUpdateResource(WebA
             autoland_request = AutolandRequest.objects.get(
                 pk=fields['request_id'])
         except AutolandRequest.DoesNotExist:
             return DOES_NOT_EXIST
 
         rr = ReviewRequest.objects.get(pk=autoland_request.review_request_id)
 
         if fields['landed']:
-            autoland_request.repository_revision=fields['result']
+            autoland_request.repository_revision = fields['result']
             autoland_request.save()
 
             # If we've landed to the "inbound" repository, we'll close the
             # review request automatically.
             landing_repo = rr.repository.extra_data.get(
                 'landing_repository_url')
             if autoland_request.repository_url == landing_repo:
                 rr.close(ReviewRequest.SUBMITTED)
--- a/pylib/mozreview/mozreview/bugzilla/models.py
+++ b/pylib/mozreview/mozreview/bugzilla/models.py
@@ -205,17 +205,17 @@ def prune_inactive_users(commit=False, v
     - been asked to review a request
     - submitted a review
     - logged in since their account was created
 
     By default, this function will perform a dry run and will only
     commit the user deletion to the database if the commit argument
     is True.
     """
-    MAX_LOGIN_DIFFERENCE = datetime.timedelta(0,1) # 1 second
+    MAX_LOGIN_DIFFERENCE = datetime.timedelta(0, 1)  # 1 second
     SPECIAL_USERNAMES = [
         'admin',
         'mozreview',
     ]
     SPECIAL_EMAILS = [
         'autoland@mozilla.bugs',
         'mozreview+python@mozilla-com.bugs',
     ]
--- a/pylib/mozreview/mozreview/extension.py
+++ b/pylib/mozreview/mozreview/extension.py
@@ -144,27 +144,27 @@ class MozReviewExtension(Extension):
                 'mozreview/js/models/filediffreviewermodel.js',
                 'mozreview/js/collections/filediffreviewercollection.js',
                 'mozreview/js/init_filediffreviewer.js',
                 'mozreview/js/diffviewer_customizations.js',
             ],
             'apply_to': diffviewer_url_names,
         },
         'import-pullrequest': {
-            'source_filenames': ['mozreview/js/import-pullrequest.js',],
-            'apply_to': ['import_pullrequest',],
+            'source_filenames': ['mozreview/js/import-pullrequest.js', ],
+            'apply_to': ['import_pullrequest', ],
         },
         'reviews': {
             # TODO: Everything will break if init_rr.js is not first in this
             # list.
             'source_filenames': ['mozreview/js/init_rr.js',
                                  'mozreview/js/commits.js',
                                  'mozreview/js/review.js',
                                  'mozreview/js/autoland.js',
-                                 'mozreview/js/ui.mozreviewautocomplete.js',]
+                                 'mozreview/js/ui.mozreviewautocomplete.js', ]
         },
         'parent-review-requests': {
             'source_filenames': ['mozreview/js/parents.js'],
             'apply_to': review_request_url_names,
         },
     }
 
     resources = [
--- a/pylib/mozreview/mozreview/tests/test-browser-cache.py
+++ b/pylib/mozreview/mozreview/tests/test-browser-cache.py
@@ -56,17 +56,17 @@ class BrowserCacheTest(MozReviewWebDrive
         ln = WebDriverWait(self.browser, 3).until(
             EC.visibility_of_element_located((
                 By.XPATH,
                 "id('chunk0.0')/tr[1]/th[1]"
             )))
 
         ActionChains(self.browser).move_to_element(ln).click(ln).perform()
 
-        time.sleep(1) # Give time for the comment box to become focused
+        time.sleep(1)  # Give time for the comment box to become focused
         ActionChains(self.browser).send_keys("Comment!").perform()
 
         save = self.browser.find_element_by_xpath(
             "//div[@class='comment-dlg-footer']/div[@class='buttons']/"
             "input[@class='save']")
         save.click()
 
         finish_btn = WebDriverWait(self.browser, 3).until(