Finish test suite

tailflow · Sep 1, 2024 · 113d52f · 113d52f
1 parent 57de89b
commit 113d52f
Show file tree

Hide file tree

Showing 29 changed files with 409 additions and 37 deletions.
diff --git a/tests/Feature/Relations/BelongsTo/BelongsToRelationStandardDeleteOperationsTest.php b/tests/Feature/Relations/BelongsTo/BelongsToRelationStandardDeleteOperationsTest.php
@@ -181,6 +181,7 @@ public function deleting_a_single_relation_resource_with_multiple_route_paramete
     public function deleting_a_single_relation_resource_with_multiple_route_parameters_fails_with_default_key_resolver(): void
     {
         if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
             $this->expectException(QueryException::class);
         }
 

diff --git a/tests/Feature/Relations/BelongsTo/BelongsToRelationStandardRestoreOperationsTest.php b/tests/Feature/Relations/BelongsTo/BelongsToRelationStandardRestoreOperationsTest.php
@@ -115,6 +115,7 @@ public function restoring_a_single_relation_resource_with_multiple_route_paramet
     public function restoring_a_single_relation_resource_with_multiple_route_parameters_fails_with_default_key_resolver(): void
     {
         if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
             $this->expectException(QueryException::class);
         }
 

diff --git a/tests/Feature/Relations/BelongsTo/BelongsToRelationStandardShowOperationsTest.php b/tests/Feature/Relations/BelongsTo/BelongsToRelationStandardShowOperationsTest.php
@@ -132,6 +132,7 @@ public function getting_a_single_relation_resource_with_multiple_route_parameter
     public function getting_a_single_relation_resource_with_multiple_route_parameters_fails_with_default_key_resolver(): void
     {
         if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
             $this->expectException(QueryException::class);
         }
 

diff --git a/tests/Feature/Relations/BelongsTo/BelongsToRelationStandardUpdateOperationsTest.php b/tests/Feature/Relations/BelongsTo/BelongsToRelationStandardUpdateOperationsTest.php
@@ -155,6 +155,7 @@ public function updating_a_single_relation_resource_with_multiple_route_paramete
     public function updating_a_single_relation_resource_with_multiple_route_parameters_fails_with_default_key_resolver(): void
     {
         if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
             $this->expectException(QueryException::class);
         }
 

diff --git a/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationManyToManyOperationsTest.php b/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationManyToManyOperationsTest.php
@@ -4,8 +4,11 @@
 
 namespace Orion\Tests\Feature\Relations\BelongsToMany;
 
+use Illuminate\Database\QueryException;
+use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Gate;
 use Orion\Tests\Feature\TestCase;
+use Orion\Tests\Fixtures\App\Drivers\TwoRouteParameterKeyResolver;
 use Orion\Tests\Fixtures\App\Models\Role;
 use Orion\Tests\Fixtures\App\Models\User;
 use Orion\Tests\Fixtures\App\Policies\GreenPolicy;
@@ -46,7 +49,6 @@ public function attaching_relation_resources_when_authorized_only_on_parent(): v
 
         self::assertEquals(0, $user->roles()->count());
 
-
         $response = $this->post(
             "/api/users/{$user->id}/roles/attach",
             [
@@ -754,4 +756,307 @@ public function updating_pivot_of_relation_resource_casted_to_json_pivot_field()
             ['references' => ['key' => 'value']]
         );
     }
+
+    /** @test */
+    public function attaching_relation_resources_with_multiple_route_parameters(): void
+    {
+        $this->useKeyResolver(TwoRouteParameterKeyResolver::class);
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $roleA = factory(Role::class)->create();
+        $roleB = factory(Role::class)->create();
+
+        Gate::policy(User::class, GreenPolicy::class);
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        self::assertEquals(0, $user->roles()->count());
+
+        $response = $this->post(
+            "/api/v1/users/{$user->id}/roles/attach",
+            [
+                'resources' => [$roleA->id, $roleB->id],
+            ]
+        );
+
+        $this->assertResourcesAttached(
+            $response,
+            'roles',
+            $user,
+            collect([$roleA, $roleB])
+        );
+    }
+
+    /** @test */
+    public function attaching_relation_resources_with_multiple_route_parameters_fails_with_default_key_resolver(): void
+    {
+        if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
+            $this->expectException(QueryException::class);
+        }
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $roleA = factory(Role::class)->create();
+        $roleB = factory(Role::class)->create();
+
+        Gate::policy(User::class, GreenPolicy::class);
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        self::assertEquals(0, $user->roles()->count());
+
+        $response = $this->post(
+            "/api/v1/users/{$user->id}/roles/attach",
+            [
+                'resources' => [$roleA->id, $roleB->id],
+            ]
+        );
+
+        $response->assertNotFound();
+    }
+
+    /** @test */
+    public function detaching_relation_resources_with_multiple_route_parameters(): void
+    {
+        $this->useKeyResolver(TwoRouteParameterKeyResolver::class);
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $roles = factory(Role::class)->times(5)->make();
+        $user->roles()->saveMany($roles);
+
+        Gate::policy(User::class, GreenPolicy::class);
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        $roles = $user->roles()->get();
+
+        self::assertEquals(5, $roles->count());
+
+        $response = $this->delete(
+            "/api/v1/users/{$user->id}/roles/detach",
+            [
+                'resources' => $roles->pluck('id')->toArray(),
+            ]
+        );
+
+        $this->assertResourcesDetached($response, 'roles', $user, $roles);
+    }
+
+    /** @test */
+    public function detaching_relation_resources_with_multiple_route_parameters_fails_with_default_key_resolver(): void
+    {
+        if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
+            $this->expectException(QueryException::class);
+        }
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $roles = factory(Role::class)->times(5)->make();
+        $user->roles()->saveMany($roles);
+
+        Gate::policy(User::class, GreenPolicy::class);
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        $roles = $user->roles()->get();
+
+        self::assertEquals(5, $roles->count());
+
+        $response = $this->delete(
+            "/api/v1/users/{$user->id}/roles/detach",
+            [
+                'resources' => $roles->pluck('id')->toArray(),
+            ]
+        );
+
+        $response->assertNotFound();
+    }
+
+    /** @test */
+    public function syncing_relation_resources_with_multiple_route_parameters(): void
+    {
+        $this->useKeyResolver(TwoRouteParameterKeyResolver::class);
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $roleToAttach = factory(Role::class)->create();
+        $roleToDetach = factory(Role::class)->create();
+        $roleToUpdate = factory(Role::class)->create();
+        $user->roles()->attach($roleToDetach->id);
+        $user->roles()->attach($roleToUpdate->id, ['custom_name' => 'test original']);
+
+        Gate::policy(User::class, GreenPolicy::class);
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        self::assertEquals(2, $user->roles()->count());
+
+        $response = $this->patch(
+            "/api/v1/users/{$user->id}/roles/sync",
+            [
+                'resources' => [
+                    $roleToAttach->id,
+                    $roleToUpdate->id => ['custom_name' => 'test updated'],
+                ],
+            ]
+        );
+
+        $this->assertResourcesSynced(
+            $response,
+            'roles',
+            $user,
+            $this->buildSyncMap([$roleToAttach], [$roleToDetach], [$roleToUpdate]),
+            [$roleToUpdate->id => ['custom_name' => 'test updated']]
+        );
+    }
+
+    /** @test */
+    public function syncing_relation_resources_with_multiple_route_parameters_fails_with_default_key_resolver(): void
+    {
+        if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
+            $this->expectException(QueryException::class);
+        }
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $roleToAttach = factory(Role::class)->create();
+        $roleToDetach = factory(Role::class)->create();
+        $roleToUpdate = factory(Role::class)->create();
+        $user->roles()->attach($roleToDetach->id);
+        $user->roles()->attach($roleToUpdate->id, ['custom_name' => 'test original']);
+
+        Gate::policy(User::class, GreenPolicy::class);
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        self::assertEquals(2, $user->roles()->count());
+
+        $response = $this->patch(
+            "/api/v1/users/{$user->id}/roles/sync",
+            [
+                'resources' => [
+                    $roleToAttach->id,
+                    $roleToUpdate->id => ['custom_name' => 'test updated'],
+                ],
+            ]
+        );
+
+        $response->assertNotFound();
+    }
+
+    /** @test */
+    public function toggling_relation_resources_with_multiple_route_parameters(): void
+    {
+        $this->useKeyResolver(TwoRouteParameterKeyResolver::class);
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $roleToAttach = factory(Role::class)->create();
+        $roleToDetach = factory(Role::class)->create();
+        $user->roles()->attach($roleToDetach->id);
+
+        Gate::policy(User::class, GreenPolicy::class);
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        self::assertEquals(1, $user->roles()->count());
+
+        $response = $this->patch(
+            "/api/v1/users/{$user->id}/roles/toggle",
+            [
+                'resources' => [$roleToAttach->id, $roleToDetach->id],
+            ]
+        );
+
+        $this->assertResourcesToggled(
+            $response,
+            'roles',
+            $user,
+            $this->buildSyncMap([$roleToAttach], [$roleToDetach])
+        );
+    }
+
+    /** @test */
+    public function toggling_relation_resources_with_multiple_route_parameters_fails_with_default_key_resolver(): void
+    {
+        if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
+            $this->expectException(QueryException::class);
+        }
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $roleToAttach = factory(Role::class)->create();
+        $roleToDetach = factory(Role::class)->create();
+        $user->roles()->attach($roleToDetach->id);
+
+        Gate::policy(User::class, GreenPolicy::class);
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        self::assertEquals(1, $user->roles()->count());
+
+        $response = $this->patch(
+            "/api/v1/users/{$user->id}/roles/toggle",
+            [
+                'resources' => [$roleToAttach->id, $roleToDetach->id],
+            ]
+        );
+
+        $response->assertNotFound();
+    }
+
+    /** @test */
+    public function updating_pivot_of_relation_resource_with_multiple_route_parameters(): void
+    {
+        $this->useKeyResolver(TwoRouteParameterKeyResolver::class);
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $role = factory(Role::class)->create();
+        $user->roles()->save($role);
+
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        $response = $this->patch(
+            "/api/v1/users/{$user->id}/roles/{$role->id}/pivot",
+            [
+                'pivot' => [
+                    'custom_name' => 'test value',
+                ],
+            ]
+        );
+
+        $this->assertResourcePivotUpdated(
+            $response,
+            'roles',
+            $user,
+            $role,
+            ['custom_name' => 'test value']
+        );
+    }
+
+    /** @test */
+    public function updating_pivot_of_relation_resource_with_multiple_route_parameters_fails_with_default_key_resolver(): void
+    {
+        if (DB::connection()->getDriverName() === 'pgsql') {
+            $this->withoutExceptionHandling();
+            $this->expectException(QueryException::class);
+        }
+
+        /** @var User $user */
+        $user = factory(User::class)->create();
+        $role = factory(Role::class)->create();
+        $user->roles()->save($role);
+
+        Gate::policy(Role::class, GreenPolicy::class);
+
+        $response = $this->patch(
+            "/api/v1/users/{$user->id}/roles/{$role->id}/pivot",
+            [
+                'pivot' => [
+                    'custom_name' => 'test value',
+                ],
+            ]
+        );
+
+        $response->assertNotFound();
+    }
 }
diff --git a/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationStandardDeleteOperationsTest.php b/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationStandardDeleteOperationsTest.php
@@ -3,8 +3,6 @@
 namespace Orion\Tests\Feature\Relations\BelongsToMany;
 
 use Illuminate\Support\Facades\Gate;
-use Mockery;
-use Orion\Contracts\ComponentsResolver;
 use Orion\Tests\Feature\TestCase;
 use Orion\Tests\Fixtures\App\Http\Resources\SampleResource;
 use Orion\Tests\Fixtures\App\Models\Notification;

diff --git a/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationStandardIndexOperationsTest.php b/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationStandardIndexOperationsTest.php
@@ -3,8 +3,6 @@
 namespace Orion\Tests\Feature\Relations\BelongsToMany;
 
 use Illuminate\Support\Facades\Gate;
-use Mockery;
-use Orion\Contracts\ComponentsResolver;
 use Orion\Tests\Feature\TestCase;
 use Orion\Tests\Fixtures\App\Http\Resources\SampleCollectionResource;
 use Orion\Tests\Fixtures\App\Http\Resources\SampleResource;

diff --git a/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationStandardRestoreOperationsTest.php b/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationStandardRestoreOperationsTest.php
@@ -3,8 +3,6 @@
 namespace Orion\Tests\Feature\Relations\BelongsToMany;
 
 use Illuminate\Support\Facades\Gate;
-use Mockery;
-use Orion\Contracts\ComponentsResolver;
 use Orion\Tests\Feature\TestCase;
 use Orion\Tests\Fixtures\App\Http\Resources\SampleResource;
 use Orion\Tests\Fixtures\App\Models\Notification;

diff --git a/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationStandardShowOperationsTest.php b/tests/Feature/Relations/BelongsToMany/BelongsToManyRelationStandardShowOperationsTest.php
@@ -95,7 +95,6 @@ public function getting_a_single_trashed_relation_resource_when_with_trashed_que
         $this->assertResourceShown($response, $user->notifications()->withTrashed()->first()->toArray());
     }
 
-
     /** @test */
     public function getting_a_single_transformed_relation_resource(): void
     {
-Original file line number
+Diff line change
@@ Expand Up @@
             $this->assertResourceShown($response, $user->notifications()->withTrashed()->first()->toArray());
         }
         /** @test */
         public function getting_a_single_transformed_relation_resource(): void
         {
@@ Expand Down @@