Merge pull request #3 from demvsystems/fix-double-separator

fix double separator in filepath

Author: christianhaberland

src/TmpDirRegistry.php

@@ -97,7 +97,7 @@ public function createFileInSystemTmp(string $dir, string $filename): string
             throw new RuntimeException('Can\'t read info from filename.');
         }
 
-        $filepath = $dir . DIRECTORY_SEPARATOR . sprintf('%s_%s.%s', $info['filename'], $this->getUniqid(), $info['extension']);
+        $filepath = self::addSeperatorIfNecessary($dir . DIRECTORY_SEPARATOR) . sprintf('%s_%s.%s', $info['filename'], $this->getUniqid(), $info['extension']);
         $handle   = fopen($filepath, 'wb+');
         fclose($handle);
         if (!file_exists($filepath)) {

tests/TmpDirRegistryTest.php

@@ -95,6 +95,15 @@ public function testCreateFileInSystemTmpFilename(): void
         self::assertTrue(strlen($filename) < strlen($uniqueFilename));
     }
 
+    public function testCreateFileInSystemTmpHasOnlyOneSeperator(): void
+    {
+        $dirname        = $this->instance->createDirInSystemTmp('testdirectory');
+        $filename       = 'testfilename.txt';
+        $uniqueFilename = $this->instance->createFileInSystemTmp($dirname, $filename);
+
+        self::assertNotContains('//', $uniqueFilename);
+    }
+
     public function testCreateFileInSystemTmp(): void
     {
         $dirname        = 'testdirectory';