fix: do not call methodWasCalled

removed isArgsEqual as well

Author: gopherjit

mock/mock.go

@@ -712,30 +712,15 @@ func (m *Mock) AssertNotCalled(t TestingT, methodName string, arguments ...inter
 	return true
 }
 
-// IsMethodCallable checking that the method can be called
-// If the method was called more than `Repeatability` return false
+// IsMethodCallable returns true if the methodName and arguments are currently expected
 func (m *Mock) IsMethodCallable(_ TestingT, methodName string, arguments ...interface{}) bool {
 	m.mutex.Lock()
 	defer m.mutex.Unlock()
 	index, _ := m.findExpectedCall(methodName, arguments...)
 	if index != -1 {
 		return true
 	}
-	// return false if call was not expected
-	return m.methodWasCalled(methodName, arguments)
-}
-
-// isArgsEqual compares arguments
-func isArgsEqual(expected Arguments, args []interface{}) bool {
-	if len(expected) != len(args) {
-		return false
-	}
-	for i, v := range args {
-		if !reflect.DeepEqual(expected[i], v) {
-			return false
-		}
-	}
-	return true
+	return false
 }
 
 func (m *Mock) methodWasCalled(methodName string, expected []interface{}) bool {

mock/mock_test.go

@@ -1731,19 +1731,6 @@ func Test_Mock_IsMethodCallable(t *testing.T) {
 	})
 }
 
-func TestIsArgsEqual(t *testing.T) {
-	var expected = Arguments{5, 3, 4, 6, 7, 2}
-
-	// Copy elements 1 to 5
-	args := append(([]interface{})(nil), expected[1:]...)
-	args[2] = expected[1]
-	assert.False(t, isArgsEqual(expected, args))
-
-	// Clone
-	arr := append(([]interface{})(nil), expected...)
-	assert.True(t, isArgsEqual(expected, arr))
-}
-
 func Test_Mock_AssertOptional(t *testing.T) {
 	// Optional called
 	var ms1 = new(TestExampleImplementation)