jan/rusty_marvin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix: make register_bot_type idempotent (skip instead of panic on duplicate)

Browse source 
Prevents test panics when multiple strategies register the same bot type
in parallel test threads.
This commit is contained in:
Jan-Peter von Hunnius 2026-07-17 07:20:19 +02:00
parent e5e56fd772
commit 5a8a194be3
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
holdem_core/src/game/bot_factory.rs
View file

@ -65,10 +65,11 @@ where
{ {
let mut map = CUSTOM_FACTORIES.write().unwrap_or_else(|e| e.into_inner()); let mut map = CUSTOM_FACTORIES.write().unwrap_or_else(|e| e.into_inner());
if map.contains_key(bot_type) { if map.contains_key(bot_type) {
panic!( log::debug!(
"bot type '{}' is already registered; choose a different name", "bot type '{}' is already registered; skipping re-registration",
bot_type bot_type
); );
return;
} }
map.insert(bot_type.to_string(), Box::new(factory)); map.insert(bot_type.to_string(), Box::new(factory));
} }