diff --git a/schemachange/cli.py b/schemachange/cli.py index ca36feaa..1a4563a9 100644 --- a/schemachange/cli.py +++ b/schemachange/cli.py @@ -886,7 +886,7 @@ def get_all_scripts_recursively(root_directory, verbose): all_files = dict() all_versions = list() # Walk the entire directory structure recursively - for (directory_path, directory_names, file_names) in os.walk(root_directory): + for directory_path, directory_names, file_names in os.walk(root_directory): for file_name in file_names: file_full_path = os.path.join(directory_path, file_name) @@ -1008,7 +1008,7 @@ def inner_extract_dictionary_secrets( extracted_secrets: Set[str] = set() if dictionary: - for (key, value) in dictionary.items(): + for key, value in dictionary.items(): if isinstance(value, dict): if key == "secrets": extracted_secrets = ( diff --git a/tests/test_cli_misc.py b/tests/test_cli_misc.py index b5dd2e75..6924e0b1 100644 --- a/tests/test_cli_misc.py +++ b/tests/test_cli_misc.py @@ -123,12 +123,11 @@ def test_get_change_history_table_details_given__acceptable_values_produces_full ): assert schemachange.cli.get_change_history_table_details(cht) == expected + @pytest.mark.parametrize( "cht", [("fifth.fourth.third.two.one"), ("fourth.third.two.one")] ) -def test_get_change_history_table_details_given__unacceptable_values_raises_error( - cht -): +def test_get_change_history_table_details_given__unacceptable_values_raises_error(cht): with pytest.raises(ValueError) as e: schemachange.cli.get_change_history_table_details(cht)