Noissue - Adapt best practices for trailing commas (sitescripts)

sitescripts/hg/test/update_issues.py

 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-present eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
 #
 # You should have received a copy of the GNU General Public License
 # along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
 
 import ConfigParser
 import mock
 import unittest
 
 import sitescripts.hg.bin.update_issues as update_issues
 
 
 def _issue(component, milestone='', can_resolve=False):
     issue = {
         'attrs': {'_ts': 1, 'milestone': milestone, 'component': component},
-        'actions': [['leave', '', '', []]]
+        'actions': [['leave', '', '', []]],
     }
     if can_resolve:
         issue['actions'].append(['resolve', '', '', []])
     return issue
 
 
 ISSUES = {
     1337: _issue(component='one', can_resolve=True),
     2448: _issue(component='two'),
     3559: _issue(component='one', milestone='other'),
     4670: _issue(component='three', can_resolve=True),
-    5781: _issue(component='four', can_resolve=True)
+    5781: _issue(component='four', can_resolve=True),
 }
 
 MILESTONES = {
     'completed': {'completed': True, 'name': 'completed'},
     'current': {'completed': None, 'name': 'current'},
-    'other': {'completed': None, 'name': 'other'}
+    'other': {'completed': None, 'name': 'other'},
 }
 
 
 class _TestBase(unittest.TestCase):
     """Base class for hook tests that prepares the environment."""
 
     def _patchWith(self, target, return_value):
         patcher = mock.patch(target, return_value=return_value)
         patcher.start()
         self.addCleanup(patcher.stop)
@@ skipping 67 matching lines @@
         return updates
 
     def test_commits_with_invalid_message_format_ignored(self):
         self._run_hook([
             '',
             'Issue #1337 - Extraneous characters in issue number',
             'Issue 1337',  # No dash, no message.
             'Issue 1337: Colon instead of dash',
             'Noissue no dash',
             'Issue 1337-No space around dash',
-            'Fixes 1337 no dash'
+            'Fixes 1337 no dash',
         ], warning_count=7)
 
     def test_noissue_commits_ignored(self):
         self._run_hook(['Noissue - Foo', 'noissue - Bar'])  # No updates.
 
     def test_single_issue_referenced(self):
         updates = self._run_hook(['Issue 1337 - Foo'], update_count=1)
         self.assertEqual(updates[0][0][0], 1337)
 
     def test_multiline_commit_message(self):
@@ skipping 90 matching lines @@
 
     def test_fix_nonexistent(self):
         self._run_hook(['Fixes 7331 - Foo'], warning_count=1)
 
     def test_fix_closed_with_assigned_milestone(self):
         self._run_hook(['fixes 3559 - Foo'])  # No updates.
 
 
 if __name__ == '__main__':
     unittest.main()