Skip to content

Commit 2a7b71f

Browse files
committed
fix: enable testifylint on pkg/api
1 parent 9c653b8 commit 2a7b71f

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

pkg/api/v1/resource/helpers_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ import (
2020
"testing"
2121

2222
"github.com/stretchr/testify/assert"
23+
"github.com/stretchr/testify/require"
2324

2425
v1 "k8s.io/api/core/v1"
2526
"k8s.io/apimachinery/pkg/api/equality"
@@ -109,7 +110,7 @@ func TestGetResourceRequest(t *testing.T) {
109110
as := assert.New(t)
110111
for idx, tc := range cases {
111112
actual := GetResourceRequest(tc.pod, tc.resourceName)
112-
as.Equal(actual, tc.expectedValue, "expected test case [%d] %v: to return %q; got %q instead", idx, tc.cName, tc.expectedValue, actual)
113+
as.Equal(tc.expectedValue, actual, "expected test case [%d] %v: to return %q; got %q instead", idx, tc.cName, tc.expectedValue, actual)
113114
}
114115
}
115116

@@ -259,9 +260,9 @@ func TestExtractResourceValue(t *testing.T) {
259260
for idx, tc := range cases {
260261
actual, err := ExtractResourceValueByContainerName(tc.fs, tc.pod, tc.cName)
261262
if tc.expectedError != nil {
262-
as.Equal(tc.expectedError, err, "expected test case [%d] to fail with error %v; got %v", idx, tc.expectedError, err)
263+
require.EqualError(t, err, tc.expectedError.Error(), "expected test case [%d] to fail with error %v; got %v", idx, tc.expectedError, err)
263264
} else {
264-
as.Nil(err, "expected test case [%d] to not return an error; got %v", idx, err)
265+
require.NoError(t, err, "expected test case [%d] to not return an error; got %v", idx, err)
265266
as.Equal(tc.expectedValue, actual, "expected test case [%d] to return %q; got %q instead", idx, tc.expectedValue, actual)
266267
}
267268
}

0 commit comments

Comments
 (0)