Merge pull request #26 from Lomkit/feature/test-commands

✅ commands tests

Author: GautierDele

src/Console/Commands/ActionCommand.php

@@ -40,17 +40,6 @@ public function handle()
         parent::handle();
     }
 
-    /**
-     * Build the class with the given name.
-     *
-     * @param  string  $name
-     * @return string
-     */
-    protected function buildClass($name)
-    {
-        return parent::buildClass($name);
-    }
-
     /**
      * Get the stub file for the generator.
      *
@@ -61,6 +50,15 @@ protected function getStub()
         return $this->resolveStubPath('/stubs/rest/action.stub');
     }
 
+    protected function getPath($name)
+    {
+        if ($this->hasOption('path')) {
+            return $this->option('path').'/'.$this->argument('name').'.php';
+        }
+
+        return parent::getPath($name);
+    }
+
     /**
      * Get the default namespace for the class.
      *

src/Console/Commands/BaseControllerCommand.php

@@ -62,6 +62,15 @@ protected function getStub()
         return $this->resolveStubPath('/stubs/rest/base-controller.stub');
     }
 
+    protected function getPath($name)
+    {
+        if ($this->hasOption('path')) {
+            return $this->option('path').'/'.$this->argument('name').'.php';
+        }
+
+        return parent::getPath($name);
+    }
+
     /**
      * Get the default namespace for the class.
      *

src/Console/Commands/BaseResourceCommand.php

@@ -42,16 +42,6 @@ class BaseResourceCommand extends GeneratorCommand implements PromptsForMissingI
      */
     protected $description = 'Create a new base resource class';
 
-    /**
-     * Execute the console command.
-     *
-     * @return bool|null
-     */
-    public function handle()
-    {
-        parent::handle();
-    }
-
     /**
      * Get the stub file for the generator.
      *
@@ -62,6 +52,15 @@ protected function getStub()
         return $this->resolveStubPath('/stubs/rest/base-resource.stub');
     }
 
+    protected function getPath($name)
+    {
+        if ($this->hasOption('path')) {
+            return $this->option('path').'/'.$this->argument('name').'.php';
+        }
+
+        return parent::getPath($name);
+    }
+
     /**
      * Get the default namespace for the class.
      *

src/Console/Commands/ControllerCommand.php

@@ -41,9 +41,11 @@ public function handle()
     {
         parent::handle();
 
-        $this->callSilent('rest:base-controller', [
-            'name' => 'Controller',
-        ]);
+        if (!$this->hasOption('path')) {
+            $this->callSilent('rest:base-controller', [
+                'name' => 'Controller',
+            ]);
+        }
     }
 
     /**
@@ -87,6 +89,15 @@ protected function getStub()
         return $this->resolveStubPath('/stubs/rest/controller.stub');
     }
 
+    protected function getPath($name)
+    {
+        if ($this->hasOption('path')) {
+            return $this->option('path').'/'.$this->argument('name').'.php';
+        }
+
+        return parent::getPath($name);
+    }
+
     /**
      * Get the default namespace for the class.
      *

src/Console/Commands/DocumentationCommand.php

@@ -55,7 +55,8 @@ public function handle()
      */
     protected function getPath($name)
     {
-        return public_path('vendor/rest/'.$name.'.json');
+
+        return $this->hasOption('path') ? $this->option('path').'/'.$name.'.json' : public_path('vendor/rest/'.$name.'.json');
     }
 
     protected function getStub()

src/Console/Commands/InstructionCommand.php

@@ -61,6 +61,15 @@ protected function getStub()
         return $this->resolveStubPath('/stubs/rest/instruction.stub');
     }
 
+    protected function getPath($name)
+    {
+        if ($this->hasOption('path')) {
+            return $this->option('path').'/'.$this->argument('name').'.php';
+        }
+
+        return parent::getPath($name);
+    }
+
     /**
      * Get the default namespace for the class.
      *

src/Console/Commands/ResourceCommand.php

@@ -40,9 +40,20 @@ public function handle()
     {
         parent::handle();
 
-        $this->callSilent('rest:base-resource', [
-            'name' => 'Resource',
-        ]);
+        if (!$this->hasOption('path')) {
+            $this->callSilent('rest:base-resource', [
+                'name' => 'Resource',
+            ]);
+        }
+    }
+
+    protected function getPath($name)
+    {
+        if ($this->hasOption('path')) {
+            return $this->option('path').'/'.$this->argument('name').'.php';
+        }
+
+        return parent::getPath($name);
     }
 
     /**

src/Console/Commands/ResponseCommand.php

@@ -65,6 +65,15 @@ protected function getStub()
         return $this->resolveStubPath('/stubs/rest/response.stub');
     }
 
+    protected function getPath($name)
+    {
+        if ($this->hasOption('path')) {
+            return $this->option('path').'/'.$this->argument('name').'.php';
+        }
+
+        return parent::getPath($name);
+    }
+
     /**
      * Get the default namespace for the class.
      *

src/Http/Response.php

@@ -146,7 +146,7 @@ public function toResponse($request) {
             'data' => $data ?? $this->map($this->responsable, $this->modelToResponse($this->responsable, $this->resource, $request->input())),
             'meta' => [
                 config('rest.automatic_gates.key') => [
-                    config('rest.automatic_gates.names.authorized_to_create') => Gate::allows('create', $this->responsable->first()::class),
+                    config('rest.automatic_gates.names.authorized_to_create') => Gate::allows('create', $this->resource::newModel()::class),
                 ]
             ]
         ];

tests/Feature/Commands/ActionCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Lomkit\Rest\Tests\Feature\Commands;
+
+use Illuminate\Bus\PendingBatch;
+use Illuminate\Support\Facades\Bus;
+use Illuminate\Support\Facades\Gate;
+use Lomkit\Rest\Actions\CallRestApiAction;
+use Lomkit\Rest\Tests\Feature\TestCase;
+use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
+use Lomkit\Rest\Tests\Support\Models\Model;
+use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
+
+class ActionCommandTest extends TestCase
+{
+    public function test_create_action_class(): void
+    {
+        $this->artisan('rest:action', ['name' => 'TestAction', '--path' => './.phpunit.cache'])->assertOk();
+
+        $this->assertFileExists('./.phpunit.cache/TestAction.php');
+    }
+}

tests/Feature/Commands/BaseControllerCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Lomkit\Rest\Tests\Feature\Commands;
+
+use Illuminate\Bus\PendingBatch;
+use Illuminate\Support\Facades\Bus;
+use Illuminate\Support\Facades\Gate;
+use Lomkit\Rest\Actions\CallRestApiAction;
+use Lomkit\Rest\Tests\Feature\TestCase;
+use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
+use Lomkit\Rest\Tests\Support\Models\Model;
+use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
+
+class BaseControllerCommandTest extends TestCase
+{
+    public function test_create_base_controller_class(): void
+    {
+        $this->artisan('rest:base-controller', ['name' => 'TestBaseController', '--path' => './.phpunit.cache'])->assertOk();
+
+        $this->assertFileExists('./.phpunit.cache/TestBaseController.php');
+    }
+}

tests/Feature/Commands/BaseResourceCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Lomkit\Rest\Tests\Feature\Commands;
+
+use Illuminate\Bus\PendingBatch;
+use Illuminate\Support\Facades\Bus;
+use Illuminate\Support\Facades\Gate;
+use Lomkit\Rest\Actions\CallRestApiAction;
+use Lomkit\Rest\Tests\Feature\TestCase;
+use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
+use Lomkit\Rest\Tests\Support\Models\Model;
+use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
+
+class BaseResourceCommandTest extends TestCase
+{
+    public function test_create_base_resource_class(): void
+    {
+        $this->artisan('rest:base-resource', ['name' => 'TestBaseResource', '--path' => './.phpunit.cache'])->assertOk();
+
+        $this->assertFileExists('./.phpunit.cache/TestBaseResource.php');
+    }
+}

tests/Feature/Commands/ControllerCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Lomkit\Rest\Tests\Feature\Commands;
+
+use Illuminate\Bus\PendingBatch;
+use Illuminate\Support\Facades\Bus;
+use Illuminate\Support\Facades\Gate;
+use Lomkit\Rest\Actions\CallRestApiAction;
+use Lomkit\Rest\Tests\Feature\TestCase;
+use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
+use Lomkit\Rest\Tests\Support\Models\Model;
+use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
+
+class ControllerCommandTest extends TestCase
+{
+    public function test_create_controller_class(): void
+    {
+        $this->artisan('rest:controller', ['name' => 'TestController', '--path' => './.phpunit.cache'])->assertOk();
+
+        $this->assertFileExists('./.phpunit.cache/TestController.php');
+    }
+}

tests/Feature/Commands/DocumentationCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Lomkit\Rest\Tests\Feature\Commands;
+
+use Illuminate\Bus\PendingBatch;
+use Illuminate\Support\Facades\Bus;
+use Illuminate\Support\Facades\Gate;
+use Lomkit\Rest\Actions\CallRestApiAction;
+use Lomkit\Rest\Tests\Feature\TestCase;
+use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
+use Lomkit\Rest\Tests\Support\Models\Model;
+use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
+
+class DocumentationCommandTest extends TestCase
+{
+    public function test_create_documentation_class(): void
+    {
+        $this->artisan('rest:documentation', ['--path' => './.phpunit.cache'])->assertOk();
+
+        $this->assertFileExists('./.phpunit.cache/openapi.json');
+    }
+}

tests/Feature/Commands/InstructionCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Lomkit\Rest\Tests\Feature\Commands;
+
+use Illuminate\Bus\PendingBatch;
+use Illuminate\Support\Facades\Bus;
+use Illuminate\Support\Facades\Gate;
+use Lomkit\Rest\Actions\CallRestApiAction;
+use Lomkit\Rest\Tests\Feature\TestCase;
+use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
+use Lomkit\Rest\Tests\Support\Models\Model;
+use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
+
+class InstructionCommandTest extends TestCase
+{
+    public function test_create_instruction_class(): void
+    {
+        $this->artisan('rest:instruction', ['name' => 'TestInstruction', '--path' => './.phpunit.cache'])->assertOk();
+
+        $this->assertFileExists('./.phpunit.cache/TestInstruction.php');
+    }
+}

tests/Feature/Commands/ResourceCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Lomkit\Rest\Tests\Feature\Commands;
+
+use Illuminate\Bus\PendingBatch;
+use Illuminate\Support\Facades\Bus;
+use Illuminate\Support\Facades\Gate;
+use Lomkit\Rest\Actions\CallRestApiAction;
+use Lomkit\Rest\Tests\Feature\TestCase;
+use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
+use Lomkit\Rest\Tests\Support\Models\Model;
+use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
+
+class ResourceCommandTest extends TestCase
+{
+    public function test_create_resource_class(): void
+    {
+        $this->artisan('rest:resource', ['name' => 'TestResource', '--path' => './.phpunit.cache'])->assertOk();
+
+        $this->assertFileExists('./.phpunit.cache/TestResource.php');
+    }
+}

tests/Feature/Commands/ResponseCommandTest.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace Lomkit\Rest\Tests\Feature\Commands;
+
+use Illuminate\Bus\PendingBatch;
+use Illuminate\Support\Facades\Bus;
+use Illuminate\Support\Facades\Gate;
+use Lomkit\Rest\Actions\CallRestApiAction;
+use Lomkit\Rest\Tests\Feature\TestCase;
+use Lomkit\Rest\Tests\Support\Database\Factories\ModelFactory;
+use Lomkit\Rest\Tests\Support\Models\Model;
+use Lomkit\Rest\Tests\Support\Policies\GreenPolicy;
+
+class ResponseCommandTest extends TestCase
+{
+    public function test_create_response_class(): void
+    {
+        $this->artisan('rest:response', ['name' => 'TestResponse', '--path' => './.phpunit.cache'])->assertOk();
+
+        $this->assertFileExists('./.phpunit.cache/TestResponse.php');
+    }
+}