Skip to content

Commit 9107fa3

Browse files
fix: changes from code review
* Replace last assert directive with require in the test. * Change copyright notice.
1 parent 0571a2f commit 9107fa3

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

kustomize/commands/edit/remove/removeconfigmap.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright 2022 The Kubernetes Authors.
1+
// Copyright 2023 The Kubernetes Authors.
22
// SPDX-License-Identifier: Apache-2.0
33

44
package remove

kustomize/commands/edit/remove/removeconfigmap_test.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright 2022 The Kubernetes Authors.
1+
// Copyright 2023 The Kubernetes Authors.
22
// SPDX-License-Identifier: Apache-2.0
33

44
package remove //nolint:testpackage
@@ -8,8 +8,6 @@ import (
88
"testing"
99

1010
"github.com/stretchr/testify/require"
11-
12-
"github.com/stretchr/testify/assert"
1311
testutils_test "sigs.k8s.io/kustomize/kustomize/v5/commands/internal/testutils"
1412
"sigs.k8s.io/kustomize/kyaml/filesys"
1513
)
@@ -105,7 +103,7 @@ kind: Kustomization
105103

106104
if tc.expectedErr != "" {
107105
require.Error(t, err)
108-
assert.Contains(t, err.Error(), tc.expectedErr)
106+
require.Contains(t, err.Error(), tc.expectedErr)
109107
return
110108
}
111109

0 commit comments

Comments
 (0)