Simplified driver API

Author: sebastiandedeyne

src/Driver.php

@@ -22,15 +22,6 @@ public function serialize($data): string;
      */
     public function extension(): string;
 
-    /**
-     * Load the contents of a snapshot at a path.
-     *
-     * @param string $path
-     *
-     * @return mixed
-     */
-    public function load(string $path);
-
     /**
      * Match an expectation with a snapshot's actual contents. Should throw an
      * `ExpectationFailedException` if it doesn't match. This happens by

src/Drivers/JsonDriver.php

@@ -23,11 +23,6 @@ public function extension(): string
         return '.json';
     }
 
-    public function load(string $path)
-    {
-        return file_get_contents($path);
-    }
-
     public function match($expected, $actual)
     {
         Assert::assertJson($expected);

src/Drivers/VarDriver.php

@@ -17,13 +17,10 @@ public function extension(): string
         return '.php';
     }
 
-    public function load(string $path)
-    {
-        return include $path;
-    }
-
     public function match($expected, $actual)
     {
-        Assert::assertEquals($expected, $actual);
+        $evaluated = eval(substr($expected, strlen('<?php ')));
+
+        Assert::assertEquals($evaluated, $actual);
     }
 }

src/Drivers/XmlDriver.php

@@ -30,11 +30,6 @@ public function extension(): string
         return '.xml';
     }
 
-    public function load(string $path)
-    {
-        return file_get_contents($path);
-    }
-
     public function match($expected, $actual)
     {
         Assert::assertEquals(Xml::load($expected), Xml::load($actual));

src/Filesystem.php

@@ -27,7 +27,7 @@ public function has(string $filename): bool
         return file_exists($this->path($filename));
     }
 
-    public function read(string $filename): bool
+    public function read(string $filename): string
     {
         return file_get_contents($this->path($filename));
     }

src/Snapshot.php

@@ -43,19 +43,14 @@ public function filename(): string
         return $this->id.$this->driver->extension();
     }
 
-    public function path(): string
-    {
-        return $this->filesystem->path($this->filename());
-    }
-
     public function exists(): bool
     {
         return $this->filesystem->has($this->filename());
     }
 
     public function assertMatches($actual)
     {
-        $this->driver->match($this->driver->load($this->path()), $actual);
+        $this->driver->match($this->filesystem->read($this->filename()), $actual);
     }
 
     public function create($actual)