From 7c26cee1ea0b9bce1f4abeaddaf962f63202af8b Mon Sep 17 00:00:00 2001 From: Constantin Graf Date: Fri, 14 Jun 2024 14:48:10 +0200 Subject: [PATCH] Added PHPUnit annotations --- .../Commands/Admin/DeleteOrganizationCommandTest.php | 5 +++++ .../Commands/SelfHost/SelfHostGenerateKeysCommandTest.php | 5 +++++ tests/Unit/Endpoint/Api/V1/ClientEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/ImportEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/InvitationEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/MemberEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/OrganizationEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/ProjectEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/ProjectMemberEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/TagEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/TaskEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/TimeEntryEndpointTest.php | 3 +++ tests/Unit/Endpoint/Api/V1/UserTimeEntryEndpointTest.php | 3 +++ tests/Unit/Endpoint/Web/DashboardEndpointTest.php | 5 +++++ tests/Unit/Endpoint/Web/HealthCheckEndpointTest.php | 6 ++++++ tests/Unit/Endpoint/Web/HomeEndpointTest.php | 5 +++++ tests/Unit/Filament/ClientResourceTest.php | 2 ++ tests/Unit/Filament/OrganizationResourceTest.php | 2 ++ tests/Unit/Filament/ProjectResourceTest.php | 2 ++ tests/Unit/Filament/TagResourceTest.php | 2 ++ tests/Unit/Filament/TaskResourceTest.php | 2 ++ tests/Unit/Filament/TimeEntryResourceTest.php | 2 ++ tests/Unit/Filament/UserResourceTest.php | 3 +++ 23 files changed, 74 insertions(+) diff --git a/tests/Unit/Console/Commands/Admin/DeleteOrganizationCommandTest.php b/tests/Unit/Console/Commands/Admin/DeleteOrganizationCommandTest.php index c3df5e76..c3959c51 100644 --- a/tests/Unit/Console/Commands/Admin/DeleteOrganizationCommandTest.php +++ b/tests/Unit/Console/Commands/Admin/DeleteOrganizationCommandTest.php @@ -4,12 +4,17 @@ declare(strict_types=1); namespace Tests\Unit\Console\Commands\Admin; +use App\Console\Commands\Admin\DeleteOrganizationCommand; use App\Models\Organization; use App\Service\DeletionService; use Illuminate\Support\Str; use Mockery\MockInterface; +use PHPUnit\Framework\Attributes\CoversClass; +use PHPUnit\Framework\Attributes\UsesClass; use Tests\TestCaseWithDatabase; +#[CoversClass(DeleteOrganizationCommand::class)] +#[UsesClass(DeleteOrganizationCommand::class)] class DeleteOrganizationCommandTest extends TestCaseWithDatabase { public function test_it_calls_the_deletion_service_with_the_organization(): void diff --git a/tests/Unit/Console/Commands/SelfHost/SelfHostGenerateKeysCommandTest.php b/tests/Unit/Console/Commands/SelfHost/SelfHostGenerateKeysCommandTest.php index 043f7102..1dfbcddc 100644 --- a/tests/Unit/Console/Commands/SelfHost/SelfHostGenerateKeysCommandTest.php +++ b/tests/Unit/Console/Commands/SelfHost/SelfHostGenerateKeysCommandTest.php @@ -4,10 +4,15 @@ declare(strict_types=1); namespace Tests\Unit\Console\Commands\SelfHost; +use App\Console\Commands\SelfHost\SelfHostGenerateKeysCommand; use Illuminate\Console\Command; use Illuminate\Support\Facades\Artisan; +use PHPUnit\Framework\Attributes\CoversClass; +use PHPUnit\Framework\Attributes\UsesClass; use Tests\TestCase; +#[CoversClass(SelfHostGenerateKeysCommand::class)] +#[UsesClass(SelfHostGenerateKeysCommand::class)] class SelfHostGenerateKeysCommandTest extends TestCase { public function test_generates_app_key_and_passport_keys_per_default_in_env_format(): void diff --git a/tests/Unit/Endpoint/Api/V1/ClientEndpointTest.php b/tests/Unit/Endpoint/Api/V1/ClientEndpointTest.php index bd183416..b6e4e2da 100644 --- a/tests/Unit/Endpoint/Api/V1/ClientEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/ClientEndpointTest.php @@ -4,12 +4,15 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; +use App\Http\Controllers\Api\V1\ClientController; use App\Models\Client; use App\Models\Organization; use App\Models\Project; use Illuminate\Testing\Fluent\AssertableJson; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(ClientController::class)] class ClientEndpointTest extends ApiEndpointTestAbstract { public function test_index_endpoint_fails_if_user_has_no_permission_to_view_clients(): void diff --git a/tests/Unit/Endpoint/Api/V1/ImportEndpointTest.php b/tests/Unit/Endpoint/Api/V1/ImportEndpointTest.php index 99230ae6..a5ab75ea 100644 --- a/tests/Unit/Endpoint/Api/V1/ImportEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/ImportEndpointTest.php @@ -4,13 +4,16 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; +use App\Http\Controllers\Api\V1\ImportController; use App\Models\Organization; use App\Service\Import\Importers\ImportException; use App\Service\Import\Importers\ReportDto; use App\Service\Import\ImportService; use Laravel\Passport\Passport; use Mockery\MockInterface; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(ImportController::class)] class ImportEndpointTest extends ApiEndpointTestAbstract { public function test_index_fails_if_user_does_not_have_permission() diff --git a/tests/Unit/Endpoint/Api/V1/InvitationEndpointTest.php b/tests/Unit/Endpoint/Api/V1/InvitationEndpointTest.php index 3baeceee..c0bc6c23 100644 --- a/tests/Unit/Endpoint/Api/V1/InvitationEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/InvitationEndpointTest.php @@ -5,11 +5,14 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; use App\Enums\Role; +use App\Http\Controllers\Api\V1\InvitationController; use App\Models\OrganizationInvitation; use Illuminate\Support\Facades\Mail; use Laravel\Jetstream\Mail\TeamInvitation; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(InvitationController::class)] class InvitationEndpointTest extends ApiEndpointTestAbstract { public function test_index_fails_if_user_has_no_permission_to_view_invitations(): void diff --git a/tests/Unit/Endpoint/Api/V1/MemberEndpointTest.php b/tests/Unit/Endpoint/Api/V1/MemberEndpointTest.php index afd955c9..33a26848 100644 --- a/tests/Unit/Endpoint/Api/V1/MemberEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/MemberEndpointTest.php @@ -5,6 +5,7 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; use App\Enums\Role; +use App\Http\Controllers\Api\V1\MemberController; use App\Models\Member; use App\Models\Organization; use App\Models\Project; @@ -12,7 +13,9 @@ use App\Models\ProjectMember; use App\Models\TimeEntry; use App\Models\User; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(MemberController::class)] class MemberEndpointTest extends ApiEndpointTestAbstract { public function test_index_fails_if_user_has_no_permission_to_view_members(): void diff --git a/tests/Unit/Endpoint/Api/V1/OrganizationEndpointTest.php b/tests/Unit/Endpoint/Api/V1/OrganizationEndpointTest.php index 55ed1f7b..2fb8cde8 100644 --- a/tests/Unit/Endpoint/Api/V1/OrganizationEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/OrganizationEndpointTest.php @@ -4,9 +4,12 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; +use App\Http\Controllers\Api\V1\OrganizationController; use App\Models\Organization; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(OrganizationController::class)] class OrganizationEndpointTest extends ApiEndpointTestAbstract { public function test_show_endpoint_fails_if_user_has_no_permission_to_view_organizations(): void diff --git a/tests/Unit/Endpoint/Api/V1/ProjectEndpointTest.php b/tests/Unit/Endpoint/Api/V1/ProjectEndpointTest.php index 0cc1b62b..5be6af3f 100644 --- a/tests/Unit/Endpoint/Api/V1/ProjectEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/ProjectEndpointTest.php @@ -4,6 +4,7 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; +use App\Http\Controllers\Api\V1\ProjectController; use App\Models\Client; use App\Models\Organization; use App\Models\Project; @@ -11,7 +12,9 @@ use App\Models\ProjectMember; use App\Models\Task; use App\Models\TimeEntry; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(ProjectController::class)] class ProjectEndpointTest extends ApiEndpointTestAbstract { public function test_index_endpoint_fails_if_user_has_no_permission_to_view_projects(): void diff --git a/tests/Unit/Endpoint/Api/V1/ProjectMemberEndpointTest.php b/tests/Unit/Endpoint/Api/V1/ProjectMemberEndpointTest.php index 29edad42..24e6487a 100644 --- a/tests/Unit/Endpoint/Api/V1/ProjectMemberEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/ProjectMemberEndpointTest.php @@ -4,12 +4,15 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; +use App\Http\Controllers\Api\V1\ProjectMemberController; use App\Models\Member; use App\Models\Project; use App\Models\ProjectMember; use App\Models\User; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(ProjectMemberController::class)] class ProjectMemberEndpointTest extends ApiEndpointTestAbstract { public function test_index_endpoint_fails_if_user_has_no_permission_to_view_project_members(): void diff --git a/tests/Unit/Endpoint/Api/V1/TagEndpointTest.php b/tests/Unit/Endpoint/Api/V1/TagEndpointTest.php index f9666439..30d20843 100644 --- a/tests/Unit/Endpoint/Api/V1/TagEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/TagEndpointTest.php @@ -4,12 +4,15 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; +use App\Http\Controllers\Api\V1\TagController; use App\Models\Organization; use App\Models\Tag; use App\Models\TimeEntry; use Illuminate\Testing\Fluent\AssertableJson; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(TagController::class)] class TagEndpointTest extends ApiEndpointTestAbstract { public function test_index_endpoint_fails_if_user_has_no_permission_to_view_tags(): void diff --git a/tests/Unit/Endpoint/Api/V1/TaskEndpointTest.php b/tests/Unit/Endpoint/Api/V1/TaskEndpointTest.php index 8fad5433..dd0e9e49 100644 --- a/tests/Unit/Endpoint/Api/V1/TaskEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/TaskEndpointTest.php @@ -4,12 +4,15 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; +use App\Http\Controllers\Api\V1\TaskController; use App\Models\Project; use App\Models\ProjectMember; use App\Models\Task; use App\Models\TimeEntry; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(TaskController::class)] class TaskEndpointTest extends ApiEndpointTestAbstract { public function test_non_valid_uuid_for_organization_id_fails(): void diff --git a/tests/Unit/Endpoint/Api/V1/TimeEntryEndpointTest.php b/tests/Unit/Endpoint/Api/V1/TimeEntryEndpointTest.php index 4b5059e8..b7545149 100644 --- a/tests/Unit/Endpoint/Api/V1/TimeEntryEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/TimeEntryEndpointTest.php @@ -6,6 +6,7 @@ namespace Tests\Unit\Endpoint\Api\V1; use App\Enums\Role; use App\Exceptions\Api\TimeEntryCanNotBeRestartedApiException; +use App\Http\Controllers\Api\V1\TimeEntryController; use App\Models\Client; use App\Models\Member; use App\Models\Project; @@ -18,8 +19,10 @@ use Illuminate\Support\Facades\Log; use Illuminate\Support\Str; use Illuminate\Testing\Fluent\AssertableJson; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; use TiMacDonald\Log\LogEntry; +#[UsesClass(TimeEntryController::class)] class TimeEntryEndpointTest extends ApiEndpointTestAbstract { public function test_index_endpoint_fails_if_user_has_no_permission_to_view_time_entries(): void diff --git a/tests/Unit/Endpoint/Api/V1/UserTimeEntryEndpointTest.php b/tests/Unit/Endpoint/Api/V1/UserTimeEntryEndpointTest.php index fb090f1b..0a6f5253 100644 --- a/tests/Unit/Endpoint/Api/V1/UserTimeEntryEndpointTest.php +++ b/tests/Unit/Endpoint/Api/V1/UserTimeEntryEndpointTest.php @@ -4,12 +4,15 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Api\V1; +use App\Http\Controllers\Api\V1\UserTimeEntryController; use App\Models\TimeEntry; use Illuminate\Support\Carbon; use Illuminate\Support\Facades\Log; use Laravel\Passport\Passport; +use PHPUnit\Framework\Attributes\UsesClass; use TiMacDonald\Log\LogEntry; +#[UsesClass(UserTimeEntryController::class)] class UserTimeEntryEndpointTest extends ApiEndpointTestAbstract { public function test_my_active_endpoint_returns_unauthorized_if_user_is_not_logged_in(): void diff --git a/tests/Unit/Endpoint/Web/DashboardEndpointTest.php b/tests/Unit/Endpoint/Web/DashboardEndpointTest.php index 006d08cd..1222d28a 100644 --- a/tests/Unit/Endpoint/Web/DashboardEndpointTest.php +++ b/tests/Unit/Endpoint/Web/DashboardEndpointTest.php @@ -5,10 +5,15 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Web; use App\Enums\Role; +use App\Http\Controllers\Web\DashboardController; use App\Models\Organization; use App\Models\User; use Inertia\Testing\AssertableInertia as Assert; +use PHPUnit\Framework\Attributes\CoversClass; +use PHPUnit\Framework\Attributes\UsesClass; +#[CoversClass(DashboardController::class)] +#[UsesClass(DashboardController::class)] class DashboardEndpointTest extends EndpointTestAbstract { public function test_showing_dashboard_succeeds_for_empty_user_with_no_data_entries(): void diff --git a/tests/Unit/Endpoint/Web/HealthCheckEndpointTest.php b/tests/Unit/Endpoint/Web/HealthCheckEndpointTest.php index 8827f83d..3645d514 100644 --- a/tests/Unit/Endpoint/Web/HealthCheckEndpointTest.php +++ b/tests/Unit/Endpoint/Web/HealthCheckEndpointTest.php @@ -4,6 +4,12 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Web; +use App\Http\Controllers\Web\HealthCheckController; +use PHPUnit\Framework\Attributes\CoversClass; +use PHPUnit\Framework\Attributes\UsesClass; + +#[CoversClass(HealthCheckController::class)] +#[UsesClass(HealthCheckController::class)] class HealthCheckEndpointTest extends EndpointTestAbstract { public function test_up_endpoint_returns_ok(): void diff --git a/tests/Unit/Endpoint/Web/HomeEndpointTest.php b/tests/Unit/Endpoint/Web/HomeEndpointTest.php index c12de40d..69a05d10 100644 --- a/tests/Unit/Endpoint/Web/HomeEndpointTest.php +++ b/tests/Unit/Endpoint/Web/HomeEndpointTest.php @@ -4,8 +4,13 @@ declare(strict_types=1); namespace Tests\Unit\Endpoint\Web; +use App\Http\Controllers\Web\HomeController; use App\Models\User; +use PHPUnit\Framework\Attributes\CoversClass; +use PHPUnit\Framework\Attributes\UsesClass; +#[CoversClass(HomeController::class)] +#[UsesClass(HomeController::class)] class HomeEndpointTest extends EndpointTestAbstract { public function test_index_redirects_to_dashboard_if_user_is_logged_in(): void diff --git a/tests/Unit/Filament/ClientResourceTest.php b/tests/Unit/Filament/ClientResourceTest.php index 1cfcb6ac..606dede5 100644 --- a/tests/Unit/Filament/ClientResourceTest.php +++ b/tests/Unit/Filament/ClientResourceTest.php @@ -9,7 +9,9 @@ use App\Models\Client; use App\Models\User; use Illuminate\Support\Facades\Config; use Livewire\Livewire; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(ClientResource::class)] class ClientResourceTest extends FilamentTestCase { protected function setUp(): void diff --git a/tests/Unit/Filament/OrganizationResourceTest.php b/tests/Unit/Filament/OrganizationResourceTest.php index 3a55f025..63a98be2 100644 --- a/tests/Unit/Filament/OrganizationResourceTest.php +++ b/tests/Unit/Filament/OrganizationResourceTest.php @@ -11,7 +11,9 @@ use App\Service\DeletionService; use Illuminate\Support\Facades\Config; use Livewire\Livewire; use Mockery\MockInterface; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(OrganizationResource::class)] class OrganizationResourceTest extends FilamentTestCase { protected function setUp(): void diff --git a/tests/Unit/Filament/ProjectResourceTest.php b/tests/Unit/Filament/ProjectResourceTest.php index 8461a5cc..cddeba13 100644 --- a/tests/Unit/Filament/ProjectResourceTest.php +++ b/tests/Unit/Filament/ProjectResourceTest.php @@ -9,7 +9,9 @@ use App\Models\Project; use App\Models\User; use Illuminate\Support\Facades\Config; use Livewire\Livewire; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(ProjectResource::class)] class ProjectResourceTest extends FilamentTestCase { protected function setUp(): void diff --git a/tests/Unit/Filament/TagResourceTest.php b/tests/Unit/Filament/TagResourceTest.php index a6cf9db3..29d64ab2 100644 --- a/tests/Unit/Filament/TagResourceTest.php +++ b/tests/Unit/Filament/TagResourceTest.php @@ -9,7 +9,9 @@ use App\Models\Tag; use App\Models\User; use Illuminate\Support\Facades\Config; use Livewire\Livewire; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(TagResource::class)] class TagResourceTest extends FilamentTestCase { protected function setUp(): void diff --git a/tests/Unit/Filament/TaskResourceTest.php b/tests/Unit/Filament/TaskResourceTest.php index 2c223ef8..9e27d5f6 100644 --- a/tests/Unit/Filament/TaskResourceTest.php +++ b/tests/Unit/Filament/TaskResourceTest.php @@ -9,7 +9,9 @@ use App\Models\Task; use App\Models\User; use Illuminate\Support\Facades\Config; use Livewire\Livewire; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(TaskResource::class)] class TaskResourceTest extends FilamentTestCase { protected function setUp(): void diff --git a/tests/Unit/Filament/TimeEntryResourceTest.php b/tests/Unit/Filament/TimeEntryResourceTest.php index 2aa469bb..583f10f4 100644 --- a/tests/Unit/Filament/TimeEntryResourceTest.php +++ b/tests/Unit/Filament/TimeEntryResourceTest.php @@ -9,7 +9,9 @@ use App\Models\TimeEntry; use App\Models\User; use Illuminate\Support\Facades\Config; use Livewire\Livewire; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(TimeEntryResource::class)] class TimeEntryResourceTest extends FilamentTestCase { protected function setUp(): void diff --git a/tests/Unit/Filament/UserResourceTest.php b/tests/Unit/Filament/UserResourceTest.php index 30ecc61d..c1ff912d 100644 --- a/tests/Unit/Filament/UserResourceTest.php +++ b/tests/Unit/Filament/UserResourceTest.php @@ -5,13 +5,16 @@ declare(strict_types=1); namespace Tests\Unit\Filament; use App\Exceptions\Api\CanNotDeleteUserWhoIsOwnerOfOrganizationWithMultipleMembers; +use App\Filament\Resources\TimeEntryResource; use App\Filament\Resources\UserResource; use App\Models\User; use App\Service\DeletionService; use Illuminate\Support\Facades\Config; use Livewire\Livewire; use Mockery\MockInterface; +use PHPUnit\Framework\Attributes\UsesClass; +#[UsesClass(TimeEntryResource::class)] class UserResourceTest extends FilamentTestCase { protected function setUp(): void