From e42952469dadfb3535633c66c881050ac128703c Mon Sep 17 00:00:00 2001
From: historia <[not public]>
Date: Mon, 5 Oct 2026 17:05:14 -0400
Subject: fix: gather verbs, walk oom, admin path bypass, roulette refund

---
 internal/player/hardening_removeitem_test.go | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

(limited to 'internal/player/hardening_removeitem_test.go')

diff --git a/internal/player/hardening_removeitem_test.go b/internal/player/hardening_removeitem_test.go
index 5cb1e4c..b891d4e 100644
--- a/internal/player/hardening_removeitem_test.go
+++ b/internal/player/hardening_removeitem_test.go
@@ -21,3 +21,17 @@ func TestRemoveItemRejectsNonPositive(t *testing.T) {
 		t.Fatalf("stack mutated by a zero removal: %+v", slot)
 	}
 }
+
+// Regression: RemoveItem used to drain every stack and only then return false,
+// so a caller that rolled back on failure lost the items anyway.
+func TestRemoveItemIsAllOrNothing(t *testing.T) {
+	p := New("alice")
+	p.SetInvSlot(0, &InventorySlot{ItemID: "coin", Quantity: 2})
+	p.SetInvSlot(1, &InventorySlot{ItemID: "coin", Quantity: 3})
+	if p.RemoveItem("coin", 10) {
+		t.Fatal("removal of more than held reported success")
+	}
+	if got := p.CountItem("coin"); got != 5 {
+		t.Fatalf("failed removal drained stacks: count = %d, want 5", got)
+	}
+}
-- 
cgit v1.2.3