From 5a8a194be30f0bf9df0d36b2729a2385504437eb Mon Sep 17 00:00:00 2001 From: "Jan-Peter v. Hunnius" Date: Fri, 17 Jul 2026 07:20:19 +0200 Subject: [PATCH] fix: make register_bot_type idempotent (skip instead of panic on duplicate) Prevents test panics when multiple strategies register the same bot type in parallel test threads. --- holdem_core/src/game/bot_factory.rs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/holdem_core/src/game/bot_factory.rs b/holdem_core/src/game/bot_factory.rs index 5e6e88c..94d0ab8 100644 --- a/holdem_core/src/game/bot_factory.rs +++ b/holdem_core/src/game/bot_factory.rs @@ -65,10 +65,11 @@ where { let mut map = CUSTOM_FACTORIES.write().unwrap_or_else(|e| e.into_inner()); if map.contains_key(bot_type) { - panic!( - "bot type '{}' is already registered; choose a different name", + log::debug!( + "bot type '{}' is already registered; skipping re-registration", bot_type ); + return; } map.insert(bot_type.to_string(), Box::new(factory)); }