🚑🐛 Fix all existing tests to match new reality

Author: webknjaz

cherry_picker/cherry_picker/test.py

@@ -10,7 +10,7 @@
     get_full_sha_from_short, get_author_info_from_short_sha, \
     CherryPicker, InvalidRepoException, \
     normalize_commit_message, DEFAULT_CONFIG, \
-    get_sha1_from, find_config, load_config
+    get_sha1_from, find_config, load_config, validate_sha
 
 
 @pytest.fixture
@@ -116,16 +116,22 @@ def test_get_cherry_pick_branch(os_path_exists, config):
     assert cp.get_cherry_pick_branch("3.6") == "backport-22a594a-3.6"
 
 
-@mock.patch('os.path.exists')
-@mock.patch('subprocess.check_output')
-def test_get_pr_url(subprocess_check_output, os_path_exists, config):
-    os_path_exists.return_value = True
-    subprocess_check_output.return_value = b'https://github.com/mock_user/cpython.git'
+def test_get_pr_url(config):
     branches = ["3.6"]
     cp = CherryPicker('origin', '22a594a0047d7706537ff2ac676cdc0f1dcb329c',
                       branches, config=config)
-    assert cp.get_pr_url("3.6", cp.get_cherry_pick_branch("3.6")) \
-           == "https://github.com/python/cpython/compare/3.6...mock_user:backport-22a594a-3.6?expand=1"
+    backport_target_branch = cp.get_cherry_pick_branch("3.6")
+    expected_pr_url = (
+        'https://github.com/python/cpython/compare/'
+        '3.6...mock_user:backport-22a594a-3.6?expand=1'
+    )
+    with mock.patch(
+            'subprocess.check_output',
+            return_value=b'https://github.com/mock_user/cpython.git',
+    ):
+        actual_pr_url = cp.get_pr_url("3.6", backport_target_branch)
+
+    assert actual_pr_url == expected_pr_url
 
 
 @pytest.mark.parametrize('url', [
@@ -137,42 +143,44 @@ def test_get_pr_url(subprocess_check_output, os_path_exists, config):
     b'https://github.com/mock_user/cpython',
     ])
 def test_username(url, config):
+    branches = ["3.6"]
+    cp = CherryPicker('origin', '22a594a0047d7706537ff2ac676cdc0f1dcb329c',
+                      branches, config=config)
     with mock.patch('subprocess.check_output', return_value=url):
-        branches = ["3.6"]
-        cp = CherryPicker('origin', '22a594a0047d7706537ff2ac676cdc0f1dcb329c',
-                          branches, config=config)
         assert cp.username == 'mock_user'
 
 
-@mock.patch('os.path.exists')
-@mock.patch('subprocess.check_output')
-def test_get_updated_commit_message(subprocess_check_output, os_path_exists,
-                                    config):
-    os_path_exists.return_value = True
-    subprocess_check_output.return_value = b'bpo-123: Fix Spam Module (#113)'
+def test_get_updated_commit_message(config):
     branches = ["3.6"]
     cp = CherryPicker('origin', '22a594a0047d7706537ff2ac676cdc0f1dcb329c',
                       branches, config=config)
-    assert cp.get_commit_message('22a594a0047d7706537ff2ac676cdc0f1dcb329c') \
-           == 'bpo-123: Fix Spam Module (GH-113)'
+    with mock.patch(
+            'subprocess.check_output',
+            return_value=b'bpo-123: Fix Spam Module (#113)',
+    ):
+        actual_commit_message = (
+            cp.get_commit_message('22a594a0047d7706537ff2ac676cdc0f1dcb329c')
+        )
+    assert actual_commit_message == 'bpo-123: Fix Spam Module (GH-113)'
 
 
-@mock.patch('os.path.exists')
-@mock.patch('subprocess.check_output')
-def test_get_updated_commit_message_without_links_replacement(
-        subprocess_check_output, os_path_exists, config):
-    os_path_exists.return_value = True
-    subprocess_check_output.return_value = b'bpo-123: Fix Spam Module (#113)'
+def test_get_updated_commit_message_without_links_replacement(config):
     config['fix_commit_msg'] = False
     branches = ["3.6"]
     cp = CherryPicker('origin', '22a594a0047d7706537ff2ac676cdc0f1dcb329c',
                       branches, config=config)
-    assert cp.get_commit_message('22a594a0047d7706537ff2ac676cdc0f1dcb329c') \
-           == 'bpo-123: Fix Spam Module (#113)'
+    with mock.patch(
+            'subprocess.check_output',
+            return_value=b'bpo-123: Fix Spam Module (#113)',
+    ):
+        actual_commit_message = (
+            cp.get_commit_message('22a594a0047d7706537ff2ac676cdc0f1dcb329c')
+        )
+    assert actual_commit_message == 'bpo-123: Fix Spam Module (#113)'
 
 
 @mock.patch('subprocess.check_output')
-def test_is_cpython_repo(subprocess_check_output, config):
+def test_is_cpython_repo(subprocess_check_output):
     subprocess_check_output.return_value = """commit 7f777ed95a19224294949e1b4ce56bbffcb1fe9f
 Author: Guido van Rossum <[email protected]>
 Date:   Thu Aug 9 14:25:15 1990 +0000
@@ -181,8 +189,7 @@ def test_is_cpython_repo(subprocess_check_output, config):
 
 """
     # should not raise an exception
-    CherryPicker('origin', '22a594a0047d7706537ff2ac676cdc0f1dcb329c',
-                 ["3.6"], config=config)
+    validate_sha('22a594a0047d7706537ff2ac676cdc0f1dcb329c')
 
 
 def test_is_not_cpython_repo():
@@ -240,6 +247,8 @@ def test_load_full_config(tmpdir, cd):
 
 
 def test_load_partial_config(tmpdir, cd):
+    cd(tmpdir)
+    subprocess.run('git init .'.split(), check=True)
     relative_config_path = '.cherry_picker.toml'
     cfg = tmpdir.join(relative_config_path)
     cfg.write('''\