Skip to content

Commit

Permalink
testsuite: cover flux module trace --full
Browse files Browse the repository at this point in the history
Problem: there is no test coverage for flux module trace --full.

Add some.
  • Loading branch information
garlick committed Oct 4, 2024
1 parent f8f358e commit 3d6d6e7
Showing 1 changed file with 18 additions and 1 deletion.
19 changes: 18 additions & 1 deletion t/t3401-module-trace.t
Original file line number Diff line number Diff line change
Expand Up @@ -24,17 +24,34 @@ test_expect_success NO_CHAIN_LINT 'start background trace' '
flux module trace kvs >trace.out &
echo $! >trace.pid
'
test_expect_success NO_CHAIN_LINT 'start second background trace with --full' '
flux module trace --full kvs >trace1a.out &
echo $! >trace1a.pid
'
test_expect_success NO_CHAIN_LINT 'heartbeat.pulse event was captured' '
$waitfile -t 60 -p heartbeat.pulse trace.out
'
test_expect_success NO_CHAIN_LINT 'heartbeat.pulse event was captured with --full' '
$waitfile -t 60 -p heartbeat.pulse trace1a.out
'
test_expect_success NO_CHAIN_LINT 'send one kvs.ping' '
flux ping -c 1 kvs
'
test_expect_success NO_CHAIN_LINT 'kvs.ping request/response was captured' '
$waitfile -t 60 -c 2 -p kvs.ping trace.out
'
test_expect_success NO_CHAIN_LINT 'kvs.ping request/response was captured with --full' '
$waitfile -t 60 -c 2 -p kvs.ping trace1a.out
'
test_expect_success NO_CHAIN_LINT 'stop background trace' '
kill -15 $(cat trace.pid); wait || true
pid=$(cat trace.pid) &&
kill -15 $pid &&
wait $pid || true
'
test_expect_success NO_CHAIN_LINT 'stop second background trace' '
pid=$(cat trace1a.pid) &&
kill -15 $pid &&
wait $pid || true
'

test_expect_success NO_CHAIN_LINT 'start background trace on multiple modules' '
Expand Down

0 comments on commit 3d6d6e7

Please sign in to comment.