Merge pull request #145 from cschindl/main

Add type safe json matching

Author: freekmurze

src/Drivers/JsonDriver.php

@@ -32,6 +32,6 @@ public function match($expected, $actual)
             $actual = json_decode($actual, true, 512, JSON_THROW_ON_ERROR);
         }
         $expected = json_decode($expected, true, 512, JSON_THROW_ON_ERROR);
-        Assert::assertEquals($expected, $actual);
+        Assert::assertJsonStringEqualsJsonString(json_encode($expected), json_encode($actual));
     }
 }

tests/Unit/Drivers/JsonDriverTest.php

@@ -2,6 +2,7 @@
 
 namespace Spatie\Snapshots\Test\Unit\Drivers;
 
+use PHPUnit\Framework\ExpectationFailedException;
 use PHPUnit\Framework\TestCase;
 use Spatie\Snapshots\Drivers\JsonDriver;
 use Spatie\Snapshots\Exceptions\CantBeSerialized;
@@ -120,4 +121,25 @@ public function it_can_not_serialize_resources()
 
         $driver->serialize($resource);
     }
+
+    /**
+     * @test
+     * @testWith ["{}", "{}", true]
+     *           ["{}", "{\"data\":1}", false]
+     *           ["{\"data\":1}", "{\"data\":1}", true]
+     *           ["{\"data\":1}", "{\"data\":\"1\"}", false]
+     */
+    public function it_can_match_json_strings(string $expected, string $actual, bool $assertion)
+    {
+        $driver = new JsonDriver();
+
+        try {
+            $driver->match($expected, $actual);
+            $status = true;
+        } catch (ExpectationFailedException $th) {
+            $status = false;
+        }
+
+        $this->assertSame($assertion, $status);
+    }
 }