Minor refactor and comment updates

Author: outdooracorn

database/migrations/2026_06_22_083853_create_policies_table.php

@@ -18,7 +18,7 @@ public function up(): void {
             // Use Eloquent built in to create nullable `created_at` and `updated_at` timestamp fields
             $table->timestamps();
 
-            // This prevents two policies of the same type with `active_from` set to the same value
+            // Prevent two policies of the same type with `active_from` set to the same value
             // (including `null` for upcoming policies)
             $table->unique(['policy_type', 'active_from']);
         });

database/migrations/2026_06_22_083910_create_policy_acceptances_table.php

@@ -18,10 +18,10 @@ public function up(): void {
 
             $table->foreignId('policy_id')->constrained()->restrictOnUpdate()->restrictOnDelete();
 
-            // Using a separate `accepted_at` column rather than renaming the default `created_at` column because:
+            // Use a separate `accepted_at` column rather than renaming the default `created_at` column because:
             //   * it reduces confusion by remaining consistent with other tables that use these default columns
             //   * `accepted_at` will be before `created_at` when backfilling the terms-of-use acceptances
-            // Using a DATETIME field as MariaDB has odd behavior around TIMESTAMPs and default values
+            // Use a DATETIME field as MariaDB has odd behavior around TIMESTAMPs and default values
             // `accepted_at` can be NOT NULL if we use a DATETIME which provides us with more data safety
             $table->dateTime('accepted_at');
 

tests/PolicyAcceptanceTest.php

@@ -20,23 +20,20 @@ protected function setUp(): void {
         parent::setUp();
         $user = User::factory()->create();
         $this->userId = $user->id;
-        $policy = Policy::create(
-            [
-                'policy_type' => 'terms-of-use',
-                'active_from' => CarbonImmutable::yesterday(),
-                'content_vue_file' => 'terms-of-use/example.vue',
-            ]);
+        $policy = Policy::create([
+            'policy_type' => 'terms-of-use',
+            'active_from' => CarbonImmutable::yesterday(),
+            'content_vue_file' => 'terms-of-use/example.vue',
+        ]);
         $this->policyId = $policy->id;
     }
 
     public function testCreatesAndSavesSuccessfully(): void {
-        $policyAcceptance = new PolicyAcceptance(
-            [
-                'user_id' => $this->userId,
-                'policy_id' => $this->policyId,
-                'accepted_at' => CarbonImmutable::now(),
-            ]
-        );
+        $policyAcceptance = new PolicyAcceptance([
+            'user_id' => $this->userId,
+            'policy_id' => $this->policyId,
+            'accepted_at' => CarbonImmutable::now(),
+        ]);
         $policyAcceptance->save();
         $policyAcceptance->refresh();
 

tests/PolicyTest.php

@@ -10,13 +10,11 @@ class PolicyTest extends TestCase {
     use RefreshDatabase;
 
     public function testCreatesSuccessfully(): void {
-        Policy::create(
-            [
-                'policy_type' => 'terms-of-use',
-                'active_from' => CarbonImmutable::createMidnightDate(2025, 4, 1),
-                'content_vue_file' => 'terms-of-use/example.vue',
-            ]
-        );
+        Policy::create([
+            'policy_type' => 'terms-of-use',
+            'active_from' => CarbonImmutable::createMidnightDate(2025, 4, 1),
+            'content_vue_file' => 'terms-of-use/example.vue',
+        ]);
 
         $this->assertDatabaseHas('policies', [
             'policy_type' => 'terms-of-use',