wip

Author: sebastiandedeyne

tests/MatchesSnapshots.php

@@ -10,24 +10,24 @@ public function assertMatchesSnapshot($serializable)
     {
         $this->snapshotCount++;
 
-        $snapshot = Snapshot::forTestMethod(
+        $snapshotHandler = SnapshotHandler::forTestMethod(
             debug_backtrace(DEBUG_BACKTRACE_PROVIDE_OBJECT, 2)[1],
             $this->snapshotCount
         );
 
-        if (! $snapshot->exists()) {
-            $snapshot->create($this->serializeForSnapshot($serializable));
+        if (! $snapshotHandler->exists()) {
+            $snapshotHandler->create($this->serializeForSnapshot($serializable));
 
-            return $this->markTestIncomplete("Snapshot created for {$snapshot->id()}");
+            return $this->markTestIncomplete("Snapshot created for {$snapshotHandler->id()}");
         }
 
-        if ($this->updateSnapshots()) {
-            $snapshot->update($this->serializeForSnapshot($serializable));
+        if ($this->shouldUpdateSnapshots()) {
+            $snapshotHandler->update($this->serializeForSnapshot($serializable));
 
-            return $this->markTestIncomplete("Snapshot updated for {$snapshot->id()}");
+            return $this->markTestIncomplete("Snapshot updated for {$snapshotHandler->id()}");
         }
 
-        return $this->assertEquals($snapshot->get(), $serializable);
+        return $this->assertEquals($snapshotHandler->get(), $serializable);
     }
 
     /** @after **/
@@ -36,27 +36,9 @@ public function resetSnapshotCounter()
         $this->snapshotCount = 0;
     }
 
-    protected function hasSnapshot($path, $id): bool
+    protected function shouldUpdateSnapshots(): bool
     {
-        if (! file_exists($path)) {
-            return false;
-        }
-
-        $snapshots = require $path;
-
-        return isset($snapshots[$id]);
-    }
-
-    protected function getSnapshot($path, $id)
-    {
-        $snapshots = require $path;
-
-        return $snapshots[$id];
-    }
-
-    protected function updateOrCreateSnapshot($path, $id, $serializable)
-    {
-
+        return getenv('UPDATE_SNAPSHOTS');
     }
 
     protected function serializeForSnapshot($serializable)

tests/Snapshot.php tests/SnapshotHandler.phptests/Snapshot.php renamed to tests/SnapshotHandler.php

@@ -4,7 +4,7 @@
 
 use ReflectionClass;
 
-class Snapshot
+class SnapshotHandler
 {
     /** @var string */
     protected $directory, $filename, $id;
@@ -18,7 +18,7 @@ private function __construct(string $directory, string $filename, string $id)
         $this->filename = $filename;
         $this->id = $id;
 
-        $this->availableSnapshots = (@include $this->path()) ?: [];
+        $this->availableSnapshots = @include $this->path() ?? [];
     }
 
     public static function forTestMethod($backtrace, $count): self
@@ -27,7 +27,7 @@ public static function forTestMethod($backtrace, $count): self
         $method = $backtrace['function'];
 
         $directory = dirname($class->getFileName()).'/__snapshots__';
-        $filename = $class->getShortName();
+        $filename = $class->getShortName().'.php';
         $id = "{$method} {$count}";
 
         return new self($directory, $filename, $id);
@@ -50,7 +50,7 @@ public function exists(): bool
 
     public function get()
     {
-        $this->availableSnapshots[$this->id];
+        return $this->availableSnapshots[$this->id];
     }
 
     public function create($serializedValue)
@@ -61,7 +61,7 @@ public function create($serializedValue)
             mkdir($this->directory);
         }
 
-        $contents = '<?php\n\n'.print_r($this->availableSnapshots, true);
+        $contents = '<?php'.PHP_EOL.PHP_EOL.'return '.var_export($this->availableSnapshots, true).';';
 
         file_put_contents($this->path(), $contents);
     }

tests/__snapshots__/ExampleTest

@@ -0,0 +1,5 @@
+<?php
+
+return array (
+  'true_is_true 1' => 'Foo',
+);