From 166114c4a04a0139fa0158ec8d4e9632feee5f62 Mon Sep 17 00:00:00 2001 From: Stephen Solka Date: Sat, 18 Jul 2020 20:23:35 -0400 Subject: [PATCH] prefer NoError/Error over Nil/NotNil Kubernetes-commit: 203679cc6105ea490e75af1efa83497b771d7d36 --- testing/fixture_test.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/testing/fixture_test.go b/testing/fixture_test.go index 912156d3..6e026756 100644 --- a/testing/fixture_test.go +++ b/testing/fixture_test.go @@ -283,7 +283,7 @@ func TestGetWithExactMatch(t *testing.T) { constructObject := func(s schema.GroupVersionResource, name, namespace string) (*unstructured.Unstructured, schema.GroupVersionResource) { obj := getArbitraryResource(s, name, namespace) gvks, _, err := scheme.ObjectKinds(obj) - assert.Nil(t, err) + assert.NoError(t, err) gvr, _ := meta.UnsafeGuessKindToResource(gvks[0]) return obj, gvr } @@ -298,11 +298,11 @@ func TestGetWithExactMatch(t *testing.T) { // Exact match _, err = o.Get(gvr, "", "node") - assert.Nil(t, err) + assert.NoError(t, err) // Unexpected namespace provided _, err = o.Get(gvr, "ns", "node") - assert.NotNil(t, err) + assert.Error(t, err) errNotFound := errors.NewNotFound(gvr.GroupResource(), "node") assert.EqualError(t, err, errNotFound.Error()) @@ -314,11 +314,11 @@ func TestGetWithExactMatch(t *testing.T) { // Exact match _, err = o.Get(gvr, "default", "pod") - assert.Nil(t, err) + assert.NoError(t, err) // Missing namespace _, err = o.Get(gvr, "", "pod") - assert.NotNil(t, err) + assert.Error(t, err) errNotFound = errors.NewNotFound(gvr.GroupResource(), "pod") assert.EqualError(t, err, errNotFound.Error()) }