From edb11faed02217e0cd4f87b97bd729a84ee6f5dd Mon Sep 17 00:00:00 2001 From: Daniel Krol Date: Tue, 12 Dec 2023 15:11:49 -0500 Subject: [PATCH] WIP better panic messages --- squire_sdk/src/server/gathering/hall.rs | 6 +++--- squire_sdk/src/server/tournaments.rs | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/squire_sdk/src/server/gathering/hall.rs b/squire_sdk/src/server/gathering/hall.rs index 9bd9628b..39e87d2d 100644 --- 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 index a0f8bcc4..2ac910be 100644 --- a/squire_sdk/src/server/tournaments.rs +++ b/squire_sdk/src/server/tournaments.rs @@ -108,6 +108,6 @@ async fn handle_new_onlooker(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") } }