Use text snapshots by default for scalars

Author: sebastiandedeyne

src/Drivers/TextDriver.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace Spatie\Snapshots\Drivers;
+
+use Spatie\Snapshots\Driver;
+use PHPUnit\Framework\Assert;
+
+class TextDriver implements Driver
+{
+    public function serialize($data): string
+    {
+        return $data;
+    }
+
+    public function extension(): string
+    {
+        return 'txt';
+    }
+
+    public function match($expected, $actual)
+    {
+        Assert::assertEquals($expected, $this->serialize($actual));
+    }
+}

src/MatchesSnapshots.php

@@ -2,14 +2,15 @@
 
 namespace Spatie\Snapshots;
 
-use PHPUnit\Framework\ExpectationFailedException;
 use ReflectionClass;
 use ReflectionObject;
+use Spatie\Snapshots\Drivers\XmlDriver;
 use Spatie\Snapshots\Drivers\HtmlDriver;
 use Spatie\Snapshots\Drivers\JsonDriver;
-use Spatie\Snapshots\Drivers\ObjectDriver;
-use Spatie\Snapshots\Drivers\XmlDriver;
+use Spatie\Snapshots\Drivers\TextDriver;
 use Spatie\Snapshots\Drivers\YamlDriver;
+use Spatie\Snapshots\Drivers\ObjectDriver;
+use PHPUnit\Framework\ExpectationFailedException;
 
 trait MatchesSnapshots
 {
@@ -43,7 +44,19 @@ public function markTestIncompleteIfSnapshotsHaveChanged()
 
     public function assertMatchesSnapshot($actual, Driver $driver = null)
     {
-        $this->doSnapshotAssertion($actual, $driver ?? new ObjectDriver());
+        if (! is_null($driver)) {
+            $this->doSnapshotAssertion($actual, $driver);
+
+            return;
+        }
+
+        if (is_string($actual) || is_int($actual) || is_float($actual)) {
+            $this->doSnapshotAssertion($actual, new TextDriver());
+
+            return;
+        }
+
+        $this->doSnapshotAssertion($actual, new ObjectDriver());
     }
 
     public function assertMatchesHtmlSnapshot($actual)
@@ -82,6 +95,16 @@ public function assertMatchesFileSnapshot($file)
         $this->doFileSnapshotAssertion($file);
     }
 
+    public function assertMatchesObjectSnapshot($actual)
+    {
+        $this->assertMatchesSnapshot($actual, new ObjectDriver());
+    }
+
+    public function assertMatchesTextSnapshot($actual)
+    {
+        $this->assertMatchesSnapshot($actual, new TextDriver());
+    }
+
     /**
      * Determines the snapshot's id. By default, the test case's class and
      * method names are used.

tests/Integration/MatchesSnapshotTest.php

@@ -34,8 +34,8 @@ public function it_can_create_a_snapshot_from_a_string()
         });
 
         $this->assertSnapshotMatchesExample(
-            'MatchesSnapshotTest__it_can_create_a_snapshot_from_a_string__1.yml',
-            'string_snapshot.yml'
+            'MatchesSnapshotTest__it_can_create_a_snapshot_from_a_string__1.txt',
+            'string_snapshot.txt'
         );
     }
 
@@ -285,8 +285,8 @@ public function it_can_update_a_string_snapshot()
         });
 
         $this->assertSnapshotMatchesExample(
-            'MatchesSnapshotTest__it_can_update_a_string_snapshot__1.yml',
-            'string_snapshot.yml'
+            'MatchesSnapshotTest__it_can_update_a_string_snapshot__1.txt',
+            'string_snapshot.txt'
         );
     }