Skip to content

Commit

Permalink
Get rid of succ & fail marks on rate summary
Browse files Browse the repository at this point in the history
  • Loading branch information
joanlopez committed Jun 4, 2024
1 parent 93649df commit 416a934
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
3 changes: 1 addition & 2 deletions js/summary.js
Original file line number Diff line number Diff line change
Expand Up @@ -233,8 +233,7 @@ function nonTrendMetricValueForSum(metric, timeUnit) {
case 'rate':
return [
humanizeValue(metric.values.rate, metric, timeUnit),
succMark + ' ' + metric.values.passes,
failMark + ' ' + metric.values.fails,
`${metric.values.passes} out of ${metric.values.passes + metric.values.fails}`,
]
default:
return ['[no data]']
Expand Down
4 changes: 2 additions & 2 deletions js/summary_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ const (
" ✓ check1\n" +
" ✗ check3\n ↳ 66% — ✓ 10 / ✗ 5\n" +
" ✗ check2\n ↳ 33% — ✓ 5 / ✗ 10\n\n" +
" ✓ checks......: 75.00% 45 ✗ 15 \n"
" ✓ checks......: 75.00% 45 out of 60\n"
countOut = " ✗ http_reqs...: 3 3/s\n"
gaugeOut = " vus.........: 1 min=1 max=1\n"
gaugeOut = " vus.........: 1 min=1 max=1\n"
trendOut = " ✗ my_trend....: avg=15ms min=10ms med=15ms max=20ms p(90)=19ms " +
"p(95)=19.5ms p(99.9)=19.99ms\n"
)
Expand Down

0 comments on commit 416a934

Please sign in to comment.