Add support for other verbs.

rico · rico · commit 26e7bbf1e933 · 2023-11-23T18:43:21.000+01:00
diff --git a/src/Attributes/Delete.php b/src/Attributes/Delete.php
@@ -4,7 +4,7 @@
 
 use Attribute;
 
-#[Attribute(Attribute::TARGET_METHOD)]
+#[Attribute(Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
 class Delete extends Route
 {
     public function __construct(
diff --git a/src/Attributes/Options.php b/src/Attributes/Options.php
@@ -4,7 +4,7 @@
 
 use Attribute;
 
-#[Attribute(Attribute::TARGET_METHOD)]
+#[Attribute(Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
 class Options extends Route
 {
     public function __construct(
diff --git a/src/Attributes/Patch.php b/src/Attributes/Patch.php
@@ -4,7 +4,7 @@
 
 use Attribute;
 
-#[Attribute(Attribute::TARGET_METHOD)]
+#[Attribute(Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
 class Patch extends Route
 {
     public function __construct(
diff --git a/src/Attributes/Post.php b/src/Attributes/Post.php
@@ -4,7 +4,7 @@
 
 use Attribute;
 
-#[Attribute(Attribute::TARGET_METHOD)]
+#[Attribute(Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
 class Post extends Route
 {
     public function __construct(
diff --git a/src/Attributes/Put.php b/src/Attributes/Put.php
@@ -4,7 +4,7 @@
 
 use Attribute;
 
-#[Attribute(Attribute::TARGET_METHOD)]
+#[Attribute(Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
 class Put extends Route
 {
     public function __construct(
diff --git a/tests/AttributeTests/DeleteAttributeTest.php b/tests/AttributeTests/DeleteAttributeTest.php
@@ -3,6 +3,7 @@
 namespace Spatie\RouteAttributes\Tests\AttributeTests;
 
 use Spatie\RouteAttributes\Tests\TestCase;
+use Spatie\RouteAttributes\Tests\TestClasses\Controllers\DeleteMultipleTestController;
 use Spatie\RouteAttributes\Tests\TestClasses\Controllers\DeleteTestController;
 
 class DeleteAttributeTest extends TestCase
@@ -16,4 +17,15 @@ public function it_can_register_a_delete_route()
             ->assertRegisteredRoutesCount(1)
             ->assertRouteRegistered(DeleteTestController::class, 'myDeleteMethod', 'delete', 'my-delete-method');
     }
+
+    /** @test */
+    public function it_can_register_multiple_delete_routes()
+    {
+        $this->routeRegistrar->registerClass(DeleteMultipleTestController::class);
+
+        $this
+            ->assertRegisteredRoutesCount(2)
+            ->assertRouteRegistered(DeleteMultipleTestController::class, 'myDeleteMethod', 'delete', 'my-delete-method')
+            ->assertRouteRegistered(DeleteMultipleTestController::class, 'myDeleteMethod', 'delete', 'my-other-delete-method');
+    }
 }
diff --git a/tests/AttributeTests/OptionsAttributeTest.php b/tests/AttributeTests/OptionsAttributeTest.php
@@ -3,6 +3,7 @@
 namespace Spatie\RouteAttributes\Tests\AttributeTests;
 
 use Spatie\RouteAttributes\Tests\TestCase;
+use Spatie\RouteAttributes\Tests\TestClasses\Controllers\OptionsMultipleTestController;
 use Spatie\RouteAttributes\Tests\TestClasses\Controllers\OptionsTestController;
 
 class OptionsAttributeTest extends TestCase
@@ -16,4 +17,15 @@ public function it_can_register_a_options_route()
             ->assertRegisteredRoutesCount(1)
             ->assertRouteRegistered(OptionsTestController::class, 'myOptionsMethod', 'options', 'my-options-method');
     }
+
+    /** @test */
+    public function it_can_register_multiple_options_routes()
+    {
+        $this->routeRegistrar->registerClass(OptionsMultipleTestController::class);
+
+        $this
+            ->assertRegisteredRoutesCount(2)
+            ->assertRouteRegistered(OptionsMultipleTestController::class, 'myOptionsMethod', 'options', 'my-options-method')
+            ->assertRouteRegistered(OptionsMultipleTestController::class, 'myOptionsMethod', 'options', 'my-other-options-method');
+    }
 }
diff --git a/tests/AttributeTests/PatchAttributeTest.php b/tests/AttributeTests/PatchAttributeTest.php
@@ -3,6 +3,7 @@
 namespace Spatie\RouteAttributes\Tests\AttributeTests;
 
 use Spatie\RouteAttributes\Tests\TestCase;
+use Spatie\RouteAttributes\Tests\TestClasses\Controllers\PatchMultipleTestController;
 use Spatie\RouteAttributes\Tests\TestClasses\Controllers\PatchTestController;
 
 class PatchAttributeTest extends TestCase
@@ -16,4 +17,15 @@ public function it_can_register_a_patch_route()
             ->assertRegisteredRoutesCount(1)
             ->assertRouteRegistered(PatchTestController::class, 'myPatchMethod', 'patch', 'my-patch-method');
     }
+
+    /** @test */
+    public function it_can_register_multiple_patch_routes()
+    {
+        $this->routeRegistrar->registerClass(PatchMultipleTestController::class);
+
+        $this
+            ->assertRegisteredRoutesCount(2)
+            ->assertRouteRegistered(PatchMultipleTestController::class, 'myPatchMethod', 'patch', 'my-patch-method')
+            ->assertRouteRegistered(PatchMultipleTestController::class, 'myPatchMethod', 'patch', 'my-other-patch-method');
+    }
 }
diff --git a/tests/AttributeTests/PostAttributeTest.php b/tests/AttributeTests/PostAttributeTest.php
@@ -3,6 +3,7 @@
 namespace Spatie\RouteAttributes\Tests\AttributeTests;
 
 use Spatie\RouteAttributes\Tests\TestCase;
+use Spatie\RouteAttributes\Tests\TestClasses\Controllers\PostMultipleTestController;
 use Spatie\RouteAttributes\Tests\TestClasses\Controllers\PostTestController;
 
 class PostAttributeTest extends TestCase
@@ -16,4 +17,15 @@ public function it_can_register_a_post_route()
             ->assertRegisteredRoutesCount(1)
             ->assertRouteRegistered(PostTestController::class, 'myPostMethod', 'post', 'my-post-method');
     }
+
+    /** @test */
+    public function it_can_register_multiple_post_routes()
+    {
+        $this->routeRegistrar->registerClass(PostMultipleTestController::class);
+
+        $this
+            ->assertRegisteredRoutesCount(2)
+            ->assertRouteRegistered(PostMultipleTestController::class, 'myPostMethod', 'post', 'my-post-method')
+            ->assertRouteRegistered(PostMultipleTestController::class, 'myPostMethod', 'post', 'my-other-post-method');
+    }
 }
diff --git a/tests/AttributeTests/PutAttributeTest.php b/tests/AttributeTests/PutAttributeTest.php
@@ -3,6 +3,7 @@
 namespace Spatie\RouteAttributes\Tests\AttributeTests;
 
 use Spatie\RouteAttributes\Tests\TestCase;
+use Spatie\RouteAttributes\Tests\TestClasses\Controllers\PutMultipleTestController;
 use Spatie\RouteAttributes\Tests\TestClasses\Controllers\PutTestController;
 
 class PutAttributeTest extends TestCase
@@ -16,4 +17,15 @@ public function it_can_register_a_put_route()
             ->assertRegisteredRoutesCount(1)
             ->assertRouteRegistered(PutTestController::class, 'myPutMethod', 'put', 'my-put-method');
     }
+
+    /** @test */
+    public function it_can_register_multiple_put_routes()
+    {
+        $this->routeRegistrar->registerClass(PutMultipleTestController::class);
+
+        $this
+            ->assertRegisteredRoutesCount(2)
+            ->assertRouteRegistered(PutMultipleTestController::class, 'myPutMethod', 'put', 'my-put-method')
+            ->assertRouteRegistered(PutMultipleTestController::class, 'myPutMethod', 'put', 'my-other-put-method');
+    }
 }
diff --git a/tests/TestClasses/Controllers/DeleteMultipleTestController.php b/tests/TestClasses/Controllers/DeleteMultipleTestController.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace Spatie\RouteAttributes\Tests\TestClasses\Controllers;
+
+use Spatie\RouteAttributes\Attributes\Delete;
+
+class DeleteMultipleTestController
+{
+    #[Delete('my-delete-method')]
+    #[Delete('my-other-delete-method')]
+    public function myDeleteMethod()
+    {
+    }
+}
diff --git a/tests/TestClasses/Controllers/OptionsMultipleTestController.php b/tests/TestClasses/Controllers/OptionsMultipleTestController.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace Spatie\RouteAttributes\Tests\TestClasses\Controllers;
+
+use Spatie\RouteAttributes\Attributes\Options;
+
+class OptionsMultipleTestController
+{
+    #[Options('my-options-method')]
+    #[Options('my-other-options-method')]
+    public function myOptionsMethod()
+    {
+    }
+}
diff --git a/tests/TestClasses/Controllers/PatchMultipleTestController.php b/tests/TestClasses/Controllers/PatchMultipleTestController.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace Spatie\RouteAttributes\Tests\TestClasses\Controllers;
+
+use Spatie\RouteAttributes\Attributes\Patch;
+
+class PatchMultipleTestController
+{
+    #[Patch('my-patch-method')]
+    #[Patch('my-other-patch-method')]
+    public function myPatchMethod()
+    {
+    }
+}
diff --git a/tests/TestClasses/Controllers/PostMultipleTestController.php b/tests/TestClasses/Controllers/PostMultipleTestController.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace Spatie\RouteAttributes\Tests\TestClasses\Controllers;
+
+use Spatie\RouteAttributes\Attributes\Post;
+
+class PostMultipleTestController
+{
+    #[Post('my-post-method')]
+    #[Post('my-other-post-method')]
+    public function myPostMethod()
+    {
+    }
+}
diff --git a/tests/TestClasses/Controllers/PutMultipleTestController.php b/tests/TestClasses/Controllers/PutMultipleTestController.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace Spatie\RouteAttributes\Tests\TestClasses\Controllers;
+
+use Spatie\RouteAttributes\Attributes\Put;
+
+class PutMultipleTestController
+{
+    #[Put('my-put-method')]
+    #[Put('my-other-put-method')]
+    public function myPutMethod()
+    {
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use Attribute;`
`6`	`6`
`7`		`-#[Attribute(Attribute::TARGET_METHOD)]`
	`7`	`+#[Attribute(Attribute::TARGET_METHOD \| Attribute::IS_REPEATABLE)]`
`8`	`8`	`class Delete extends Route`
`9`	`9`	`{`
`10`	`10`	`public function __construct(`