Updated delete users API tests, moved non-API tests

Signed-off-by: snipe <snipe@snipe.net>

tests/Feature/Api/Users/DeleteUsersTest.php

@@ -0,0 +1,69 @@
+<?php
+
+namespace Tests\Feature\Api\Users;
+
+use App\Models\Location;
+use App\Models\User;
+use App\Models\LicenseSeat;
+use Tests\TestCase;
+
+class DeleteUsersTest extends TestCase
+{
+
+
+    public function testDisallowUserDeletionViaApiIfStillManagingPeople()
+    {
+        $manager = User::factory()->create();
+        User::factory()->count(5)->create(['manager_id' => $manager->id]);
+        $this->assertFalse($manager->isDeletable());
+
+        $this->actingAsForApi(User::factory()->deleteUsers()->create())
+            ->deleteJson(route('api.users.destroy', $manager->id))
+            ->assertOk()
+            ->assertStatus(200)
+            ->assertStatusMessageIs('error')
+            ->json();
+    }
+
+    public function testDisallowUserDeletionViaApiIfStillManagingLocations()
+    {
+        $manager = User::factory()->create();
+        Location::factory()->count(5)->create(['manager_id' => $manager->id]);
+
+        $this->assertFalse($manager->isDeletable());
+
+        $this->actingAsForApi(User::factory()->deleteUsers()->create())
+            ->deleteJson(route('api.users.destroy', $manager->id))
+            ->assertOk()
+            ->assertStatus(200)
+            ->assertStatusMessageIs('error')
+            ->json();
+    }
+
+    public function testDisallowUserDeletionViaApiIfStillHasLicenses()
+    {
+        $manager = User::factory()->create();
+        LicenseSeat::factory()->count(5)->create(['assigned_to' => $manager->id]);
+
+        $this->assertFalse($manager->isDeletable());
+
+        $this->actingAsForApi(User::factory()->deleteUsers()->create())
+            ->deleteJson(route('api.users.destroy', $manager->id))
+            ->assertOk()
+            ->assertStatus(200)
+            ->assertStatusMessageIs('error')
+            ->json();
+    }
+
+    public function testDisallowUserDeletionIfNoDeletePermissions()
+    {
+
+        $this->actingAsForApi(User::factory()->create())
+            ->deleteJson(route('api.users.destroy', User::factory()->create()))
+            ->assertStatus(403)
+            ->json();
+    }
+
+
+
+}

tests/Feature/Users/DeleteUsersTest.php

@@ -1,13 +1,13 @@
 <?php
 
-namespace Tests\Feature\Api\Users;
+namespace Tests\Feature\Users;
 
 use App\Models\Location;
 use App\Models\User;
 use Laravel\Passport\Passport;
 use Tests\TestCase;
 
-class UsersDeleteTest extends TestCase
+class DeleteUsersTest extends TestCase
 {