refactor: update notes feature

Author: mascam97

README.md

@@ -8,7 +8,7 @@ Personal project to learn [Tailwindcss](https://tailwindcss.com/) and [Laravel 8
 
 Due to I have another project with Jetstream [Laravel Livewire Publications](https://github.com/mascam97/laravel-livewire-publications), this project is no longer maintained.
 
-### Achievements :star2:
+### Achievements 2021 :star2:
 
 - Install all the package from Jetstream and Sail.
 - Learn better practices with Docker compose.
@@ -17,6 +17,11 @@ Due to I have another project with Jetstream [Laravel Livewire Publications](htt
 - Create a Middleware to allow request about notes just from its owner.
 - Tested with PHPUnit (**Test-Driven Development**) to Models, CRUD implemented and Middleware.
 
+### Achievements 2023 :star2:
+
+- Refactored some code with best practices
+- Implemented some design patters: QueryBuilder, Data Transfer Object and Actions 
+
 ---
 
 ## Getting Started :rocket:

app/Actions/Notes/UpdateNoteAction.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace App\Actions\Notes;
+
+use App\Dtos\Notes\UpdateNoteData;
+use App\Models\Note;
+
+class UpdateNoteAction
+{
+    public function handle(Note $note, UpdateNoteData $data): Note
+    {
+        if ($data->title){
+            $note->title = $data->title;
+        }
+
+        if ($data->content){
+            $note->content = $data->content;
+        }
+
+        $note->update();
+
+        return $note;
+    }
+}

app/Dtos/Notes/UpdateNoteData.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace App\Dtos\Notes;
+
+use Illuminate\Foundation\Http\FormRequest;
+
+class UpdateNoteData
+{
+    public function __construct(
+        public ?string $title = null,
+        public ?string $content = null,
+    ) {
+    }
+
+    public static function fromRequest(FormRequest $request): self
+    {
+        return new static(
+            title: $request->input('title'),
+            content: $request->input('content'),
+        );
+    }
+}

app/Http/Controllers/NoteController.php

@@ -3,7 +3,9 @@
 namespace App\Http\Controllers;
 
 use App\Actions\Notes\StoreNoteAction;
+use App\Actions\Notes\UpdateNoteAction;
 use App\Dtos\Notes\StoreNoteData;
+use App\Dtos\Notes\UpdateNoteData;
 use App\Exceptions\NoteExceptions;
 use App\Models\Note;
 use App\Models\User;
@@ -67,7 +69,9 @@ public function edit(Note $note): Response
 
     public function update(NoteRequest $request, Note $note): RedirectResponse
     {
-        $note->update($request->all());
+        $data = UpdateNoteData::fromRequest($request);
+
+        (new UpdateNoteAction())->handle($note, $data);
 
         return redirect()->route('notes.index')->with('status', 'Note updated!!');
     }

tests/Unit/Actions/Notes/UpdateNoteActionTest.php

@@ -0,0 +1,51 @@
+<?php
+
+namespace Tests\Unit\Actions\Notes;
+
+use App\Actions\Notes\UpdateNoteAction;
+use App\Dtos\Notes\UpdateNoteData;
+use App\Models\Note;
+use App\Models\User;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Tests\TestCase;
+
+class UpdateNoteActionTest extends TestCase
+{
+    use RefreshDatabase;
+
+    private User $user;
+
+    private Note $note;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->user = User::factory()->create();
+        $this->note = Note::factory()->create([
+            'user_id' => $this->user->getKey(),
+            'title' => 'Old title',
+            'content' => 'Old content',
+        ]);
+    }
+
+    public function test_can_update_the_title_note()
+    {
+        $data = new UpdateNoteData(title: 'New title');
+
+        $note = (new UpdateNoteAction())->handle($this->note, $data);
+
+        $this->assertEquals('New title', $note->title);
+        $this->assertEquals('Old content', $note->content);
+    }
+
+    public function test_can_update_the_content_note()
+    {
+        $data = new UpdateNoteData(content: 'New content');
+
+        $note = (new UpdateNoteAction())->handle($this->note, $data);
+
+        $this->assertEquals('Old title', $note->title);
+        $this->assertEquals('New content', $note->content);
+    }
+}