diff --git a/python-sortinghat.changes b/python-sortinghat.changes index 1db9c38..0d6ccb9 100644 --- a/python-sortinghat.changes +++ b/python-sortinghat.changes @@ -1,5 +1,5 @@ ------------------------------------------------------------------- -Tue May 13 03:02:07 UTC 2025 - Steve Kowalik +Thu May 15 02:33:32 UTC 2025 - Steve Kowalik - Update to 1.7.1: * New features: diff --git a/support-click-8.2.patch b/support-click-8.2.patch index 1cd4ad5..8ce4d91 100644 --- a/support-click-8.2.patch +++ b/support-click-8.2.patch @@ -2,12 +2,72 @@ Index: grimoirelab-sortinghat-1.7.1/tests/cli/test_cmd_add.py =================================================================== --- grimoirelab-sortinghat-1.7.1.orig/tests/cli/test_cmd_add.py +++ grimoirelab-sortinghat-1.7.1/tests/cli/test_cmd_add.py -@@ -225,7 +225,7 @@ class TestAddCommand(unittest.TestCase): +@@ -20,6 +20,7 @@ + # Santiago Dueñas + # + ++import inspect + import unittest + import unittest.mock + +@@ -107,7 +108,10 @@ class TestAddCommand(unittest.TestCase): + client = MockClient(responses) + mock_client.return_value = client + +- runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() + + # Create a new identity + params = [ +@@ -135,7 +139,10 @@ class TestAddCommand(unittest.TestCase): + client = MockClient(responses) + mock_client.return_value = client + +- runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() + + # Create a new identity setting partial data + params = [ +@@ -161,7 +168,10 @@ class TestAddCommand(unittest.TestCase): + client = MockClient(responses) + mock_client.return_value = client + +- runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() + + # Assign to John Smith - a9b403e150dd4af8953a52a4bb841051e4b705d9 + # individual +@@ -190,7 +200,11 @@ class TestAddCommand(unittest.TestCase): + client = MockClient(responses) + mock_client.return_value = client + +- runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() ++ + + params = [ + '--source', 'scm', +@@ -225,7 +239,10 @@ class TestAddCommand(unittest.TestCase): client = MockClient(responses) mock_client.return_value = client - runner = click.testing.CliRunner(mix_stderr=False) -+ runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() params = [ '--source', 'scm', @@ -15,75 +75,180 @@ Index: grimoirelab-sortinghat-1.7.1/tests/cli/test_cmd_config.py =================================================================== --- grimoirelab-sortinghat-1.7.1.orig/tests/cli/test_cmd_config.py +++ grimoirelab-sortinghat-1.7.1/tests/cli/test_cmd_config.py -@@ -104,7 +104,7 @@ class TestInitConfig(unittest.TestCase): +@@ -20,6 +20,7 @@ + # Santiago Dueñas + # + ++import inspect + import os.path + import shutil + import unittest +@@ -53,7 +54,10 @@ class TestInitConfig(unittest.TestCase): + def test_init(self): + """Check if it initializes a configuration file.""" + +- runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() + + with runner.isolated_filesystem() as fs: + filepath = os.path.join(fs, MOCK_CONFIG_FILE) +@@ -79,7 +83,10 @@ class TestInitConfig(unittest.TestCase): + def test_default_filename(self, mock_basepath): + """Check if it uses the default filename when filepath is not given""" + +- runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() + + with runner.isolated_filesystem() as fs: + dirpath = os.path.join(fs, '.sortinghat') +@@ -104,7 +111,10 @@ class TestInitConfig(unittest.TestCase): def test_config_is_not_overwritten(self): """Check whether an existing config file is not replaced""" - runner = click.testing.CliRunner(mix_stderr=False) -+ runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() with runner.isolated_filesystem() as fs: shutil.copy(MOCK_CONFIG_FILEPATH, fs) -@@ -132,7 +132,7 @@ class TestInitConfig(unittest.TestCase): +@@ -132,7 +142,10 @@ class TestInitConfig(unittest.TestCase): def test_overwrite_config(self): """Check whether an existing config file is overwritten""" - runner = click.testing.CliRunner(mix_stderr=False) -+ runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() with runner.isolated_filesystem() as fs: shutil.copy(MOCK_CONFIG_FILEPATH, fs) -@@ -240,7 +240,7 @@ class TestSetConfig(unittest.TestCase): +@@ -168,7 +181,10 @@ class TestSetConfig(unittest.TestCase): + def test_set_value(self): + """Check set method""" + +- runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() + + with runner.isolated_filesystem() as fs: + shutil.copy(MOCK_CONFIG_FILEPATH, fs) +@@ -210,7 +226,10 @@ class TestSetConfig(unittest.TestCase): + def test_default_filename(self, mock_basepath): + """Check if it uses the default filename when filepath is not given""" + +- runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() + + with runner.isolated_filesystem() as fs: + dirpath = os.path.join(fs, '.sortinghat') +@@ -240,7 +259,10 @@ class TestSetConfig(unittest.TestCase): def test_not_available_keys(self): """Check if it raises an error when the key is not available""" - runner = click.testing.CliRunner(mix_stderr=False) -+ runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() with runner.isolated_filesystem() as fs: shutil.copy(MOCK_CONFIG_FILEPATH, fs) -@@ -260,7 +260,7 @@ class TestSetConfig(unittest.TestCase): +@@ -260,7 +282,10 @@ class TestSetConfig(unittest.TestCase): def test_invalid_keys(self): """Check if it raises an error when the key is invalid""" - runner = click.testing.CliRunner(mix_stderr=False) -+ runner = click.testing.CliRunner() ++ if inspect.signature(click.testing.CliRunner).parameters.get("mix_stderr"): ++ runner = click.testing.CliRunner(mix_stderr=False) ++ else: ++ runner = click.testing.CliRunner() # Test keys that do not follow '
.