2
0
Isaac 2 жил өмнө
parent
commit
453b78c852

+ 9 - 23
tests/unit/commands/test_completions.py

@@ -91,47 +91,33 @@ test_data: list[TestCase] = [
 
 @pytest.mark.parametrize('action, option_type', test_data)
 def test_has_file_options_detects_file_options(action: Action, option_type: OptionType):
-    assert (
-        has_file_options(action) == option_type.file
-    ), f'Action: {action} should be file={option_type.file}'
+    assert has_file_options(action) == option_type.file
 
 
 @pytest.mark.parametrize('action, option_type', test_data)
 def test_has_choice_options_detects_choice_options(action: Action, option_type: OptionType):
-    assert (
-        has_choice_options(action) == option_type.choice
-    ), f'Action: {action} should be choice={option_type.choice}'
+    assert has_choice_options(action) == option_type.choice
 
 
 @pytest.mark.parametrize('action, option_type', test_data)
 def test_has_unknown_required_param_options_detects_unknown_required_param_options(
     action: Action, option_type: OptionType
 ):
-    assert (
-        has_unknown_required_param_options(action) == option_type.unknown_required
-    ), f'Action: {action} should be unknown_required={option_type.unknown_required}'
+    assert has_unknown_required_param_options(action) == option_type.unknown_required
 
 
 @pytest.mark.parametrize('action, option_type', test_data)
 def test_has_exact_options_detects_exact_options(action: Action, option_type: OptionType):
-    assert has_exact_options(action) == (
-        True in option_type
-    ), f'Action: {action} should have exact options given {option_type}'
+    assert has_exact_options(action) == (True in option_type)
 
 
 @pytest.mark.parametrize('action, option_type', test_data)
 def test_produce_exact_options_completion(action: Action, option_type: OptionType):
-    try:
-        completion = exact_options_completion(action)
-        if True in option_type:
-            assert completion.startswith(
-                '\ncomplete -c borgmatic'
-            ), f'Completion should start with "complete -c borgmatic", got {completion}'
-        else:
-            assert completion == '', f'Completion should be empty, got {completion}'
-
-    except ValueError as value_error:
-        assert False, f'exact_options_completion raised ValueError: {value_error}'
+    completion = exact_options_completion(action)
+    if True in option_type:
+        assert completion.startswith('\ncomplete -c borgmatic')
+    else:
+        assert completion == ''
 
 
 def test_dedent_strip_as_tuple_does_not_raise():