mirror of https://github.com/stretchr/testify.git
mock: refactor TestIsArgsEqual
Refactor tests to use 'append' to build copies of slices instead of manual loops. Thanks golangci-lint.pull/1444/head
parent
862e41010c
commit
c740480fc1
|
@ -1616,17 +1616,14 @@ func Test_Mock_IsMethodCallable(t *testing.T) {
|
||||||
|
|
||||||
func TestIsArgsEqual(t *testing.T) {
|
func TestIsArgsEqual(t *testing.T) {
|
||||||
var expected = Arguments{5, 3, 4, 6, 7, 2}
|
var expected = Arguments{5, 3, 4, 6, 7, 2}
|
||||||
var args = make([]interface{}, 5)
|
|
||||||
for i := 1; i < len(expected); i++ {
|
// Copy elements 1 to 5
|
||||||
args[i-1] = expected[i]
|
args := append(([]interface{})(nil), expected[1:]...)
|
||||||
}
|
|
||||||
args[2] = expected[1]
|
args[2] = expected[1]
|
||||||
assert.False(t, isArgsEqual(expected, args))
|
assert.False(t, isArgsEqual(expected, args))
|
||||||
|
|
||||||
var arr = make([]interface{}, 6)
|
// Clone
|
||||||
for i := 0; i < len(expected); i++ {
|
arr := append(([]interface{})(nil), expected...)
|
||||||
arr[i] = expected[i]
|
|
||||||
}
|
|
||||||
assert.True(t, isArgsEqual(expected, arr))
|
assert.True(t, isArgsEqual(expected, arr))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue