Merge pull request #53 from Kanti/patch-1

[BUGFIX] sanitize filename

Author: freekmurze

src/Snapshot.php

@@ -40,7 +40,14 @@ public function id(): string
 
     public function filename(): string
     {
-        return $this->id.'.'.$this->driver->extension();
+        $file = $this->id.'.'.$this->driver->extension();
+        // Remove anything which isn't a word, whitespace, number
+        // or any of the following caracters -_~,;[]().
+        $file = preg_replace("([^\w\s\d\-_~,;\[\]\(\).])", '', $file);
+        // Remove any runs of periods
+        $file = preg_replace("([\.]{2,})", '', $file);
+
+        return $file;
     }
 
     public function exists(): bool

tests/Unit/SnapshotTest.php

@@ -43,4 +43,17 @@ public function it_has_a_filename_based_on_its_id_and_its_drivers_extension()
 
         $this->assertEquals('abc.php', $snapshot->filename());
     }
+
+    /** @test */
+    public function it_has_a_filename_which_is_valid_on_all_systems()
+    {
+        $this->driver
+            ->expects($this->once())
+            ->method('extension')
+            ->willReturn('php');
+
+        $snapshot = new Snapshot('ClassTest__testOne with data set "Empty"', $this->filesystem, $this->driver);
+
+        $this->assertEquals('ClassTest__testOne with data set Empty.php', $snapshot->filename());
+    }
 }