From d0716cccc24b92af9f44d38875a46c38a3961045 Mon Sep 17 00:00:00 2001 From: Daniel Milroy Date: Mon, 10 Jul 2023 13:58:58 -0700 Subject: [PATCH] testsuite: update double-booking tests to test_expect_success Problem: https://github.com/flux-framework/flux-sched/pull/1044 introduced tests to check for allocation failures due to jobs that exceed their time limit. PR #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. --- t/t1024-alloc-check.t | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/t/t1024-alloc-check.t b/t/t1024-alloc-check.t index 3cb850874..cd5ced8d8 100755 --- a/t/t1024-alloc-check.t +++ b/t/t1024-alloc-check.t @@ -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' ' @@ -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' '