Skip to content

Commit

Permalink
t2000-wreck: fix ngpus tests
Browse files Browse the repository at this point in the history
Somehow the ngpus tests in t2000-wreck.t were prefixed with a '+'
and more concerning, this just caused the tests to be skipped.
Remove the stray character and fix one of the tests.
  • Loading branch information
grondo committed May 8, 2018
1 parent 5e1b7ac commit 19dd1d2
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions t/t2000-wreck.t
Original file line number Diff line number Diff line change
Expand Up @@ -210,18 +210,18 @@ test_expect_success 'wreckrun: -t2 -N${SIZE} sets correct ntasks in kvs' '
test "$n" = $((${SIZE}*2))
'

+test_expect_success 'wreckrun: ngpus is 0 by default' '
test_expect_success 'wreckrun: ngpus is 0 by default' '
flux wreckrun -n 2 /bin/true &&
LWJ=$(last_job_path) &&
n=$(flux kvs get --json ${LWJ}.ngpus) &&
test "$n" = "0"
'

+test_expect_success 'wreckrun: -g, --ngpus sets ngpus in kvs' '
test_expect_success 'wreckrun: -g, --ngpus sets ngpus in kvs' '
flux wreckrun -n 2 -g 4 /bin/true &&
LWJ=$(last_job_path) &&
n=$(flux kvs get --json ${LWJ}.ngpus) &&
test "$n" = "4"
test "$n" = "8"
'
test_expect_success 'wreckrun: job with more nodes than tasks fails' '
test_must_fail flux wreckrun -n2 \
Expand Down

0 comments on commit 19dd1d2

Please sign in to comment.