feat: record page view for names (#53)

app/Http/Controllers/NameController.php

@@ -6,6 +6,7 @@
 use App\Http\ViewModels\Names\NameViewModel;
 use App\Http\ViewModels\User\ListViewModel;
 use App\Http\ViewModels\User\UserViewModel;
+use App\Jobs\IncrementPageViewForName;
 use App\Models\Name;
 use App\Services\ToggleNameToNameList;
 use Illuminate\Contracts\View\View;
@@ -65,6 +66,8 @@ public function show(Request $request): View
             $note = $requestedName->getNoteForUser();
         }
 
+        IncrementPageViewForName::dispatch($requestedName->id);
+
         return view('names.show', [
             'name' => $name,
             'popularity' => $popularity,

app/Jobs/IncrementPageViewForName.php

@@ -0,0 +1,31 @@
+<?php
+
+namespace App\Jobs;
+
+use Illuminate\Bus\Queueable;
+use Illuminate\Contracts\Queue\ShouldQueue;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\InteractsWithQueue;
+use Illuminate\Queue\SerializesModels;
+use Illuminate\Support\Facades\DB;
+
+class IncrementPageViewForName implements ShouldQueue
+{
+    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
+
+    public function __construct(
+        public int $nameId
+    ) {
+    }
+
+    public function handle(): void
+    {
+        // we use the DB facade and not the model for performance reasons
+        // if we call the model inside the job, the model will be serialized
+        // and will consume resources. we don't need to serialize the model for
+        // this operation.
+        DB::table('names')
+            ->where('id', $this->nameId)
+            ->increment('page_views');
+    }
+}

tests/Unit/Jobs/IncrementPageViewForNameTest.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace Tests\Unit\Jobs;
+
+use App\Jobs\IncrementPageViewForName;
+use App\Models\Name;
+use Illuminate\Foundation\Testing\DatabaseTransactions;
+use Tests\TestCase;
+
+class IncrementPageViewForNameTest extends TestCase
+{
+    use DatabaseTransactions;
+
+    /** @test */
+    public function it_increments_the_page_view_for_a_name(): void
+    {
+        $name = Name::factory()->create();
+
+        IncrementPageViewForName::dispatch($name->id);
+
+        $this->assertDatabaseHas('names', [
+            'id' => $name->id,
+            'page_views' => 2,
+        ]);
+    }
+}