Skip to content

implemented test group class #484

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

implemented test group class #484

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
fdb9935
implemented test group class
mzuenni Nov 20, 2025
60a79fc
update tests
mzuenni Nov 20, 2025
5d7af40
fix test
mzuenni Nov 20, 2025
0d88635
fix test and implementation
mzuenni Nov 20, 2025
1fbf60f
better handling of ignored files
mzuenni Nov 20, 2025
649370d
skip includes if restricted
mzuenni Nov 21, 2025
e0fc70c
show more warnings
mzuenni Nov 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion bin/generate.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1407,6 +1407,8 @@ def __init__(
def walk(
self,
testcase_f: Optional[Callable[["TestcaseRule | Directory"], object]],
*,
skip_restricted: bool = True,
) -> None: ...

# This overload takes one function for test cases and a separate function for directories.
Expand All @@ -1415,6 +1417,8 @@ def walk(
self,
testcase_f: Optional[Callable[[TestcaseRule], object]],
dir_f: Optional[Callable[["Directory"], object]],
*,
skip_restricted: bool = True,
) -> None: ...

# Map a function over all test cases directory tree.
Expand All @@ -1428,6 +1432,8 @@ def walk(
| Optional[
Callable[["TestcaseRule | Directory"], object] | Callable[["Directory"], object]
] = True,
*,
skip_restricted: bool = True,
) -> None:
if dir_f is True:
dir_f = cast(Optional[Callable[["TestcaseRule | Directory"], object]], testcase_f)
Expand All @@ -1438,7 +1444,9 @@ def walk(
if isinstance(d, Directory):
d.walk(testcase_f, dir_f)
elif isinstance(d, TestcaseRule):
if testcase_f and d.process:
if not d.process and skip_restricted:
continue
if testcase_f:
testcase_f(d)
else:
assert False
Expand Down Expand Up @@ -1872,6 +1880,7 @@ def add_included_case(t: TestcaseRule) -> None:
obj.walk(
add_included_case,
lambda d: list(map(add_included_case, d.includes.values())),
skip_restricted=False,
)
pass
else:
Expand Down
Loading