Unify assertion method

snipe · Sep 16, 2024 · f325c4a · f325c4a
1 parent 1fddacd
commit f325c4a
Show file tree

Hide file tree

Showing 17 changed files with 34 additions and 0 deletions.
diff --git a/tests/Feature/Accessories/Api/DeleteAccessoriesTest.php b/tests/Feature/Accessories/Api/DeleteAccessoriesTest.php
@@ -18,6 +18,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.accessories.destroy', $accessory))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($accessory);
     }
 
     public function testAdheresToMultipleFullCompanySupportScoping()

diff --git a/tests/Feature/AssetMaintenances/Api/DeleteAssetMaintenancesTest.php b/tests/Feature/AssetMaintenances/Api/DeleteAssetMaintenancesTest.php
@@ -18,6 +18,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.maintenances.destroy', $assetMaintenance))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($assetMaintenance);
     }
 
     public function testAdheresToMultipleFullCompanySupportScoping()

diff --git a/tests/Feature/AssetModels/Api/DeleteAssetModelsTest.php b/tests/Feature/AssetModels/Api/DeleteAssetModelsTest.php
@@ -17,6 +17,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.models.destroy', $assetModel))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($assetModel);
     }
 
     public function testCannotDeleteAssetModelThatStillHasAssociatedAssets()

diff --git a/tests/Feature/Assets/Api/DeleteAssetsTest.php b/tests/Feature/Assets/Api/DeleteAssetsTest.php
@@ -18,6 +18,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.assets.destroy', $asset))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($asset);
     }
 
     public function testAdheresToMultipleFullCompanySupportScoping()

diff --git a/tests/Feature/Categories/Api/DeleteCategoriesTest.php b/tests/Feature/Categories/Api/DeleteCategoriesTest.php
@@ -17,6 +17,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.categories.destroy', $category))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($category);
     }
 
     public function testCannotDeleteCategoryThatStillHasAssociatedItems()

diff --git a/tests/Feature/Companies/Api/DeleteCompaniesTest.php b/tests/Feature/Companies/Api/DeleteCompaniesTest.php
@@ -16,6 +16,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.companies.destroy', $company))
             ->assertForbidden();
+
+        $this->assertDatabaseHas('companies', ['id' => $company->id]);
     }
 
     public function testCannotDeleteCompanyThatHasAssociatedItems()

diff --git a/tests/Feature/Components/Api/DeleteComponentsTest.php b/tests/Feature/Components/Api/DeleteComponentsTest.php
@@ -18,6 +18,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.components.destroy', $component))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($component);
     }
 
     public function testAdheresToMultipleFullCompanySupportScoping()

diff --git a/tests/Feature/Consumables/Api/DeleteConsumablesTest.php b/tests/Feature/Consumables/Api/DeleteConsumablesTest.php
@@ -18,6 +18,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.consumables.destroy', $consumable))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($consumable);
     }
 
     public function testAdheresToMultipleFullCompanySupportScoping()

diff --git a/tests/Feature/CustomFields/Api/DeleteCustomFieldsTest.php b/tests/Feature/CustomFields/Api/DeleteCustomFieldsTest.php
@@ -19,6 +19,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.customfields.destroy', $customField))
             ->assertForbidden();
+
+        $this->assertDatabaseHas('custom_fields', ['id' => $customField->id]);
     }
 
     public function testCustomFieldsCannotBeDeletedIfTheyHaveAssociatedFieldsets()

diff --git a/tests/Feature/CustomFieldsets/Api/DeleteCustomFieldsetsTest.php b/tests/Feature/CustomFieldsets/Api/DeleteCustomFieldsetsTest.php
@@ -19,6 +19,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.fieldsets.destroy', $customFieldset))
             ->assertForbidden();
+
+        $this->assertDatabaseHas('custom_fieldsets', ['id' => $customFieldset->id]);
     }
 
     public function testCannotDeleteCustomFieldsetWithAssociatedFields()

diff --git a/tests/Feature/Departments/Api/DeleteDepartmentTest.php b/tests/Feature/Departments/Api/DeleteDepartmentTest.php
@@ -18,6 +18,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.departments.destroy', $department))
             ->assertForbidden();
+
+        $this->assertDatabaseHas('departments', ['id' => $department->id]);
     }
 
     public function testAdheresToMultipleFullCompanySupportScoping()

diff --git a/tests/Feature/Depreciations/Api/DeleteDepreciationTest.php b/tests/Feature/Depreciations/Api/DeleteDepreciationTest.php
@@ -16,6 +16,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.depreciations.destroy', $depreciation))
             ->assertForbidden();
+
+        $this->assertDatabaseHas('depreciations', ['id' => $depreciation->id]);
     }
 
     public function testCannotDeleteDepreciationThatHasAssociatedModels()

diff --git a/tests/Feature/Groups/Api/DeleteGroupTest.php b/tests/Feature/Groups/Api/DeleteGroupTest.php
@@ -16,6 +16,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.groups.destroy', $group))
             ->assertForbidden();
+
+        $this->assertDatabaseHas('permission_groups', ['id' => $group->id]);
     }
 
     public function testCanDeleteGroup()

diff --git a/tests/Feature/Licenses/Api/DeleteLicenseTest.php b/tests/Feature/Licenses/Api/DeleteLicenseTest.php
@@ -18,6 +18,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.licenses.destroy', $license))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($license);
     }
 
     public function testAdheresToMultipleFullCompanySupportScoping()

diff --git a/tests/Feature/PredefinedKits/Api/DeletePredefinedKitTest.php b/tests/Feature/PredefinedKits/Api/DeletePredefinedKitTest.php
@@ -17,6 +17,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.kits.destroy', $predefinedKit))
             ->assertForbidden();
+
+        $this->assertDatabaseHas('kits', ['id' => $predefinedKit->id]);
     }
 
     public function testCanDeletePredefinedKits()

diff --git a/tests/Feature/StatusLabels/Api/DeleteStatusLabelTest.php b/tests/Feature/StatusLabels/Api/DeleteStatusLabelTest.php
@@ -16,6 +16,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.statuslabels.destroy', $statusLabel))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($statusLabel);
     }
 
     public function testCannotDeleteStatusLabelWhileStillAssociatedToAssets()

diff --git a/tests/Feature/Suppliers/Api/DeleteSupplierTest.php b/tests/Feature/Suppliers/Api/DeleteSupplierTest.php
@@ -17,6 +17,8 @@ public function testRequiresPermission()
         $this->actingAsForApi(User::factory()->create())
             ->deleteJson(route('api.suppliers.destroy', $supplier))
             ->assertForbidden();
+
+        $this->assertNotSoftDeleted($supplier);
     }
 
     public function testCannotDeleteSupplierWithDataStillAssociated()