Skip to content
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.

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

Speed up final constraint enforcement #1286

Merged
merged 8 commits into from
Sep 3, 2024
Prev Previous commit
Next Next commit
t3027: use test_cmp rather than _json
problem: the files _start_ with json but then have other output after
that, so putting them through jq causes errors

solution: use test_cmp instead
trws committed Aug 31, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit 89a0502569127d365a7f1ea8be1044dd9049abea
6 changes: 3 additions & 3 deletions t/t3027-resource-RV.t
Original file line number Diff line number Diff line change
@@ -24,7 +24,7 @@ test_expect_success 'RV1 is correct on rank and node ID order match' '
quit
EOF
${query} -L ${jgf_orig} -f jgf -F rv1_nosched -t R1.out -P high < cmds001 &&
test_cmp_json R1.out ${exp_dir}/R1.out
test_cmp R1.out ${exp_dir}/R1.out
'

test_expect_success 'RV1 is correct on core ID modified' '
@@ -33,7 +33,7 @@ test_expect_success 'RV1 is correct on core ID modified' '
quit
EOF
${query} -L ${jgf_mod1} -f jgf -F rv1_nosched -t R2.out -P high < cmds002 &&
test_cmp_json R2.out ${exp_dir}/R2.out
test_cmp R2.out ${exp_dir}/R2.out
'

test_expect_success 'RV1 correct on rank/node ID mismatch + core ID modified' '
@@ -42,7 +42,7 @@ test_expect_success 'RV1 correct on rank/node ID mismatch + core ID modified' '
quit
EOF
${query} -L ${jgf_mod2} -f jgf -F rv1_nosched -t R3.out -P high < cmds003 &&
test_cmp_json R3.out ${exp_dir}/R3.out
test_cmp R3.out ${exp_dir}/R3.out
'

test_expect_success 'RV1 correct on heterogeneous configuration' '