Merge pull request #146 from mallardduck/normalize-all-windows-output

Add tests for Text Driver and Windows EOL fixes

Author: freekmurze

src/Drivers/TextDriver.php

@@ -9,6 +9,11 @@ class TextDriver implements Driver
 {
     public function serialize($data): string
     {
+        // Normalize line endings for cross-platform tests.
+        if (PHP_OS_FAMILY === 'Windows') {
+            $data = implode("\n", explode("\r\n", $data));
+        }
+
         return $data;
     }
 

tests/Unit/Drivers/TextDriverTest.php

@@ -0,0 +1,54 @@
+<?php
+
+namespace Spatie\Snapshots\Test\Unit\Drivers;
+
+use PHPUnit\Framework\TestCase;
+use Spatie\Snapshots\Drivers\TextDriver;
+use Spatie\Snapshots\Exceptions\CantBeSerialized;
+
+class TextDriverTest extends TestCase
+{
+    /** @test */
+    public function it_can_serialize_laravel_route_list()
+    {
+        $driver = new TextDriver();
+
+        $expected = implode("\n", [
+            '',
+            '  GET|HEAD       / ..................................................... index',
+            '',
+            '                                                            Showing [1] routes'
+        ]);
+
+        $this->assertEquals($expected, $driver->serialize(<<<EOF
+
+  GET|HEAD       / ..................................................... index
+
+                                                            Showing [1] routes
+EOF));
+    }
+
+    /** @test */
+    public function it_can_serialize_when_given_OS_dependant_line_endings()
+    {
+        $driver = new TextDriver();
+
+        $expected = implode("\n", [
+            '',
+            '  GET|HEAD       / ..................................................... index',
+            '',
+            '                                                            Showing [1] routes'
+        ]);
+
+        // Due to using PHP_EOL this should fail (conditionally) when run on windows
+        $actual = implode(PHP_EOL, [
+            '',
+            '  GET|HEAD       / ..................................................... index',
+            '',
+            '                                                            Showing [1] routes'
+        ]);
+
+        $this->assertEquals($expected, $driver->serialize($actual));
+    }
+
+}