Skip to content
Snippets Groups Projects
Commit f16876c6 authored by Morris Jette's avatar Morris Jette
Browse files

Correct bad test logic

AND should have been an OR.
Modify test to log more details on failure.
parent 4656b969
No related branches found
No related tags found
No related merge requests found
...@@ -96,11 +96,19 @@ expect { ...@@ -96,11 +96,19 @@ expect {
} }
} }
if {$failed_cnt != 2 && $timeout_cnt != 1} { if {$failed_cnt != 2 || $timeout_cnt != 1} {
send_user "\nFAILURE: batch job $job_id did not FAIL and/or the step " send_user "\nFAILURE: batch job $job_id did not FAIL and/or the step "
send_user "did not TIMEOUT. Count for FAILED was $failed_cnt and " send_user "did not TIMEOUT. Count for FAILED was $failed_cnt (not 2) and "
send_user "TIMEOUT was $timeout_cnt\n" send_user "TIMEOUT was $timeout_cnt (not 1)\n"
set exit_code 1 set exit_code 1
send_user "\nGathering more job information:\n"
spawn $sacct --jobs=$job_id
expect {
eof {
wait
}
}
} }
if {$exit_code == 0} { if {$exit_code == 0} {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment