From 9192d94dea6be5498ab080f3e2df5667af871d7a Mon Sep 17 00:00:00 2001 From: historia <[not public]> Date: Wed, 16 Sep 2026 14:41:15 -0400 Subject: fix: casino room eviction, session close race fix, open-payout recovery evidence --- internal/admin/api_rooms_casino_evict_test.go | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 internal/admin/api_rooms_casino_evict_test.go (limited to 'internal/admin/api_rooms_casino_evict_test.go') diff --git a/internal/admin/api_rooms_casino_evict_test.go b/internal/admin/api_rooms_casino_evict_test.go new file mode 100644 index 0000000..426c13a --- /dev/null +++ b/internal/admin/api_rooms_casino_evict_test.go @@ -0,0 +1,27 @@ +package admin + +// Regression: deleting a single room through the editor API must evict the +// room's cached casino games and refund their outstanding wagers, exactly +// like the bulk-delete path does. Skipping eviction strands seated players' +// wagers in games whose room no longer exists. + +import ( + "net/http" + "net/http/httptest" + "testing" +) + +func TestRoomDeleteEvictsCasinoGames(t *testing.T) { + s := newTestAdminServer(t, map[int]string{1: "id: 1\nname: Room\n"}) + var evicted []int + s.evictRoom = func(id int) { evicted = append(evicted, id) } + + rec := httptest.NewRecorder() + s.handleRoomByID(rec, httptest.NewRequest(http.MethodDelete, "/api/rooms/1", nil)) + if _, ok := s.world.GetRoomPath(1); ok { + t.Fatalf("room was not removed: %s", rec.Body.String()) + } + if len(evicted) != 1 || evicted[0] != 1 { + t.Fatalf("room deletion evicted %v, want [1] (outstanding wagers were not resolved)", evicted) + } +} -- cgit v1.2.3