fix(output): package

thepercival · Apr 29, 2024 · 576f522 · 576f522
1 parent 5bc0095
commit 576f522
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/composer.lock b/composer.lock
diff --git a/domain/Game/Assigner.php b/domain/Game/Assigner.php
@@ -23,7 +23,7 @@ public function __construct(protected LoggerInterface $logger)
         $this->throwOnTimeout = true;
     }
 
-    public function assignGames(Planning $planning): bool
+    public function assignGames(Planning $planning): PlanningState
     {
         $games = (new PreAssignSorter())->getGames($planning);
         // (new GameOutput($this->logger))->outputGames($games);
@@ -46,7 +46,7 @@ public function assignGames(Planning $planning): bool
             } else {
                 $planning->setTimeoutState(null);
             }
-            return false;
+            return $state;
         }
 
         $firstBatch = $planning->createFirstBatch();
@@ -68,13 +68,13 @@ public function assignGames(Planning $planning): bool
                     $planning->setTimeoutState(null);
                 }
                 $this->logger->error('   could not assign refereeplaces (plId:' . (string)$planning->getId() . ')');
-                return false;
+                return $state;
             }
         }
         $planning->setState(PlanningState::Succeeded);
         $planning->setTimeoutState(null);
         $planning->setNrOfBatches($firstBatch->getLeaf()->getNumber());
-        return true;
+        return PlanningState::Succeeded;
     }
 
     public function disableThrowOnTimeout(): void

diff --git a/tests/helpers/PlanningCreator.php b/tests/helpers/PlanningCreator.php
@@ -175,7 +175,7 @@ protected function createPlanning(
         if ($showHighestCompletedBatchNr) {
             $gameAssigner->showHighestCompletedBatchNr();
         }
-        if( $gameAssigner->assignGames($planning) === false ) {
+        if( $gameAssigner->assignGames($planning) !== PlanningState::Succeeded ) {
             throw new Exception("planning could not be created", E_ERROR);
         }
         return $planning;