From 9b10892092e116ace2d22a240af3316902f3fab2 Mon Sep 17 00:00:00 2001 From: Matt Harden Date: Fri, 10 Dec 2021 18:47:43 -0800 Subject: [PATCH] Add a missing name to the *testing.T argument. When using t.Run, the subtest must take a named *testing.T argument. Otherwise, inner calls to t.Error etc. are invoked on the outer test object, which is pretty much always wrong. Signed-off-by: Matt Harden --- proc_psi_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/proc_psi_test.go b/proc_psi_test.go index 3903a18b1..bd6fb09aa 100644 --- a/proc_psi_test.go +++ b/proc_psi_test.go @@ -19,7 +19,7 @@ import ( ) func TestPSIStats(t *testing.T) { - t.Run("fake", func(*testing.T) { + t.Run("fake", func(t *testing.T) { stats, err := getProcFixtures(t).PSIStatsForResource("fake") if err == nil { t.Fatal("fake resource does not have PSI statistics")