Snake case FMCS and permission methods in tests

This commit is contained in:
Marcus Moore
2026-03-16 17:36:35 -07:00
parent bb72dab877
commit d84eb43278
57 changed files with 77 additions and 77 deletions
@@ -4,5 +4,5 @@ namespace Tests\Concerns;
interface TestsFullMultipleCompaniesSupport
{
public function testAdheresToFullMultipleCompaniesSupportScoping();
public function test_adheres_to_full_multiple_companies_support_scoping();
}
@@ -4,5 +4,5 @@ namespace Tests\Concerns;
interface TestsPermissionsRequirement
{
public function testRequiresPermission();
public function test_requires_permission();
}
@@ -10,7 +10,7 @@ use Tests\TestCase;
class AccessoriesForSelectListTest extends TestCase implements TestsFullMultipleCompaniesSupport
{
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -12,7 +12,7 @@ use Tests\TestCase;
class DeleteAccessoriesTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$accessory = Accessory::factory()->create();
@@ -23,7 +23,7 @@ class DeleteAccessoriesTest extends TestCase implements TestsFullMultipleCompani
$this->assertNotSoftDeleted($accessory);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
class IndexAccessoryCheckoutsTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$accessory = Accessory::factory()->create();
@@ -20,7 +20,7 @@ class IndexAccessoryCheckoutsTest extends TestCase implements TestsFullMultipleC
->assertForbidden();
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -11,14 +11,14 @@ use Tests\TestCase;
class IndexAccessoryTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create())
->getJson(route('api.accessories.index'))
->assertForbidden();
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
class ShowAccessoryTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$accessory = Accessory::factory()->create();
@@ -20,7 +20,7 @@ class ShowAccessoryTest extends TestCase implements TestsFullMultipleCompaniesSu
->assertForbidden();
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -14,14 +14,14 @@ use Tests\TestCase;
class StoreAccessoryTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create())
->postJson(route('api.accessories.store'))
->assertForbidden();
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
$this->markTestSkipped('This behavior is not implemented');
@@ -15,7 +15,7 @@ use Tests\TestCase;
class UpdateAccessoryTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$accessory = Accessory::factory()->create();
@@ -24,7 +24,7 @@ class UpdateAccessoryTest extends TestCase implements TestsFullMultipleCompanies
->assertForbidden();
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -8,7 +8,7 @@ use Tests\TestCase;
class AccessoriesIndexTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->get(route('accessories.index'))
@@ -10,7 +10,7 @@ use Tests\TestCase;
class DeleteAssetModelsTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$assetModel = AssetModel::factory()->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteAssetsTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$asset = Asset::factory()->create();
@@ -22,7 +22,7 @@ class DeleteAssetsTest extends TestCase implements TestsFullMultipleCompaniesSup
$this->assertNotSoftDeleted($asset);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteCategoriesTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$category = Category::factory()->create();
@@ -10,7 +10,7 @@ use Tests\TestCase;
class BulkDeleteCategoriesTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->post(route('categories.bulk.delete'), [
@@ -51,4 +51,4 @@ class BulkDeleteCategoriesTest extends TestCase implements TestsPermissionsRequi
}
}
}
@@ -11,7 +11,7 @@ use Tests\TestCase;
class AccessoryCheckinTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$accessory = Accessory::factory()->checkedOutToUser()->create();
$accessoryCheckout = $accessory->checkouts->first();
@@ -21,7 +21,7 @@ class AccessoryCheckinTest extends TestCase implements TestsFullMultipleCompanie
->assertForbidden();
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -14,7 +14,7 @@ use Tests\TestCase;
class ComponentCheckinTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$component = Component::factory()->checkedOutToAsset()->create();
@@ -121,7 +121,7 @@ class ComponentCheckinTest extends TestCase implements TestsFullMultipleCompanie
});
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
$this->settings->enableMultipleFullCompanySupport();
@@ -15,7 +15,7 @@ use Tests\TestCase;
class AccessoryCheckoutTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create())
->postJson(route('api.accessories.checkout', Accessory::factory()->create()))
@@ -14,7 +14,7 @@ use Tests\TestCase;
class ComponentCheckoutTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$component = Component::factory()->create();
@@ -124,7 +124,7 @@ class ComponentCheckoutTest extends TestCase implements TestsFullMultipleCompani
]);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -8,7 +8,7 @@ use Tests\TestCase;
class CreateCompaniesTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create())
->postJson(route('api.companies.store'))
@@ -9,7 +9,7 @@ use Tests\TestCase;
class DeleteCompaniesTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$company = Company::factory()->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteComponentTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$component = Component::factory()->create();
@@ -22,7 +22,7 @@ class DeleteComponentTest extends TestCase implements TestsFullMultipleCompanies
$this->assertNotSoftDeleted($component);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -12,7 +12,7 @@ use Tests\TestCase;
class DeleteComponentTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$component = Component::factory()->create();
@@ -80,7 +80,7 @@ class DeleteComponentTest extends TestCase implements TestsFullMultipleCompanies
]);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
$this->settings->enableMultipleFullCompanySupport();
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteConsumablesTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$consumable = Consumable::factory()->create();
@@ -22,7 +22,7 @@ class DeleteConsumablesTest extends TestCase implements TestsFullMultipleCompani
$this->assertNotSoftDeleted($consumable);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -13,7 +13,7 @@ use Tests\TestCase;
class CreateConsumableTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->get(route('consumables.create'))
@@ -10,7 +10,7 @@ use Tests\TestCase;
class DeleteCustomFieldsTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->markIncompleteIfMySQL('Custom Fields tests do not work on MySQL');
@@ -10,7 +10,7 @@ use Tests\TestCase;
class DeleteCustomFieldsetsTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->markIncompleteIfMySQL('Custom Fields tests do not work on MySQL');
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteDepartmentsTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$department = Department::factory()->create();
@@ -22,7 +22,7 @@ class DeleteDepartmentsTest extends TestCase implements TestsFullMultipleCompani
$this->assertDatabaseHas('departments', ['id' => $department->id]);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -9,7 +9,7 @@ use Tests\TestCase;
class DeleteDepreciationsTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$depreciation = Depreciation::factory()->create();
@@ -9,7 +9,7 @@ use Tests\TestCase;
class DeleteGroupsTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$group = Group::factory()->create();
@@ -30,7 +30,7 @@ class ImportAccessoriesTest extends ImportDataTestCase implements TestsPermissio
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -30,7 +30,7 @@ class ImportAssetModelsTest extends ImportDataTestCase implements TestsPermissio
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -36,7 +36,7 @@ class ImportAssetsTest extends ImportDataTestCase implements TestsPermissionsReq
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -28,7 +28,7 @@ class ImportCategoriesTest extends ImportDataTestCase implements TestsPermission
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -30,7 +30,7 @@ class ImportComponentsTest extends ImportDataTestCase implements TestsPermission
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -30,7 +30,7 @@ class ImportConsumablesTest extends ImportDataTestCase implements TestsPermissio
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -29,7 +29,7 @@ class ImportLicenseTest extends ImportDataTestCase implements TestsPermissionsRe
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -28,7 +28,7 @@ class ImportLocationsTest extends ImportDataTestCase implements TestsPermissions
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -28,7 +28,7 @@ class ImportManufacturersTest extends ImportDataTestCase implements TestsPermiss
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -29,7 +29,7 @@ class ImportSuppliersTest extends ImportDataTestCase implements TestsPermissions
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -32,7 +32,7 @@ class ImportUsersTest extends ImportDataTestCase implements TestsPermissionsRequ
}
#[Test]
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAsForApi(User::factory()->create());
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteLicensesTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$license = License::factory()->create();
@@ -22,7 +22,7 @@ class DeleteLicensesTest extends TestCase implements TestsFullMultipleCompaniesS
$this->assertNotSoftDeleted($license);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -13,7 +13,7 @@ use Tests\TestCase;
class DeleteLocationsTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$location = Location::factory()->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteMaintenancesTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$maintenance = Maintenance::factory()->create();
@@ -22,7 +22,7 @@ class DeleteMaintenancesTest extends TestCase implements TestsFullMultipleCompan
$this->assertNotSoftDeleted($maintenance);
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
[$companyA, $companyB] = Company::factory()->count(2)->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteManufacturersTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$manufacturer = Manufacturer::factory()->create();
@@ -10,7 +10,7 @@ use Tests\TestCase;
class BulkDeleteManufacturersTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->post(route('manufacturers.bulk.delete'), [
@@ -11,7 +11,7 @@ use Tests\TestCase;
class DeleteManufacturersTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->delete(route('categories.destroy', Category::factory()->create()))
@@ -41,4 +41,4 @@ class DeleteManufacturersTest extends TestCase implements TestsPermissionsRequir
$this->assertSoftDeleted($manufacturer);
}
}
}
@@ -10,7 +10,7 @@ use Tests\TestCase;
class DeletePredefinedKitsTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$predefinedKit = PredefinedKit::factory()->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
#[Group('custom-reporting')]
class DeleteReportTemplateTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$reportTemplate = ReportTemplate::factory()->create();
@@ -11,7 +11,7 @@ use Tests\TestCase;
#[Group('custom-reporting')]
class EditReportTemplateTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->get(route('report-templates.edit', ReportTemplate::factory()->create()))
@@ -11,7 +11,7 @@ use Tests\TestCase;
#[Group('custom-reporting')]
class ShowReportTemplateTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->get(route('report-templates.show', ReportTemplate::factory()->create()))
@@ -11,7 +11,7 @@ use Tests\TestCase;
#[Group('custom-reporting')]
class StoreReportTemplateTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->post(route('report-templates.store'))
@@ -11,7 +11,7 @@ use Tests\TestCase;
#[Group('custom-reporting')]
class UpdateReportTemplateTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->post(route('report-templates.update', ReportTemplate::factory()->create()))
+1 -1
View File
@@ -48,7 +48,7 @@ class CustomReportTest extends TestCase implements TestsPermissionsRequirement
);
}
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->get(route('reports/custom'))
@@ -9,7 +9,7 @@ use Tests\TestCase;
class DeleteStatusLabelsTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$statusLabel = Statuslabel::factory()->create();
@@ -10,7 +10,7 @@ use Tests\TestCase;
class DeleteSuppliersTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$supplier = Supplier::factory()->create();
@@ -10,7 +10,7 @@ use Tests\TestCase;
class BulkDeleteSuppliersTest extends TestCase implements TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$this->actingAs(User::factory()->create())
->post(route('suppliers.bulk.delete'), [
@@ -49,4 +49,4 @@ class BulkDeleteSuppliersTest extends TestCase implements TestsPermissionsRequir
$this->assertSoftDeleted($supplier2);
$this->assertSoftDeleted($supplier3);
}
}
}
+2 -2
View File
@@ -12,7 +12,7 @@ use Tests\TestCase;
class DeleteUsersTest extends TestCase implements TestsFullMultipleCompaniesSupport, TestsPermissionsRequirement
{
public function testRequiresPermission()
public function test_requires_permission()
{
$user = User::factory()->create();
@@ -100,7 +100,7 @@ class DeleteUsersTest extends TestCase implements TestsFullMultipleCompaniesSupp
}
public function testAdheresToFullMultipleCompaniesSupportScoping()
public function test_adheres_to_full_multiple_companies_support_scoping()
{
$this->settings->enableMultipleFullCompanySupport();