Bug fixes

Author: AlexMordred

src/FileUploader.php

@@ -123,12 +123,14 @@ public function upload($path = '', $storage = 'public')
      */
     public function uploadAs($filename, $path = '', $storage = 'public')
     {
-        $filename = $filename
-            ? $filename . '.' . $this->file->getClientOriginalExtension()
-            : $this->file->hashName();
+        $filename = $filename ?: $this->file->hashName();
 
         if ($this->image) {
-            $imagePath = $path . $filename;
+            $imagePath = "{$path}/{$filename}";
+
+            if (!Storage::disk($storage)->exists($path)) {
+                Storage::disk($storage)->makeDirectory($path);
+            }
 
             $this->image->save(Storage::disk($storage)->path($imagePath));
 

tests/FileUploaderTest.php

@@ -31,11 +31,11 @@ public function testItUploadsFilesUnderSpecifiedNames()
 
         $file = UploadedFile::fake()->create('file.pdf', 100);
 
-        $filename = 'document';
+        $filename = 'document.pdf';
 
         $path = FileUploader::make($file)->uploadAs($filename);
 
-        $this->assertEquals("{$filename}.pdf", $path);
+        $this->assertEquals($filename, $path);
         Storage::disk('public')->assertExists($path);
     }
 
@@ -84,12 +84,12 @@ public function testItReplacesTheOldFileWithTheNewFileUnderSpecifiedName()
 
         $newFile = UploadedFile::fake()->create('new_file.pdf', 100);
 
-        $newFilename = 'new_document';
+        $newFilename = 'new_document.pdf';
         $newPath = FileUploader::make($newFile)->replaceAs($oldPath, $newFilename);
 
         Storage::disk('public')->assertMissing($oldPath);
 
-        $this->assertEquals("{$newFilename}.pdf", $newPath);
+        $this->assertEquals($newFilename, $newPath);
         Storage::disk('public')->assertExists($newPath);
     }