Merge pull request #48 from 7OMI/patch-1

fix: update level_id in experiences table when levelled up
cjmellor · Oct 4, 2023 · e809a42 · e809a42
2 parents 304cfb3 + 59f73bb
commit e809a42
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/Concerns/GiveExperience.php b/src/Concerns/GiveExperience.php
@@ -211,7 +211,7 @@ public function levelUp(int $to): void
             ->save();
 
         $this->experience->status()->associate(model: $to);
-        $this->save();
+        $this->experience->save();
 
         // Fire an event for each level gained
         for ($lvl = $this->getLevel(); $lvl <= $to; $lvl++) {

diff --git a/tests/Concerns/GiveExperienceTest.php b/tests/Concerns/GiveExperienceTest.php
@@ -50,6 +50,17 @@
     ]);
 });
 
+test(description: 'levels are associated on point increments', closure: function () {
+    $this->user->addPoints(amount: 10);
+
+    expect($this->user)->level_id->toBe(expected: 1);
+
+    $this->user->addPoints(amount: 100);
+
+    expect($this->user)->level_id->toBe(expected: 2)
+        ->and($this->user)->getLevel()->toBe(expected: 2);
+});
+
 it(description: 'can deduct points from a User', closure: function (): void {
     Event::fake();