Skip to content

Commit f0c2187

Browse files
committed
Fix get-jobs test check
This commit makes the end-to-end tests for get-jobs actually check the result of the test, and fixes a test that was failing silently.
1 parent cad81c0 commit f0c2187

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

test/e2e/run

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,8 @@ def get_jobs(litani, run_dir, mod):
112112
cmd.extend(configure_args(*args.get("args", []), **args.get("kwargs", {})))
113113
jobs = subprocess.check_output(cmd)
114114

115-
mod.check_get_jobs(jobs)
115+
if not mod.check_get_jobs(jobs):
116+
sys.exit(1)
116117

117118
for job in mod.get_post_check_jobs():
118119
run_litani(

test/e2e/tests/get_jobs_single_job.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919

2020
EXPECTED_JOB = {
2121
"command": "echo foo",
22-
"ci-stage": "build",
22+
"ci_stage": "build",
2323
"pipeline": "foo",
2424
}
2525

0 commit comments

Comments
 (0)