feat(phase-9): complete REST API layer — routes, controller fixes, and TenantFactory

- Wire all 28 new module route groups into routes/api.php (auth:sanctum protected)
- Fix FinanceApiController: use issue_date (not bill_date) to match bills schema
- Fix PmApiController: status enum matches DB (draft/active/on_hold/completed/cancelled)
- Create TenantFactory for 13 test files that use Tenant::factory()->create()
- Fix test data: add required fields (po_number, slug, reference, unit_price)
- Fix test data: use valid enum values (survey draft, project draft)
- All 101 API tests now pass

Co-Authored-By: Claude <noreply@anthropic.com>

Author: claude

erp/app/Http/Controllers/Api/V1/FinanceApiController.php

@@ -46,7 +46,7 @@ public function store(Request $request): JsonResponse
     {
         $validated = $request->validate([
             'contact_id' => 'required|integer|exists:contacts,id',
-            'bill_date'  => 'required|date',
+            'issue_date' => 'required|date',
             'due_date'   => 'nullable|date',
             'notes'      => 'nullable|string',
         ]);

erp/app/Http/Controllers/Api/V1/PmApiController.php

@@ -45,7 +45,7 @@ public function storeProject(Request $request): JsonResponse
             'name'        => 'required|string|max:255',
             'code'        => 'nullable|string|max:50',
             'description' => 'nullable|string',
-            'status'      => 'nullable|string|in:planning,active,on_hold,completed,cancelled',
+            'status'      => 'nullable|string|in:draft,active,on_hold,completed,cancelled',
             'priority'    => 'nullable|string|in:low,medium,high,critical',
             'budget'      => 'nullable|numeric|min:0',
             'start_date'  => 'nullable|date',

erp/database/factories/Modules/Core/Models/TenantFactory.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace Database\Factories\Modules\Core\Models;
+
+use App\Modules\Core\Models\Tenant;
+use Illuminate\Database\Eloquent\Factories\Factory;
+
+class TenantFactory extends Factory
+{
+    protected $model = Tenant::class;
+
+    public function definition(): array
+    {
+        return [
+            'name'      => $this->faker->company(),
+            'slug'      => $this->faker->unique()->slug(2),
+            'is_active' => true,
+        ];
+    }
+}

erp/routes/api.php

@@ -25,7 +25,7 @@
     Bill::create([
         'tenant_id'  => $this->tenant->id,
         'contact_id' => $contact->id,
-        'bill_date'  => now()->toDateString(),
+        'issue_date' => now()->toDateString(),
     ]);
 
     $response = $this->withToken($this->token)->getJson('/api/v1/finance/bills');
@@ -52,7 +52,7 @@
 
     $response = $this->withToken($this->token)->postJson('/api/v1/finance/bills', [
         'contact_id' => $contact->id,
-        'bill_date'  => now()->toDateString(),
+        'issue_date' => now()->toDateString(),
     ]);
 
     $response->assertStatus(201)
@@ -69,7 +69,7 @@
     $bill = Bill::create([
         'tenant_id'  => $this->tenant->id,
         'contact_id' => $contact->id,
-        'bill_date'  => now()->toDateString(),
+        'issue_date' => now()->toDateString(),
         'status'     => 'paid',
     ]);
 

erp/tests/Feature/Api/ApiFinanceTest.php

@@ -19,6 +19,7 @@
     KbArticle::create([
         'tenant_id' => $this->tenant->id,
         'title'     => 'Test Article',
+        'slug'      => 'test-article',
         'content'   => 'Article content here',
         'status'    => 'published',
         'author_id' => $this->user->id,

erp/tests/Feature/Api/ApiKnowledgeBaseTest.php

@@ -42,7 +42,7 @@
     $response = $this->withToken($this->token)->postJson('/api/v1/pm/projects', [
         'name'        => 'New Project',
         'description' => 'Project description',
-        'status'      => 'planning',
+        'status'      => 'draft',
     ]);
 
     $response->assertStatus(201)

erp/tests/Feature/Api/ApiPmTest.php

@@ -60,6 +60,7 @@
 
     Po::create([
         'tenant_id'    => $this->tenant->id,
+        'po_number'    => 'PO-TEST-001',
         'po_vendor_id' => $vendor->id,
         'status'       => 'draft',
         'order_date'   => now()->toDateString(),
@@ -82,6 +83,7 @@
 
     Po::create([
         'tenant_id'    => $this->tenant->id,
+        'po_number'    => 'PO-TEST-002',
         'po_vendor_id' => $vendor->id,
         'status'       => 'confirmed',
         'order_date'   => now()->toDateString(),

erp/tests/Feature/Api/ApiPurchaseTest.php

@@ -18,8 +18,10 @@
     SubcontractOrder::create([
         'tenant_id'        => $this->tenant->id,
         'vendor_id'        => 1,
+        'reference'        => 'SC-TEST-001',
         'finished_product' => 'Widget A',
         'finished_qty'     => 100,
+        'unit_price'       => 0,
         'status'           => 'draft',
     ]);
 

erp/tests/Feature/Api/ApiSubcontractingTest.php

@@ -18,7 +18,7 @@
     Survey::create([
         'tenant_id'  => $this->tenant->id,
         'title'      => 'Customer Satisfaction',
-        'status'     => 'active',
+        'status'     => 'draft',
         'created_by' => $this->user->id,
     ]);