WIP better panic messages

SquireTournamentServices · Dec 12, 2023 · 4d7a7d3 · 4d7a7d3
1 parent f7bff89
commit 4d7a7d3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/squire_sdk/src/server/gathering/hall.rs b/squire_sdk/src/server/gathering/hall.rs
@@ -92,7 +92,7 @@ where
             GatheringHallMessage::NewGathering(id) => self.process_new_gathering(id).await,
             GatheringHallMessage::NewConnection(id, user, ws) => {
                 if !self.process_new_onlooker(id, user, ws).await {
-                    panic!("TODO what if this fails")
+                    panic!("process_new_onlooker failed")
                 }
             }
             GatheringHallMessage::Persist => {
@@ -106,7 +106,7 @@ where
                     let (send, recv) = oneshot_channel();
                     let msg = GatheringMessage::GetTournament(send);
                     if !sender.send(msg) {
-                        panic!("TODO what if this fails")
+                        panic!("GatheringMessage::GetTournament failed")
                     }
                     let tourn = recv.await.unwrap();
                     let _ = persist_reqs.insert(id, tourn);
@@ -116,7 +116,7 @@ where
                     .drain()
                     .for_each(|(_, tourn)| {
                         if !self.persister.send(tourn) {
-                            panic!("TODO what if this fails")
+                            panic!("Failed to persist tournament")
                         };
                     });
                 schedule_persist(scheduler);

diff --git a/squire_sdk/src/server/tournaments.rs b/squire_sdk/src/server/tournaments.rs
@@ -108,6 +108,6 @@ async fn handle_new_onlooker<S: ServerState>(state: S, id: TournamentId, mut ws:
     };
     let user = state.watch_session(session).await.unwrap();
     if !state.handle_new_onlooker(id, user, ws).await {
-        panic!("Failed to handle new onlooker")
+        panic!("handle_new_onlooker failed")
     }
 }