floraachy
/
pytest2
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

chore: remove obsolete `TODO`s

This commit is contained in:
lovetheguitar 2024-06-22 20:49:21 +02:00
parent dd57196953
commit 3d07791c36
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
testing/test_mark.py
View File

@ -235,7 +235,7 @@ def test_mark_option(
@pytest.mark.parametrize( @pytest.mark.parametrize(
("expr", "expected_passed"), ("expr", "expected_passed"),
[ # TODO: improve/sort out [
("car(color='red')", ["test_one"]), ("car(color='red')", ["test_one"]),
("car(color='red') or car(color='blue')", ["test_one", "test_two"]), ("car(color='red') or car(color='blue')", ["test_one", "test_two"]),
("car and not car(temp=5)", ["test_one", "test_three"]), ("car and not car(temp=5)", ["test_one", "test_three"]),

4
testing/test_mark_expression.py
View File

@ -231,7 +231,7 @@ def test_invalid_idents(ident: str) -> None:
("'str'", "expected not OR left parenthesis OR identifier; got string literal"), ("'str'", "expected not OR left parenthesis OR identifier; got string literal"),
), ),
) )
def test_invalid_kwarg_name_or_value( # TODO: move to `test_syntax_errors` ? def test_invalid_kwarg_name_or_value(
expr: str, expected_error_msg: str, mark_matcher: MarkMatcher expr: str, expected_error_msg: str, mark_matcher: MarkMatcher
) -> None: ) -> None:
with pytest.raises(ParseError, match=expected_error_msg): with pytest.raises(ParseError, match=expected_error_msg):
@ -290,7 +290,7 @@ def test_keyword_expressions_with_numbers(
("builtin_matchers_mark(z=1)", False), ("builtin_matchers_mark(z=1)", False),
), ),
) )
def test_builtin_matchers_keyword_expressions( # TODO: naming when decided def test_builtin_matchers_keyword_expressions(
expr: str, expected: bool, mark_matcher: MarkMatcher expr: str, expected: bool, mark_matcher: MarkMatcher
) -> None: ) -> None:
assert evaluate(expr, mark_matcher) is expected assert evaluate(expr, mark_matcher) is expected