Skip to content

Commit

Permalink
testsuite: update double-booking tests to test_expect_success
Browse files Browse the repository at this point in the history
Problem: flux-framework#1044 introduced
tests to check for allocation failures due to jobs that
exceed their time limit. PR flux-framework#1044 had to define the two
`alloc-check` tests as `test_expect_failure` because the
mechanism to prevent multiple booking was not in place.

Update the two `alloc-check` tests to test_expect_success
since the PR of this commit fixes multiple booking in
Fluxion.
  • Loading branch information
milroy committed Jul 10, 2023
1 parent 0efd133 commit 2253a08
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions t/t1024-alloc-check.t
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ test_expect_success 'submit node-exclusive jobs that exceed their time limit' '
test_expect_success 'some jobs received timeout exception' '
grep "job.exception type=timeout" joberr
'
test_expect_failure 'no jobs received alloc-check exception' '
test_expect_success 'no jobs received alloc-check exception' '
test_must_fail grep "job.exception type=alloc-check" joberr
'
test_expect_success 'clean up' '
Expand All @@ -57,7 +57,7 @@ test_expect_success 'submit non-exclusive jobs that exceed their time limit' '
test_expect_success 'some jobs received timeout exception' '
grep "job.exception type=timeout" joberr2
'
test_expect_failure 'no jobs received alloc-check exception' '
test_expect_success 'no jobs received alloc-check exception' '
test_must_fail grep "job.exception type=alloc-check" joberr2
'
test_expect_success 'clean up' '
Expand Down

0 comments on commit 2253a08

Please sign in to comment.