Issue #4. Throw exception if component or mixin already exists.

Author: zachleigh

src/Commands/MakeComponent.php

@@ -33,6 +33,8 @@ public function handle()
 
         $fullPath = resource_path("{$path}/{$name}");
 
+        $this->checkFileExists($filesystem, $fullPath, $name);
+
         $stub = $this->getStub($filesystem);
 
         $filesystem->put($fullPath, $stub);

src/Commands/MakeMixin.php

@@ -33,6 +33,8 @@ public function handle()
 
         $fullPath = resource_path("{$path}/{$name}");
 
+        $this->checkFileExists($filesystem, $fullPath, $name);
+
         $stub = $this->getStub($filesystem);
 
         $filesystem->put($fullPath, $stub);

src/Commands/VueGeneratorsCommand.php

@@ -5,6 +5,7 @@
 use VueGenerators\Paths;
 use Illuminate\Console\Command;
 use Illuminate\Filesystem\Filesystem;
+use VueGenerators\Exceptions\ResourceAlreadyExists;
 
 class VueGeneratorsCommand extends Command
 {
@@ -30,4 +31,11 @@ protected function createPath(Filesystem $filesystem, $type)
 
         return $path;
     }
+
+    protected function checkFileExists(Filesystem $filesystem, $path, $name)
+    {
+        if ($filesystem->exists($path)) {
+            throw ResourceAlreadyExists::fileExists($name);
+        }
+    }
 }

src/Exceptions/ResourceAlreadyExists.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace VueGenerators\Exceptions;
+
+use Exception;
+
+class ResourceAlreadyExists extends Exception
+{
+    /**
+     * File of given name already exists at path.
+     *
+     * @param string $name
+     *
+     * @return static
+     */
+    public static function fileExists($name)
+    {
+        return new static(
+            "File {$name} already exists at path."
+        );
+    }
+}

tests/CommandTest.php

@@ -76,7 +76,7 @@ public function it_saves_component_file_to_specified_path()
     /**
      * @test
      */
-    public function it_saves_componets_to_path_set_in_config()
+    public function it_saves_components_to_path_set_in_config()
     {
         app()['config']->set('vue-generators.paths.components', 'custom/path');
 
@@ -91,6 +91,27 @@ public function it_saves_componets_to_path_set_in_config()
         $this->assertFileExists($file);
     }
 
+    /**
+     * @test
+     *
+     * @expectedException VueGenerators\Exceptions\ResourceAlreadyExists
+     * @expectedExceptionMessage File NewComponent.vue already exists at path.
+     */
+    public function it_doesnt_overwrite_components_that_already_exist()
+    {
+        $file = resource_path('assets/js/components/NewComponent.vue');
+
+        Artisan::call('vueg:component', [
+            'name'   => 'NewComponent'
+        ]);
+
+        $this->assertFileExists($file);
+
+        Artisan::call('vueg:component', [
+            'name'   => 'NewComponent'
+        ]);
+    }
+
     /**
      * @test
      */
@@ -175,4 +196,25 @@ public function it_saves_mixins_to_path_set_in_config()
 
         $this->assertFileExists($file);
     }
+
+    /**
+     * @test
+     *
+     * @expectedException VueGenerators\Exceptions\ResourceAlreadyExists
+     * @expectedExceptionMessage File NewMixin.js already exists at path.
+     */
+    public function it_doesnt_overwrite_mixin_that_already_exist()
+    {
+        $file = resource_path('assets/js/mixins/NewMixin.js');
+
+        Artisan::call('vueg:mixin', [
+            'name'   => 'NewMixin'
+        ]);
+
+        $this->assertFileExists($file);
+
+        Artisan::call('vueg:mixin', [
+            'name'   => 'NewMixin'
+        ]);
+    }
 }