feat:[lar-140] add transaction and add feature test to remove point

Author: cybersoldattech

app/Livewire/Pages/Discussions/SingleDiscussion.php

@@ -14,6 +14,7 @@
 use Filament\Forms\Concerns\InteractsWithForms;
 use Filament\Forms\Contracts\HasForms;
 use Illuminate\Contracts\View\View;
+use Illuminate\Support\Facades\DB;
 use Livewire\Component;
 
 final class SingleDiscussion extends Component implements HasActions, HasForms
@@ -81,8 +82,17 @@ public function deleteAction(): Action
             ->authorize('delete', $this->discussion)
             ->requiresConfirmation()
             ->successNotificationTitle(__('notifications.discussion.deleted'))
-            ->successRedirectUrl(route('discussions.index'))
-            ->after(fn ($record) => undoPoint(new DiscussionCreated($record)));
+            ->action(function (): void {
+                DB::beginTransaction();
+
+                undoPoint(new DiscussionCreated($this->discussion));
+
+                $this->discussion->delete();
+
+                DB::commit();
+
+                $this->redirectRoute('discussions.index',  navigate: true);
+            });
     }
 
     public function render(): View

app/Livewire/Pages/Forum/DetailThread.php

@@ -12,6 +12,7 @@
 use Filament\Forms\Concerns\InteractsWithForms;
 use Filament\Forms\Contracts\HasForms;
 use Illuminate\Contracts\View\View;
+use Illuminate\Support\Facades\DB;
 use Livewire\Attributes\Layout;
 use Livewire\Attributes\On;
 use Livewire\Component;
@@ -54,9 +55,12 @@ public function deleteAction(): Action
             ->authorize('delete', $this->thread)
             ->requiresConfirmation()
             ->action(function (): void {
-                $this->thread->delete();
+                DB::beginTransaction();
 
                 undoPoint(new ThreadCreated($this->thread));
+                $this->thread->delete();
+
+                DB::commit();
 
                 $this->redirectRoute('forum.index', navigate: true);
             });

composer.lock

@@ -22,7 +22,7 @@
             ->assertSuccessful();
     });
 
-    it('user can create a new thread', function (): void {
+    it('user can create a new discussion', function (): void {
         $user = $this->login();
         $tags = Tag::factory()->count(3)->create();
 
@@ -36,10 +36,13 @@
             ->assertHasNoFormErrors();
 
         $discussion = Discussion::query()->first();
+        $user->refresh();
 
         expect($discussion?->user)->toBeInstanceOf(User::class)
             ->and($discussion?->user->is($user))
-            ->toBeTrue();
+            ->toBeTrue()
+            ->and($user->getPoints())
+            ->toBe(20);
     });
 
     it('validate forms input', function (): void {

tests/Feature/Livewire/Components/Slideovers/DiscussionFormTest.php

@@ -75,10 +75,13 @@
             ->assertHasNoFormErrors();
 
         $thread = Thread::query()->first();
+        $user->refresh();
 
         expect($thread?->user)->toBeInstanceOf(User::class)
             ->and($thread?->user->is($user))
-            ->toBeTrue();
+            ->toBeTrue()
+            ->and($user->getPoints())
+            ->toBe(55);
     });
 
     it('user cannot create thread with and unverified email address', function (): void {

tests/Feature/Livewire/Components/Slideovers/ThreadFormTest.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+
+use App\Gamify\Points\DiscussionCreated;
+use App\Livewire\Pages\Discussions\SingleDiscussion;
+use App\Models\Discussion;
+use App\Models\Tag;
+use Livewire\Livewire;
+
+it('delete user action can remove discussion point ', function (): void {
+    $user = $this->login();
+    $discussion = Discussion::factory()->create(['user_id' => $user->id]);
+    $tags = Tag::factory()->count(3)->create();
+
+    $discussion->tags()->attach($tags->modelKeys());
+
+    givePoint(new DiscussionCreated($discussion));
+
+    Livewire::test(SingleDiscussion::class, ['discussion' => $discussion])
+        ->callAction('deleteAction')
+        ->assertStatus(200);
+
+    $user->refresh();
+
+    expect($user->getPoints())
+        ->toBe(0);
+
+});