Skip to content

Commit

Permalink
t: add coverage of -L color short option
Browse files Browse the repository at this point in the history
Problem: In several tests the -L (short option of --color) is tested
without arguments, but it is not tested with arguments.

Add "-Lalways" as an additional test in t1008-kvs-eventlog.t and
t2230-job-info-lookup.t.
  • Loading branch information
chu11 committed Apr 17, 2024
1 parent 90f0d30 commit 5db6f9a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion t/t1008-kvs-eventlog.t
Expand Up @@ -69,7 +69,7 @@ test_expect_success 'flux kvs eventlog get/wait-event reject invalid --color' '
test_must_fail flux kvs eventlog get --color=foo test.human &&
test_must_fail flux kvs eventlog wait-event --color=foo test.human primus
'
for opt in "-L" "--color" "--color=always"; do
for opt in "-L" "-Lalways" "--color" "--color=always"; do
test_expect_success "flux kvs eventlog get $opt forces color on" '
name=notty${opt##--color=} &&
outfile=color-${name:-default}.out &&
Expand Down
2 changes: 1 addition & 1 deletion t/t2230-job-info-lookup.t
Expand Up @@ -255,7 +255,7 @@ test_expect_success 'flux job eventlog/wait-event reject invalid --color' '
test_must_fail flux job eventlog --color=foo $jobid &&
test_must_fail flux job wait-event --color=foo $jobid
'
for opt in "-L" "--color" "--color=always"; do
for opt in "-L" "-Lalways" "--color" "--color=always"; do
test_expect_success "flux job eventlog $opt forces color on" '
name=notty${opt##--color=} &&
outfile=color-${name:-default}.out &&
Expand Down

0 comments on commit 5db6f9a

Please sign in to comment.