diff --git a/internal/cmd/config_consolidation_test.go b/internal/cmd/config_consolidation_test.go index 9b362203b..db9cfc881 100644 --- a/internal/cmd/config_consolidation_test.go +++ b/internal/cmd/config_consolidation_test.go @@ -379,10 +379,10 @@ func getConfigConsolidationTestCases() []configConsolidationTestCase { {opts{cli: []string{"--summary-trend-stats", ""}}, exp{}, func(t *testing.T, c Config) { assert.Equal(t, []string{}, c.SummaryTrendStats) }}, - {opts{cli: []string{"--summary-trend-stats", "coun"}}, exp{consolidationError: true}, nil}, - {opts{cli: []string{"--summary-trend-stats", "med,avg,p("}}, exp{consolidationError: true}, nil}, - {opts{cli: []string{"--summary-trend-stats", "med,avg,p(-1)"}}, exp{consolidationError: true}, nil}, - {opts{cli: []string{"--summary-trend-stats", "med,avg,p(101)"}}, exp{consolidationError: true}, nil}, + {opts{cli: []string{"--summary-trend-stats", "coun"}}, exp{cliReadError: true}, nil}, + {opts{cli: []string{"--summary-trend-stats", "med,avg,p("}}, exp{cliReadError: true}, nil}, + {opts{cli: []string{"--summary-trend-stats", "med,avg,p(-1)"}}, exp{cliReadError: true}, nil}, + {opts{cli: []string{"--summary-trend-stats", "med,avg,p(101)"}}, exp{cliReadError: true}, nil}, {opts{cli: []string{"--summary-trend-stats", "med,avg,p(99.999)"}}, exp{}, func(t *testing.T, c Config) { assert.Equal(t, []string{"med", "avg", "p(99.999)"}, c.SummaryTrendStats) }}, diff --git a/internal/cmd/options.go b/internal/cmd/options.go index c2ea741b2..5ee2e1c19 100644 --- a/internal/cmd/options.go +++ b/internal/cmd/options.go @@ -193,7 +193,7 @@ func getOptions(flags *pflag.FlagSet) (lib.Options, error) { if errSts != nil { return opts, errSts } - if _, errSts = metrics.GetResolversForTrendColumns(trendStats); err != nil { + if _, errSts = metrics.GetResolversForTrendColumns(trendStats); errSts != nil { return opts, errSts } opts.SummaryTrendStats = trendStats