Skip to content

Commit

Permalink
selftests/check.py add check for cleanup result dir
Browse files Browse the repository at this point in the history
Let's ensure that the selftests/check.py doesn't leave test results
behind in user's default results directory.

Reference: #5554
Signed-off-by: Jan Richter <[email protected]>
  • Loading branch information
richtja committed Dec 22, 2022
1 parent 54fbf14 commit c5975e2
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions selftests/check.py
Original file line number Diff line number Diff line change
Expand Up @@ -798,7 +798,15 @@ def main(args): # pylint: disable=W0621
suite.config["run.max_parallel_tasks"] = max_parallel

with Job(config, suites) as j:
pre_job_test_result_dirs = set(os.listdir(os.path.dirname(j.logdir)))
exit_code = j.run()
post_job_test_result_dirs = set(os.listdir(os.path.dirname(j.logdir)))
if len(pre_job_test_result_dirs) != len(post_job_test_result_dirs):
if exit_code == 0:
exit_code = 1
print("check.py didn't clean test results.")
print("uncleaned directories:")
print(post_job_test_result_dirs.difference(pre_job_test_result_dirs))
return exit_code


Expand Down

0 comments on commit c5975e2

Please sign in to comment.