Merge pull request #253 from nhaarman/check-behavior

Adhere to Mockito.argThat contract in check()

Author: nhaarman

mockito-kotlin/src/main/kotlin/com/nhaarman/mockitokotlin2/Verification.kt

@@ -241,7 +241,13 @@ If you are trying to verify an argument to be null, use `isNull()`.
 If you are using `check` as part of a stubbing, use `argThat` or `argForWhich` instead.
 """.trimIndent()
         )
-        predicate(arg)
-        true
+
+        try {
+            predicate(arg)
+            true
+        } catch (e: Error) {
+            e.printStackTrace()
+            false
+        }
     } ?: createInstance(T::class)
 }

tests/src/test/kotlin/test/MatchersTest.kt

@@ -129,6 +129,21 @@ class MatchersTest : TestBase() {
         }
     }
 
+    @Test
+    fun checkProperlyFails() {
+        mock<Methods>().apply {
+            closedList(listOf(Closed(), Closed()))
+
+            expectErrorWithMessage("Argument(s) are different!") on {
+                verify(this).closedList(
+                      check {
+                          expect(it.size).toBe(1)
+                      }
+                )
+            }
+        }
+    }
+
     @Test
     fun checkWithNullArgument_throwsError() {
         mock<Methods>().apply {