Fix tests

PennyDreadfulMTG · Nov 1, 2023 · c75e571 · c75e571
1 parent 9a7d178
commit c75e571
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/gatherling/models/Event.php b/gatherling/models/Event.php
@@ -680,7 +680,7 @@ public function getEntries()
      *
      * @return Entry[]
      */
-    public function getRegisteredEntries($deleteinvalid = false)
+    public function getRegisteredEntries($deleteinvalid = false, $skip_invalid = false)
     {
         $players = $this->getPlayers();
 
@@ -692,6 +692,9 @@ public function getRegisteredEntries($deleteinvalid = false)
                     $entry->removeEntry($player);
                     continue;
                 }
+                if ($skip_invalid) {
+                    continue;
+                }
             }
             $entries[] = $entry;
         }

diff --git a/tests/EventsTest.php b/tests/EventsTest.php
@@ -112,7 +112,7 @@ public function testRegistration($event)
         // 8 players have expressed interest in the event.
         $this->assertEquals(10, count($event->getEntries()));
         // No players have filled out decklists.
-        $this->assertEquals(0, count($event->getRegisteredEntries()));
+        $this->assertEquals(0, count($event->getRegisteredEntries(false, true)));
 
         $deck = insertDeck('testplayer0', $event, '60 Plains', '');
         $this->assertEmpty($deck->errors, json_encode($deck->errors));
@@ -131,7 +131,7 @@ public function testRegistration($event)
         $deck = insertDeck('testplayer7', $event, "55 Mountain\n5 Seven Dwarves", '5 Seven Dwarves');
         $this->assertNotEmpty($deck->errors, json_encode($deck->errors), 'Too Many Dwarves');
         // 5 Valid decks (0, 1, 2, and 4, 5), 3 invalid deck (3, 6, 7), and 3 not submitted decks.
-        $this->assertEquals(5, count($event->getRegisteredEntries()));
+        $this->assertEquals(5, count($event->getRegisteredEntries(false, true)));
 
         return $event;
     }