Refactor User model purge method to delete associated questions sent …

…by the user
pinkary-project · Sep 20, 2024 · 2c67a73 · 2c67a73
1 parent 28b76c5
commit 2c67a73
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/app/Models/User.php b/app/Models/User.php
@@ -252,6 +252,7 @@ public function purge(): void
         $this->notifications()->delete();
 
         $this->questionsReceived->each->delete();
+        $this->questionsSent->each->delete();
 
         $this->delete();
     }

diff --git a/tests/Unit/Models/UserTest.php b/tests/Unit/Models/UserTest.php
@@ -179,7 +179,7 @@
 });
 
 test('purge questions, comments and decendants with user', function () {
-    $user = User::factory()->create();
+    $user = User::factory()->hasQuestionsSent(2)->create();
     Question::factory()
         ->hasChildren(2)
         ->hasDescendants(2)
@@ -188,7 +188,7 @@
             'to_id' => $user->id,
         ]);
 
-    $this->assertDatabaseCount('questions', 5);
+    $this->assertDatabaseCount('questions', 7);
 
     $this->actingAs($user)
         ->delete(route('profile.destroy'), [