From f3550cea474d43a911484960af6eec6643d91675 Mon Sep 17 00:00:00 2001 From: William Banfield Date: Wed, 11 May 2022 09:54:50 -0400 Subject: [PATCH] use want as name in test --- scripts/metricsgen/metricsdiff/metricsdiff.go | 3 ++- scripts/metricsgen/metricsdiff/metricsdiff_test.go | 8 ++++---- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/scripts/metricsgen/metricsdiff/metricsdiff.go b/scripts/metricsgen/metricsdiff/metricsdiff.go index 4bfdd38fa..bd0a40fde 100644 --- a/scripts/metricsgen/metricsdiff/metricsdiff.go +++ b/scripts/metricsgen/metricsdiff/metricsdiff.go @@ -48,7 +48,8 @@ type LabelDiff struct { func main() { flag.Parse() if flag.NArg() != 2 { - log.Fatalf("Incorrect number of arguments, expected 2 but received %d", len(os.Args)-1) + log.Fatalf("Usage is %s , got %d arguments", + filepath.Base(os.Args[0]), flag.NArg()) } fa, err := os.Open(os.Args[1]) if err != nil { diff --git a/scripts/metricsgen/metricsdiff/metricsdiff_test.go b/scripts/metricsgen/metricsdiff/metricsdiff_test.go index 0bd6d045e..3f1d8b0e4 100644 --- a/scripts/metricsgen/metricsdiff/metricsdiff_test.go +++ b/scripts/metricsgen/metricsdiff/metricsdiff_test.go @@ -15,7 +15,7 @@ func TestDiff(t *testing.T) { aContents string bContents string - expectedDiff metricsdiff.MetricsDiff + want metricsdiff.MetricsDiff }{ { name: "labels", @@ -25,7 +25,7 @@ func TestDiff(t *testing.T) { bContents: ` metric_one{label_three="content", label_four="content"} 0 `, - expectedDiff: metricsdiff.MetricsDiff{ + want: metricsdiff.MetricsDiff{ LabelDiffs: []metricsdiff.LabelDiff{ { MetricsName: "metric_one", @@ -43,7 +43,7 @@ func TestDiff(t *testing.T) { bContents: ` metric_two{label_two="content"} 0 `, - expectedDiff: metricsdiff.MetricsDiff{ + want: metricsdiff.MetricsDiff{ Adds: []string{"metric_two"}, Removes: []string{"metric_one"}, }, @@ -58,7 +58,7 @@ func TestDiff(t *testing.T) { require.NoError(t, err) md, err := metricsdiff.MetricsDiffFromReaders(bufA, bufB) require.NoError(t, err) - require.Equal(t, tc.expectedDiff, md) + require.Equal(t, tc.want, md) }) } }