#71 Null value matching problem

Author: Michal Szczur

src/Matcher/ArrayMatcher.php

@@ -12,7 +12,7 @@ final class ArrayMatcher extends Matcher
     const UNBOUNDED_PATTERN = '@...@';
 
     /**
-     * @var PropertyMatcher
+     * @var ValueMatcher
      */
     private $propertyMatcher;
 
@@ -180,7 +180,7 @@ private function valueMatchPattern($value, $pattern)
      */
     private function valueExist($path, array $haystack)
     {
-        return null !== $this->getPropertyAccessor()->getValue($haystack, $path);
+        return $this->getPropertyAccessor()->isReadable($haystack, $path);
     }
 
     /**
@@ -203,6 +203,7 @@ private function getPropertyAccessor()
         }
 
         $accessorBuilder = PropertyAccess::createPropertyAccessorBuilder();
+        $accessorBuilder->enableExceptionOnInvalidIndex();
         $this->accessor = $accessorBuilder->getPropertyAccessor();
 
         return $this->accessor;

src/Matcher/NullMatcher.php

@@ -26,6 +26,6 @@ public function match($value, $pattern)
      */
     public function canMatch($pattern)
     {
-        return $pattern === null || (is_string($pattern) && 0 !== preg_match(self::MATCH_PATTERN, $pattern));
+        return is_null($pattern) || (is_string($pattern) && 0 !== preg_match(self::MATCH_PATTERN, $pattern));
     }
 }

tests/Matcher/ArrayMatcherTest.php

@@ -72,7 +72,7 @@ public function test_error_when_path_in_nested_pattern_does_not_exist()
         $array = array('foo' => array('bar' => array('baz' => 'bar value')));
         $pattern = array('foo' => array('bar' => array('faz' => 'faz value')));
 
-        $this->assertFalse($this->matcher->match($array,$pattern));
+        $this->assertFalse($this->matcher->match($array, $pattern));
 
         $this->assertEquals($this->matcher->getError(), 'There is no element under path [foo][bar][baz] in pattern.');
     }
@@ -124,7 +124,7 @@ public function test_matching_array_to_array_pattern()
 
     public static function positiveMatchData()
     {
-        $simpleArr =  array(
+        $simpleArr = array(
             'users' => array(
                 array(
                     'firstName' => 'Norbert',
@@ -141,7 +141,7 @@ public static function positiveMatchData()
             6.66
         );
 
-        $simpleArrPattern =  array(
+        $simpleArrPattern = array(
             'users' => array(
                 array(
                     'firstName' => '@string@',
@@ -169,7 +169,7 @@ public static function positiveMatchData()
 
     public static function negativeMatchData()
     {
-        $simpleArr =  array(
+        $simpleArr = array(
             'users' => array(
                 array(
                     'firstName' => 'Norbert',
@@ -186,7 +186,7 @@ public static function negativeMatchData()
             6.66
         );
 
-        $simpleDiff =  array(
+        $simpleDiff = array(
             'users' => array(
                 array(
                     'firstName' => 'Norbert',

tests/Matcher/JsonMatcherTest.php

@@ -161,6 +161,10 @@ public static function positiveMatches()
                 '{"null":null}',
                 '{"null":@null@}'
             ),
+            array(
+                '{"username":null,"some_data":"test"}',
+                '{"username":null, "some_data": @string@}'
+            ),
             array(
                 '{"null":null}',
                 '{"null":null}'
@@ -176,7 +180,7 @@ public static function positiveMatches()
             array(
                 '[{"name": "Norbert"},{"name":"Michał"},{"name":"Bob"},{"name":"Martin"}]',
                 '[{"name": "Norbert"},@...@]'
-            ),
+            )
         );
     }