From 433908999bd6bb324074dce45b33f66869c5ec2c Mon Sep 17 00:00:00 2001 From: Julen Landa Alustiza Date: Nov 21 2019 10:15:05 +0000 Subject: some black formatting --- diff --git a/pagure/hooks/files/git_multimail_upstream.py b/pagure/hooks/files/git_multimail_upstream.py index 8cdd156..0780563 100755 --- a/pagure/hooks/files/git_multimail_upstream.py +++ b/pagure/hooks/files/git_multimail_upstream.py @@ -3636,9 +3636,10 @@ class Push(object): "%(objectname) %(objecttype) %(refname)\n" "%(*objectname) %(*objecttype) %(refname)" ) - ref_filter_regex, is_inclusion_filter = ( - self.environment.get_ref_filter_regex() - ) + ( + ref_filter_regex, + is_inclusion_filter, + ) = self.environment.get_ref_filter_regex() for line in read_git_lines( ["for-each-ref", "--format=%s" % (fmt,)] ): @@ -3865,9 +3866,10 @@ def include_ref(refname, ref_filter_regex, is_inclusion_filter): def run_as_post_receive_hook(environment, mailer): environment.check() - send_filter_regex, send_is_inclusion_filter = environment.get_ref_filter_regex( - True - ) + ( + send_filter_regex, + send_is_inclusion_filter, + ) = environment.get_ref_filter_regex(True) ref_filter_regex, is_inclusion_filter = environment.get_ref_filter_regex( False ) @@ -3902,9 +3904,10 @@ def run_as_update_hook( environment, mailer, refname, oldrev, newrev, force_send=False ): environment.check() - send_filter_regex, send_is_inclusion_filter = environment.get_ref_filter_regex( - True - ) + ( + send_filter_regex, + send_is_inclusion_filter, + ) = environment.get_ref_filter_regex(True) ref_filter_regex, is_inclusion_filter = environment.get_ref_filter_regex( False ) diff --git a/tests/test_pagure_flask.py b/tests/test_pagure_flask.py index 00e56ed..0a1b8d1 100644 --- a/tests/test_pagure_flask.py +++ b/tests/test_pagure_flask.py @@ -301,7 +301,7 @@ class PagureGetRemoteRepoPath(tests.SimplePagureTest): @mock.patch.dict("pagure.config.config", {"EXTERNAL_COMMITTER": config}) def test_is_repo_committer_external_committer_restricting_another_one( - self + self, ): """ Test is_repo_committer in pagure with EXTERNAL_COMMITTER configured to give access the provenpackager just for one repo not diff --git a/tests/test_tasks.py b/tests/test_tasks.py index fe2f8b6..2c19538 100644 --- a/tests/test_tasks.py +++ b/tests/test_tasks.py @@ -59,9 +59,12 @@ class TestCommitsAuthorStats(unittest.TestCase): "alice@example.com": MockUser("Alice", "alice@example.com") } - num_commits, authors, num_authors, last_time = tasks.commits_author_stats( - self.repopath - ) + ( + num_commits, + authors, + num_authors, + last_time, + ) = tasks.commits_author_stats(self.repopath) self.assertEqual(num_commits, 1) self.assertEqual(num_authors, 1) @@ -109,9 +112,12 @@ class TestCommitsAuthorStats(unittest.TestCase): "alice@example.com": MockUser("Alice", "alice@example.com") } - num_commits, authors, num_authors, last_time = tasks.commits_author_stats( - self.repopath - ) + ( + num_commits, + authors, + num_authors, + last_time, + ) = tasks.commits_author_stats(self.repopath) self.assertEqual(num_commits, 2) self.assertEqual(num_authors, 1) @@ -156,9 +162,12 @@ class TestCommitsAuthorStats(unittest.TestCase): ] self.authors = {} - num_commits, authors, num_authors, last_time = tasks.commits_author_stats( - self.repopath - ) + ( + num_commits, + authors, + num_authors, + last_time, + ) = tasks.commits_author_stats(self.repopath) self.assertEqual(num_commits, 1) self.assertEqual(num_authors, 1) @@ -206,9 +215,12 @@ class TestCommitsAuthorStats(unittest.TestCase): ] self.authors = {} - num_commits, authors, num_authors, last_time = tasks.commits_author_stats( - self.repopath - ) + ( + num_commits, + authors, + num_authors, + last_time, + ) = tasks.commits_author_stats(self.repopath) self.assertEqual(num_commits, 3) self.assertEqual(num_authors, 2)