Skip to content

Fix Skipped Test Issue #904

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

Merged
merged 6 commits into from
Jun 28, 2025
Merged

Fix Skipped Test Issue #904

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
8528af5
Fix Skipped Test Issue
XZTian64 Jun 26, 2025
e1a6605
Modify for Reviewer
XZTian64 Jun 26, 2025
9ce824e
mpi single
sbryngelson Jun 27, 2025
79f4692
Merge branch 'master' into SkipTests
XZTian64 Jun 27, 2025
889f159
Discard changes to .github/workflows/test.yml
sbryngelson Jun 27, 2025
295be10
Merge branch 'master' into SkipTests
sbryngelson Jun 27, 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
9 changes: 7 additions & 2 deletions toolchain/mfc/test/test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ def __filter(cases_) -> typing.List[TestCase]:
# Do not "continue" because "--to" might be the same as "--from"
if bFoundFrom and case.get_uuid() == ARG("to"):
cases = cases[from_i:i+1]
skipped_cases = [case for case in cases_ if case not in cases]
bFoundTo = True
break

Expand All @@ -51,6 +52,7 @@ def __filter(cases_) -> typing.List[TestCase]:
checkCase.append(case.get_uuid())
if not set(ARG("only")).issubset(set(checkCase)):
cases.remove(case)
skipped_cases.append(case)

for case in cases[:]:
if case.ppn > 1 and not ARG("mpi"):
Expand All @@ -63,18 +65,21 @@ def __filter(cases_) -> typing.List[TestCase]:
,'Axisymmetric', 'Transducer', 'Transducer Array', 'Cylindrical', 'HLLD', 'Example']
if any(label in case.trace for label in skip):
cases.remove(case)
skipped_cases.append(case)


if ARG("no_examples"):
cases = [case for case in cases if not "Example" in case.trace]
example_cases = [case for case in cases if "Example" in case.trace]
skipped_cases += example_cases
cases = [case for case in cases if case not in example_cases]

if ARG("percent") == 100:
return cases, skipped_cases

seed(time.time())

selected_cases = sample(cases, k=int(len(cases)*ARG("percent")/100.0))
skipped_cases = [item for item in cases if item not in selected_cases]
skipped_cases += [item for item in cases if item not in selected_cases]

return selected_cases, skipped_cases

Expand Down
Loading