feat: (LAR-87) Update spamReportsTest, ReportSpamAction and ReportSpam

Author: Stephen2304

app/Actions/SpamReport/ReportSpamAction.php

@@ -12,31 +12,33 @@
 
 class ReportSpamAction
 {
-    public function execute(int $contentId, string $contentType, string $reason): SpamReport
+    public function execute(array $data): void
     {
-        $user = Auth::user();
+        $user = $data['user'];
+        $recordId = $data['recordId'];
+        $recordType = $data['recordType'];
+        $reason = $data['reason'];
 
-        if (! $user->hasVerifiedEmail()) {
-            throw new \Exception('Vous devez avoir un email vérifié pour signaler du contenu.');
-        }
+        $alreadyReported = SpamReport::where([
+            'user_id' => $user->id,
+            'reportable_id' => $recordId,
+            'reportable_type' => $recordType,
+        ])->exists();
 
-        if (SpamReport::where('user_id', $user->id)
-            ->where('reportable_id', $contentId)
-            ->where('reportable_type', $contentType)
-            ->exists()) {
-            throw new \Exception('Vous avez déjà signalé ce contenu.');
+        if ($alreadyReported) {
+            throw new \Exception('already_reported');
         }
 
-        $report = SpamReport::create([
+        SpamReport::create([
             'user_id' => $user->id,
-            'reportable_id' => $contentId,
-            'reportable_type' => $contentType,
+            'reportable_id' => $recordId,
+            'reportable_type' => $recordType,
             'reason' => $reason,
         ]);
 
-        $adminsAndModerators = User::whereIn('role', ['admin', 'moderator'])->get();
-        Notification::send($adminsAndModerators, new ContentReportedSpamNotification($contentId, $contentType));
-
-        return $report;
+        Notification::send(
+            User::whereIn('role', ['admin', 'moderator'])->get(),
+            new ContentReportedSpamNotification($recordId, $recordType)
+        );
     }
 }

app/Livewire/ReportSpam.php

@@ -4,12 +4,10 @@
 
 namespace App\Livewire;
 
-use App\Models\User;
-use Livewire\Component;
+use App\Actions\SpamReport\ReportSpamAction;
 use App\Models\SpamReport;
 use Illuminate\Support\Facades\Auth;
-use Illuminate\Support\Facades\Notification;
-use App\Notifications\ContentReportedSpamNotification;
+use Livewire\Component;
 
 final class ReportSpam extends Component
 {
@@ -48,33 +46,33 @@ public function report(): void
     {
         $user = Auth::user();
 
-        if (!$user) {
+        if (! $user) {
             abort(403, 'Vous devez être connecté pour signaler du contenu.');
         }
 
-        if (!$user->hasVerifiedEmail()) {
+        if (! $user->hasVerifiedEmail()) {
             abort(403, 'Vous devez avoir un email vérifié pour signaler du contenu');
         }
 
-        if ($this->alreadyReported) {
-            $this->addError('alreadyReported', 'Vous avez déjà signalé ce contenu.');
-
-            return;
-        }
+        try {
+            app(ReportSpamAction::class)->execute([
+                'user' => $user,
+                'recordId' => $this->recordId,
+                'recordType' => $this->recordType,
+                'reason' => $this->reason,
+            ]);
 
-        SpamReport::create([
-            'user_id' => Auth::id(),
-            'reportable_id' => $this->recordId,
-            'reportable_type' => $this->recordType,
-            'reason' => $this->reason,
-        ]);
+            $this->reset('reason');
 
-        Notification::send(User::whereIn('role', ['admin', 'moderator'])->get(), new ContentReportedSpamNotification($this->recordId, $this->recordType));
+            session()->flash('message', 'Signalement créé avec succès');
 
-        $this->alreadyReported = true;
-        $this->reset('reason');
-
-        session()->flash('message', 'Signalement créé avec succès');
+        } catch (\Exception $e) {
+            if ($e->getMessage() === 'already_reported') {
+                $this->addError('alreadyReported', 'Vous avez déjà signalé ce contenu.');
+            } else {
+                session()->flash('error', 'Une erreur est survenue lors du signalement.');
+            }
+        }
     }
 
     public function render()

resources/views/livewire/components/spam-report.blade.php

@@ -8,6 +8,12 @@
             <button class="btn btn-link text-danger" wire:click="openModal">Signaler</button>
         @endif
     @endcan
+    
+    @if ($errors->has('alreadyReported'))
+        <div class="text-danger">
+            {{ $errors->first('alreadyReported') }}
+        </div>
+    @endif
 
     <!-- Modal -->
     @if ($isModalOpen)
@@ -39,7 +45,6 @@ class="form-control"
         </div>
     @endif
 
-    <!-- Messages de feedback -->
     @if (session()->has('message'))
         <div class="alert alert-success mt-2">
             {{ session('message') }}

tests/Feature/Actions/SpamReports/SpamReportsTest.php

@@ -5,8 +5,10 @@
 use App\Models\User;
 use App\Models\Thread;
 use Livewire\Livewire;
+use App\Models\SpamReport;
 use App\Livewire\ReportSpam;
 use Spatie\Permission\Models\Role;
+use App\Actions\SpamReport\ReportSpamAction;
 use Illuminate\Support\Facades\Notification;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use App\Notifications\ContentReportedSpamNotification;
@@ -45,39 +47,48 @@
 
     $thread = Thread::factory()->create();
 
+    $this->mock(ReportSpamAction::class)
+        ->shouldReceive('execute')
+        ->once()
+        ->with([
+            'user' => $user,
+            'recordId' => $thread->id,
+            'recordType' => Thread::class,
+            'reason' => 'Contenu inapproprié',
+        ]);
+
     Livewire::test(ReportSpam::class, [
         'recordId' => $thread->id,
         'recordType' => Thread::class,
     ])
     ->set('reason', 'Contenu inapproprié')
     ->call('report');
+});
 
-    $this->assertDatabaseHas('spam_reports', [
+it('does not allow a user to report the same content multiple times', function (): void {
+    $user = User::factory()->create();
+    $this->actingAs($user);
+
+    $thread = Thread::factory()->create();
+
+    SpamReport::create([
         'user_id' => $user->id,
         'reportable_id' => $thread->id,
         'reportable_type' => Thread::class,
         'reason' => 'Contenu inapproprié',
     ]);
-});
 
-// it('does not allow a user to report the same content multiple times', function (): void {
-//     $user = User::factory()->create();
-//     $this->actingAs($user);
+    $this->mock(ReportSpamAction::class)
+        ->shouldNotReceive('execute');
 
-//     $thread = Thread::factory()->create();
-
-    // Livewire::test(ReportSpam::class, [
-    //     'recordId' => $thread->id,
-    //     'recordType' => Thread::class,
-    // ])
-    // ->set('reason', 'Contenu inapproprié')
-    // ->call('report')
-    // ->assertSessionHas('message', 'Signalement créé avec succès');
-
-//     Livewire::test('spam-report-content', ['contentId' => $thread->id, 'contentType' => Thread::class])
-//         ->call('spamReport', 'Contenu toujours inapproprié')
-//         ->assertSee('Vous avez déjà signalé ce contenu.');
-// });
+    Livewire::test(ReportSpam::class, [
+        'recordId' => $thread->id,
+        'recordType' => Thread::class,
+    ])
+    ->set('reason', 'Même Contenu inapproprié')
+    ->call('report')
+    ->assertSee('Vous avez déjà signalé ce contenu.');
+});
 
 // it('shows a reported badge only to the user who reported the content', function (): void {
 //     $reporter = User::factory()->create();