Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add optional 'count' column to CLI summary output #1143

Merged
merged 4 commits into from Oct 30, 2019
Merged

Add optional 'count' column to CLI summary output #1143

merged 4 commits into from Oct 30, 2019

Commits on Oct 30, 2019

  1. feat(ui): add optional 'count' column to CLI summary output 

    This adds an optional count column for Trend metrics in the CLI summary
output, while refactoring parts of ui/summary.go for code style
(avoiding globals, explicit validation, etc.) and negligible performance
improvements (using a map for column lookups).

To enable it run e.g. `k6 run --summary-trend-stats 'avg,p(99.99),count' test.js`.

Closes #1087
    Ivan Mirić committed Oct 30, 2019
    Configuration menu
    Copy the full SHA
    da086e3 View commit details
    Browse the repository at this point in the history

  2. fix(cmd): respect empty SummaryTrendStats value 

    Resolves: b91229a#r35621359

This changes current behavior on `master`, but as discussed above, it makes
sense to respect whatever the user specifies, so this can be considered a fix.
    Ivan Mirić committed Oct 30, 2019
    Configuration menu
    Copy the full SHA
    16a5449 View commit details
    Browse the repository at this point in the history

  3. fix(ui): return error on out of range percentiles 

    Also add a few missing summary-trend-stats tests.

Resolves: #1143 (comment)
    Ivan Mirić committed Oct 30, 2019
    Configuration menu
    Copy the full SHA
    26f932a View commit details
    Browse the repository at this point in the history

  4. refactor(cmd): simplify applyDefault 

    Resolves: #1143 (comment)
    Ivan Mirić committed Oct 30, 2019
    Configuration menu
    Copy the full SHA
    51e88e0 View commit details
    Browse the repository at this point in the history