Skip to content

Commit

Permalink
testsuite: require flux-security v0.14.0 where needed
Browse files Browse the repository at this point in the history
Problem: Tests that load the sdexec module require flux-security
v0.14.0 or later, but this is not checked.

Add checks to the sdexec tests for the correct flux-security version.
  • Loading branch information
grondo committed Feb 28, 2025
1 parent 8be6869 commit 7a536d5
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 0 deletions.
4 changes: 4 additions & 0 deletions t/t2409-sdexec.t
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,10 @@ if ! busctl --user status >/dev/null; then
skip_all="user dbus is not running"
test_done
fi
if ! test_flux_security_version 0.14.0; then
skip_all="requires flux-security >= v0.14, got ${FLUX_SECURITY_VERSION}"
test_done
fi

test_under_flux 2 minimal

Expand Down
4 changes: 4 additions & 0 deletions t/t2410-sdexec-memlimit.t
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,10 @@ if ! systemctl show user@$(id -u) -p DelegateControllers | grep memory; then
skip_all="cgroups memory controller is not delegated"
test_done
fi
if ! test_flux_security_version 0.14.0; then
skip_all="requires flux-security >= v0.14, got ${FLUX_SECURITY_VERSION}"
test_done
fi
if stress=$(which stress); then
test_set_prereq STRESS
fi
Expand Down
4 changes: 4 additions & 0 deletions t/t2411-sdexec-job.t
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,10 @@ if ! busctl --user status >/dev/null; then
skip_all="user dbus is not running"
test_done
fi
if ! test_flux_security_version 0.14.0; then
skip_all="requires flux-security >= v0.14, got ${FLUX_SECURITY_VERSION}"
test_done
fi

mkdir -p config
cat >config/config.toml <<EOT
Expand Down

0 comments on commit 7a536d5

Please sign in to comment.