From 672aabf4ac9dc5e692c04b81d81fb0288a1b00a3 Mon Sep 17 00:00:00 2001
From: Marcus Moore <contact@marcusmoore.io>
Date: Mon, 17 Jun 2024 14:51:59 -0700
Subject: [PATCH] Add more tests for Asset Model index and store methods

---
 .../AssetModels/Ui/AssetModelIndexTest.php    | 23 +++++++++++
 .../AssetModels/Ui/AssetModelStoreTest.php    | 35 +++++++++++++++++
 .../AssetModels/Ui/AssetModelsTest.php        | 38 -------------------
 3 files changed, 58 insertions(+), 38 deletions(-)
 create mode 100644 tests/Feature/AssetModels/Ui/AssetModelIndexTest.php
 create mode 100644 tests/Feature/AssetModels/Ui/AssetModelStoreTest.php
 delete mode 100644 tests/Feature/AssetModels/Ui/AssetModelsTest.php

diff --git a/tests/Feature/AssetModels/Ui/AssetModelIndexTest.php b/tests/Feature/AssetModels/Ui/AssetModelIndexTest.php
new file mode 100644
index 0000000000..a73d425723
--- /dev/null
+++ b/tests/Feature/AssetModels/Ui/AssetModelIndexTest.php
@@ -0,0 +1,23 @@
+<?php
+
+namespace Tests\Feature\AssetModels\Ui;
+
+use App\Models\User;
+use Tests\TestCase;
+
+class AssetModelIndexTest extends TestCase
+{
+    public function testPermissionRequiredToViewAssetModelList()
+    {
+        $this->actingAs(User::factory()->create())
+            ->get(route('models.index'))
+            ->assertForbidden();
+    }
+
+    public function testUserCanListAssetModels()
+    {
+        $this->actingAs(User::factory()->superuser()->create())
+            ->get(route('models.index'))
+            ->assertOk();
+    }
+}
diff --git a/tests/Feature/AssetModels/Ui/AssetModelStoreTest.php b/tests/Feature/AssetModels/Ui/AssetModelStoreTest.php
new file mode 100644
index 0000000000..81b07237c0
--- /dev/null
+++ b/tests/Feature/AssetModels/Ui/AssetModelStoreTest.php
@@ -0,0 +1,35 @@
+<?php
+
+namespace Tests\Feature\AssetModels\Ui;
+
+use App\Models\AssetModel;
+use App\Models\Category;
+use App\Models\User;
+use Tests\TestCase;
+
+class AssetModelStoreTest extends TestCase
+{
+    public function testPermissionRequiredToStoreAssetModel()
+    {
+        $this->actingAs(User::factory()->create())
+            ->post(route('models.store'), [
+                'name' => 'Test Model',
+                'category_id' => Category::factory()->create()->id
+            ])
+            ->assertForbidden();
+    }
+
+    public function testUserCanCreateAssetModels()
+    {
+        $this->assertFalse(AssetModel::where('name', 'Test Model')->exists());
+
+        $this->actingAs(User::factory()->superuser()->create())
+            ->post(route('models.store'), [
+                'name' => 'Test Model',
+                'category_id' => Category::factory()->create()->id
+            ])
+            ->assertRedirect(route('models.index'));
+
+        $this->assertTrue(AssetModel::where('name', 'Test Model')->exists());
+    }
+}
diff --git a/tests/Feature/AssetModels/Ui/AssetModelsTest.php b/tests/Feature/AssetModels/Ui/AssetModelsTest.php
deleted file mode 100644
index bb60a25fe5..0000000000
--- a/tests/Feature/AssetModels/Ui/AssetModelsTest.php
+++ /dev/null
@@ -1,38 +0,0 @@
-<?php
-
-namespace Tests\Feature\AssetModels\Ui;
-
-use App\Models\Category;
-use App\Models\AssetModel;
-use App\Models\Company;
-use App\Models\CustomField;
-use App\Models\Statuslabel;
-use App\Models\Supplier;
-use App\Models\User;
-use Illuminate\Support\Facades\Crypt;
-use Tests\TestCase;
-
-class AssetModelsTest extends TestCase
-{
-    public function testUserCanListAssetModels()
-    {
-
-        $this->actingAs(User::factory()->superuser()->create())
-            ->get(route('models.index'))
-            ->assertStatus(200);
-
-    }
-
-    public function testUserCanCreateAssetModels()
-    {
-        $this->actingAs(User::factory()->superuser()->create())
-            ->post(route('models.index'), [
-                'name' => 'Test Model',
-                'category_id' => Category::factory()->create()->id
-            ])
-            ->assertStatus(302)
-            ->assertRedirect(route('models.index'));
-    }
-
-
-}