CallbackMatcher should not be executed for internal function names

coduo · Feb 5, 2015 · 1e21f66 · 1e21f66
1 parent 272f536
commit 1e21f66
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/Coduo/PHPMatcher/Matcher/CallbackMatcher.php b/src/Coduo/PHPMatcher/Matcher/CallbackMatcher.php
@@ -17,6 +17,6 @@ public function match($value, $pattern)
      */
     public function canMatch($pattern)
     {
-        return is_callable($pattern);
+        return is_object($pattern) && is_callable($pattern);
     }
 }
diff --git a/tests/Coduo/PHPMatcher/Matcher/CallbackMatcherTest.php b/tests/Coduo/PHPMatcher/Matcher/CallbackMatcherTest.php
@@ -15,6 +15,7 @@ function test_negative_can_match()
     {
         $matcher = new CallbackMatcher();
         $this->assertFalse($matcher->canMatch(new \DateTime()));
+        $this->assertFalse($matcher->canMatch('SIN'));
     }
 
     function test_positive_matches()