diff --git a/tests/test_workflows.py b/tests/test_workflows.py index 9d02b3a6f1..a20c0ab90f 100644 --- a/tests/test_workflows.py +++ b/tests/test_workflows.py @@ -2491,7 +2491,7 @@ def test_validate_missing_condition(self): assert any("missing 'condition'" in e for e in errors) @pytest.mark.parametrize("bad", [["a", "b"], {"k": "v"}, 5, 1.5]) - def test_validate_rejects_non_string_condition(self, bad): + def test_validate_rejects_non_string_non_bool_condition(self, bad): # A list/dict/number condition is returned unchanged by # evaluate_expression, and evaluate_condition then bool()-coerces it, so # it silently resolves to its truthiness (e.g. [1, 2] is always True) @@ -2908,7 +2908,7 @@ def test_validate_missing_fields(self): # max_iterations is optional (defaults to 10) @pytest.mark.parametrize("bad", [["a", "b"], {"k": "v"}, 5, 1.5]) - def test_validate_rejects_non_string_condition(self, bad): + def test_validate_rejects_non_string_non_bool_condition(self, bad): from specify_cli.workflows.steps.while_loop import WhileStep step = WhileStep() @@ -3040,7 +3040,7 @@ def test_validate_missing_fields(self): # max_iterations is optional (defaults to 10) @pytest.mark.parametrize("bad", [["a", "b"], {"k": "v"}, 5, 1.5]) - def test_validate_rejects_non_string_condition(self, bad): + def test_validate_rejects_non_string_non_bool_condition(self, bad): from specify_cli.workflows.steps.do_while import DoWhileStep step = DoWhileStep()