Skip to content

Commit da87f82

Browse files
committed
pkg: Use testify/require instead of testify/assert
Signed-off-by: Stephen Augustus <[email protected]>
1 parent 83f6990 commit da87f82

File tree

2 files changed

+8
-10
lines changed

2 files changed

+8
-10
lines changed

pkg/kepctl/create_test.go

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import (
2323
"path/filepath"
2424
"testing"
2525

26-
"github.com/stretchr/testify/assert"
2726
"github.com/stretchr/testify/require"
2827

2928
"k8s.io/enhancements/api"
@@ -98,19 +97,19 @@ func TestWriteKep(t *testing.T) {
9897
err = c.writeKEP(&p, &tc.opts.CommonArgs)
9998

10099
if tc.expectError {
101-
assert.Error(t, err)
100+
require.Error(t, err)
102101
} else {
103-
assert.NoError(t, err)
102+
require.NoError(t, err)
104103

105104
computedPath := filepath.Join(tempDir, tc.expectedPath)
106105
dirStat, err := os.Stat(computedPath)
107-
assert.NoError(t, err)
106+
require.NoError(t, err)
108107
require.NotNil(t, dirStat)
109-
assert.True(t, dirStat.IsDir())
108+
require.True(t, dirStat.IsDir())
110109
p := filepath.Join(computedPath, "kep.yaml")
111110
fileStat, err := os.Stat(p)
112-
assert.NoError(t, err)
113-
assert.NotNil(t, fileStat)
111+
require.NoError(t, err)
112+
require.NotNil(t, fileStat)
114113
}
115114
})
116115
}

pkg/kepctl/kepctl_test.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ import (
2121
"io/ioutil"
2222
"testing"
2323

24-
"github.com/stretchr/testify/assert"
2524
"github.com/stretchr/testify/require"
2625
"gopkg.in/yaml.v2"
2726

@@ -55,9 +54,9 @@ func TestValidate(t *testing.T) {
5554
require.NoError(t, err)
5655
err = validateKEP(&p)
5756
if tc.err == nil {
58-
assert.NoError(t, err)
57+
require.NoError(t, err)
5958
} else {
60-
assert.EqualError(t, err, tc.err.Error())
59+
require.EqualError(t, err, tc.err.Error())
6160
}
6261
})
6362
}

0 commit comments

Comments
 (0)