Skip to content

Commit

Permalink
testsuite: ensure parent-* attributes are set only for jobs
Browse files Browse the repository at this point in the history
Problem: No tests in the testsuite ensure that parent-uri and
parent-kvs-namespace are set only for instance that are jobs.

Add a couple tests to t0001-basic.t.
  • Loading branch information
grondo committed Jan 21, 2025
1 parent 5cae75f commit fb6f976
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion t/t0001-basic.t
Original file line number Diff line number Diff line change
Expand Up @@ -557,7 +557,17 @@ test_expect_success 'broker broker.pid attribute is readable' '
test -n "$BROKERPID" &&
test "$BROKERPID" -eq "$BROKERPID"
'

test_expect_success 'broker sets parent-uri attribute only for jobs' '
flux start flux run flux start flux getattr parent-uri &&
test_must_fail \
flux start flux run flux start -s1 flux getattr parent-uri
'
test_expect_success 'broker sets parent-kvs-namespace attribute only for jobs' '
flux start flux run flux start flux getattr parent-kvs-namespace &&
test_must_fail \
flux start flux run \
flux start -s1 flux getattr parent-kvs-namespace
'
test_expect_success 'local-uri override works' '
sockdir=$(mktemp -d) &&
newsock=local://$sockdir/meep &&
Expand Down

0 comments on commit fb6f976

Please sign in to comment.