Implement ticket creation and backlog

2026-03-25 01:33:35 +01:00
parent 35aacff318
commit f7a0543ea1
17 changed files with 873 additions and 3 deletions
@@ -7,6 +7,7 @@ use App\Models\Project;
 use Database\Seeders\BoardColumnSeeder;
 use Database\Seeders\ProjectSeeder;
 use Database\Seeders\ProjectTicketMetadataSeeder;
+use Database\Seeders\TicketSeeder;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Tests\TestCase;

@@ -121,4 +122,23 @@ class ProjectTest extends TestCase
            $coreProject->ticketFieldDefinitions->pluck('label')->all(),
        );
    }
+
+    /**
+     * Ensure projects expose tickets ordered by descending sequence number.
+     */
+    public function test_projects_expose_tickets_ordered_by_sequence_number_desc(): void
+    {
+        $this->seed([
+            ProjectSeeder::class,
+            ProjectTicketMetadataSeeder::class,
+            TicketSeeder::class,
+        ]);
+
+        $coreProject = Project::query()->where('key', 'CORE')->firstOrFail();
+
+        $this->assertSame(
+            ['CORE-2', 'CORE-1'],
+            $coreProject->tickets->pluck('ticket_key')->all(),
+        );
+    }
 }
@@ -0,0 +1,58 @@
+<?php
+
+namespace Tests\Feature\Models;
+
+use App\Models\Project;
+use App\Models\Ticket;
+use Database\Seeders\BoardColumnSeeder;
+use Database\Seeders\ProjectSeeder;
+use Database\Seeders\ProjectTicketMetadataSeeder;
+use Database\Seeders\TicketSeeder;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Tests\TestCase;
+
+class TicketTest extends TestCase
+{
+    use RefreshDatabase;
+
+    /**
+     * Ensure the ticket factory creates backlog tickets with immutable keys.
+     */
+    public function test_ticket_factory_creates_backlog_tickets(): void
+    {
+        $ticket = Ticket::factory()->create();
+
+        $this->assertNull($ticket->board_column_id);
+        $this->assertStringContainsString('-', $ticket->ticket_key);
+        $this->assertSame(1, $ticket->sequence_number);
+    }
+
+    /**
+     * Ensure the ticket seeder creates duplicate titles with project-local keys.
+     */
+    public function test_ticket_seeder_creates_backlog_tickets_with_project_local_keys(): void
+    {
+        $this->seed([
+            ProjectSeeder::class,
+            BoardColumnSeeder::class,
+            ProjectTicketMetadataSeeder::class,
+            TicketSeeder::class,
+        ]);
+
+        $coreProject = Project::query()->where('key', 'CORE')->firstOrFail();
+
+        $this->assertDatabaseHas('tickets', [
+            'project_id' => $coreProject->id,
+            'ticket_key' => 'CORE-1',
+            'title' => 'Seed backlog item',
+            'board_column_id' => null,
+        ]);
+
+        $this->assertDatabaseHas('tickets', [
+            'project_id' => $coreProject->id,
+            'ticket_key' => 'CORE-2',
+            'title' => 'Seed backlog item',
+            'board_column_id' => null,
+        ]);
+    }
+}
@@ -0,0 +1,137 @@
+<?php
+
+namespace Tests\Feature\Projects;
+
+use App\Actions\ProjectPermissions\ProjectPermissionService;
+use App\Actions\ProjectTicketMetadata\InitializeProjectTicketMetadata;
+use App\Models\Project;
+use App\Models\Ticket;
+use App\Models\TicketPriority;
+use App\Models\User;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Inertia\Testing\AssertableInertia as Assert;
+use Tests\TestCase;
+
+class ProjectBacklogControllerTest extends TestCase
+{
+    use RefreshDatabase;
+
+    /**
+     * Ensure users with project view access can open the backlog page.
+     */
+    public function test_users_with_view_permission_can_view_a_projects_backlog(): void
+    {
+        $user = User::factory()->create();
+        $project = Project::factory()->create([
+            'key' => 'AUTH',
+        ]);
+
+        app(InitializeProjectTicketMetadata::class)->handle($project);
+
+        $priority = TicketPriority::factory()->for($project)->create([
+            'name' => 'Medium',
+            'position' => 1,
+        ]);
+
+        $ticket = Ticket::factory()->create([
+            'project_id' => $project->id,
+            'ticket_priority_id' => $priority->id,
+            'ticket_type_id' => $project->ticketTypes()->firstOrFail()->id,
+            'ticket_key' => 'AUTH-1',
+            'sequence_number' => 1,
+            'title' => 'Seed backlog item',
+        ]);
+
+        app(ProjectPermissionService::class)->syncUserPermissions($project, $user, ['view']);
+
+        $this->actingAs($user)
+            ->get(route('projects.backlog.index', $project))
+            ->assertOk()
+            ->assertInertia(fn (Assert $page) => $page
+                ->component('projects/Backlog')
+                ->where('project.id', $project->id)
+                ->where('tickets.0.ticket_key', $ticket->ticket_key)
+                ->where('canCreateTickets', false),
+            );
+    }
+
+    /**
+     * Ensure creating tickets requires the project create permission.
+     */
+    public function test_creating_tickets_requires_project_create_permission(): void
+    {
+        $user = User::factory()->create();
+        $project = Project::factory()->create();
+
+        app(InitializeProjectTicketMetadata::class)->handle($project);
+
+        $priority = TicketPriority::factory()->for($project)->create([
+            'name' => 'Medium',
+            'position' => 1,
+        ]);
+
+        app(ProjectPermissionService::class)->syncUserPermissions($project, $user, ['view']);
+
+        $this->actingAs($user)
+            ->post(route('projects.backlog.store', $project), [
+                'title' => 'New ticket',
+                'description' => 'Description',
+                'acceptance_criteria' => 'Criteria',
+                'ticket_type_id' => $project->ticketTypes()->firstOrFail()->id,
+                'ticket_priority_id' => $priority->id,
+            ])
+            ->assertForbidden();
+    }
+
+    /**
+     * Ensure project-local keys increment and backlog accepts duplicate titles.
+     */
+    public function test_users_with_create_permission_can_add_duplicate_titled_tickets_to_backlog(): void
+    {
+        $user = User::factory()->create();
+        $project = Project::factory()->create([
+            'key' => 'AUTH',
+        ]);
+
+        app(InitializeProjectTicketMetadata::class)->handle($project);
+
+        $priority = TicketPriority::factory()->for($project)->create([
+            'name' => 'High',
+            'position' => 1,
+        ]);
+
+        app(ProjectPermissionService::class)->syncUserPermissions($project, $user, ['create', 'view']);
+
+        $payload = [
+            'title' => 'Same title',
+            'description' => 'Description',
+            'acceptance_criteria' => 'Criteria',
+            'ticket_type_id' => $project->ticketTypes()->firstOrFail()->id,
+            'ticket_priority_id' => $priority->id,
+        ];
+
+        $this->actingAs($user)
+            ->post(route('projects.backlog.store', $project), $payload)
+            ->assertRedirect(route('projects.backlog.index', $project))
+            ->assertSessionHas('status', 'AUTH-1 was added to the backlog.');
+
+        $this->actingAs($user)
+            ->post(route('projects.backlog.store', $project), $payload)
+            ->assertRedirect(route('projects.backlog.index', $project))
+            ->assertSessionHas('status', 'AUTH-2 was added to the backlog.');
+
+        $this->assertDatabaseHas('tickets', [
+            'project_id' => $project->id,
+            'ticket_key' => 'AUTH-1',
+            'title' => 'Same title',
+            'board_column_id' => null,
+        ]);
+
+        $this->assertDatabaseHas('tickets', [
+            'project_id' => $project->id,
+            'ticket_key' => 'AUTH-2',
+            'title' => 'Same title',
+            'board_column_id' => null,
+        ]);
+    }
+}