app/Models/Card.php: in getBySettings() implement support to include X number of jackpot cards in the result.

app/Models/Card.php

@@ -78,17 +78,30 @@ class Card extends Model
     /**
      * Get cards depending on settings.
      */
-    public static function getBySettings(GameSettings $settings, ?int $max = null)
+    public static function getBySettings(GameSettings $settings, ?int $max = null, int $num_jackpot = 0)
     {
         $query = self::settingsQuery($settings)
-            ->inRandomOrder();
+            ->where('jackpot', '=', 0);
 
-        if ($max !== null) {
+        $jitems = \collect([]);
-            $query->limit($max);
+        if ($num_jackpot > 0) {
+            $jitems = self::settingsQuery($settings)
+                ->where('jackpot', '!=', 0)
+                ->limit($num_jackpot)
+                ->get();
         }
 
-        return $query->get()
+        if ($max !== null) {
-            ->makeHidden(['character', 'class', 'deleted_at']);
+            $query->limit($max - count($jitems));
+        }
+
+        if (count($jitems) > 0) {
+            $items = $query->get()->merge($jitems)->shuffle();
+        } else {
+            $items = $query->inRandomOrder()->get();
+        }
+
+        return $items->makeHidden(['character', 'class', 'deleted_at']);
     }
 
     protected static function settingsQuery(GameSettings $settings): Builder

tests/Feature/Models/CardTest.php

@@ -155,4 +155,22 @@ class CardTest extends TestCase
             $this->assertContains($card->role, $expected);
         }
     }
+
+    public function test_get_with_jackpot_cards()
+    {
+        Card::factory()->count(10)->create(['jackpot' => 1]);
+
+        $cards = Card::getBySettings(new GameSettings(), 4, 2);
+
+        $this->assertSame(4, count($cards));
+
+        $jackpot_count = 0;
+        foreach ($cards as $card) {
+            if ($card->jackpot != 0) {
+                $jackpot_count++;
+            }
+        }
+
+        $this->assertEquals(2, $jackpot_count);
+    }
 }