Skip to content

Commit

Permalink
fix: mark tests that return None as skipped
Browse files Browse the repository at this point in the history
  • Loading branch information
0x0ece committed May 28, 2024
1 parent 18a4063 commit 4524483
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/test_suite/multiprocessing_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -365,6 +365,9 @@ def build_test_results(results: dict[str, str | None]) -> tuple[int, dict | None

protobuf_structures[target] = instruction_effects

if protobuf_structures[globals.solana_shared_library] is None:
return 0, None

test_case_passed = all(
protobuf_structures[globals.solana_shared_library] == result
for result in protobuf_structures.values()
Expand Down
3 changes: 3 additions & 0 deletions src/test_suite/test_suite.py
Original file line number Diff line number Diff line change
Expand Up @@ -335,10 +335,12 @@ def run_tests(
failed = 0
skipped = 0
failed_tests = []
skipped_tests = []
target_log_files = {target: None for target in shared_libraries}
for file_stem, status, stringified_results in test_case_results:
if stringified_results is None:
skipped += 1
skipped_tests.append(file_stem)
continue

for target, string_result in stringified_results.items():
Expand Down Expand Up @@ -377,6 +379,7 @@ def run_tests(
print(f"Passed: {passed}, Failed: {failed}, Skipped: {skipped}")
if verbose:
print(f"Failed tests: {failed_tests}")
print(f"Skipped tests: {skipped_tests}")


@app.command()
Expand Down

0 comments on commit 4524483

Please sign in to comment.