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 @@ -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) + 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', 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 @@ -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) + 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 +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) + 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) @@ -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) + 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 +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) + 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 '
.