Skip to content

Commit

Permalink
Fix broken && chains in t4014
Browse files Browse the repository at this point in the history
Problem: fedora40 CI fails on t4014 due to """
  error: bug in the test script: broken &&-chain:
      load_feasibility
      flux dmesg -c | grep -q "File exists"
"""

Add && between successive statements in t4014.
  • Loading branch information
jacobtkeio committed Oct 28, 2024
1 parent d785222 commit 17a1d65
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions t/t4014-match-feasibility.t
Original file line number Diff line number Diff line change
Expand Up @@ -21,28 +21,28 @@ test_debug '
'

test_expect_success 'loading feasibility module over sched-simple fails' '
load_feasibility
flux dmesg -c | grep -q "File exists"
load_feasibility 2>&1 | grep -q "File exists"
'

test_expect_success 'removing sched-simple works' '
flux module remove sched-simple
flux module remove sched-simple &&
flux dmesg -c | grep -q "rmmod sched-simple"
'

test_expect_success 'loading feasibility module before resource fails' '
load_feasibility
load_feasibility &&
flux dmesg -c | grep -q "Function not implemented"
'

test_expect_success 'loading resource module with a tiny machine config works' '
load_resource load-file=${grug} load-format=grug \
prune-filters=ALL:core subsystems=containment policy=high
prune-filters=ALL:core subsystems=containment policy=high &&
test -z "$(flux dmesg -c | grep -q err)"
'

test_expect_success 'loading feasibility module with a tiny machine config works' '
load_feasibility load-file=${grug} load-format=grug \
prune-filters=ALL:core subsystems=containment policy=high
prune-filters=ALL:core subsystems=containment policy=high &&
test -z "$(flux dmesg -c | grep -q err)"
'

Expand Down Expand Up @@ -72,7 +72,7 @@ test_expect_success 'jobspec is still satisfiable even when no available resourc
'

test_expect_success 'removing load-file feasibility module works' '
remove_feasibility
remove_feasibility &&
test -z "$(flux dmesg -c | grep -q err)"
'

Expand All @@ -89,12 +89,12 @@ test_expect_success 'removing resource module works' '
'

test_expect_success 'loading non-load-file resource module works' '
load_resource
load_resource &&
test -z "$(flux dmesg -c | grep -q err)"
'

test_expect_success 'loading feasibility from non-load-file resource module works' '
load_feasibility
load_feasibility &&
test -z "$(flux dmesg -c | grep -q err)"
'

Expand Down

0 comments on commit 17a1d65

Please sign in to comment.