Skip to content

Commit 7a40e83

Browse files
committed
testsuite: update usage of flux-ping --pad option
Problem: Tests in t0007-ping.t use the full number of bytes with the pad option, but this unnecessary and verbose. Update the uses of the --pad option to include a K or M suffix on the argument for readability.
1 parent d1d828d commit 7a40e83

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

t/t0007-ping.t

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,23 +17,23 @@ test_expect_success 'ping: 10K 1K byte echo requests' '
1717
'
1818

1919
test_expect_success 'ping: 1K 10K byte echo requests' '
20-
run_timeout 15 flux ping --pad 10240 --count 1024 --interval 0 0
20+
run_timeout 15 flux ping --pad 10K --count 1024 --interval 0 0
2121
'
2222

2323
test_expect_success 'ping: 100 100K byte echo requests' '
24-
run_timeout 15 flux ping --pad 102400 --count 100 --interval 0 0
24+
run_timeout 15 flux ping --pad 100K --count 100 --interval 0 0
2525
'
2626

2727
test_expect_success 'ping: 10 1M byte echo requests' '
28-
run_timeout 15 flux ping --pad 1048576 --count 10 --interval 0 0
28+
run_timeout 15 flux ping --pad 1M --count 10 --interval 0 0
2929
'
3030

3131
test_expect_success 'ping: 10 1M byte echo requests (batched)' '
32-
run_timeout 15 flux ping --pad 1048576 --count 10 --batch --interval 0 0
32+
run_timeout 15 flux ping --pad 1M --count 10 --batch --interval 0 0
3333
'
3434

3535
test_expect_success 'ping: 1K 10K byte echo requests (batched)' '
36-
run_timeout 20 flux ping --pad 10240 --count 1024 --batch --interval 0 0
36+
run_timeout 20 flux ping --pad 10K --count 1024 --batch --interval 0 0
3737
'
3838

3939
test_expect_success 'ping --rank 1 works' '

0 commit comments

Comments
 (0)