From e447542957ad97f6f5e86207e52dfb9ef1835a31 Mon Sep 17 00:00:00 2001 From: DawnFire42 Date: Thu, 9 Jul 2026 14:16:55 -0400 Subject: [PATCH] Add test case to pin tax counter ID boundary --- tests/command_zone_tests/new_counter_id_test.cpp | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/tests/command_zone_tests/new_counter_id_test.cpp b/tests/command_zone_tests/new_counter_id_test.cpp index ddf627983..f5a56a8b4 100644 --- a/tests/command_zone_tests/new_counter_id_test.cpp +++ b/tests/command_zone_tests/new_counter_id_test.cpp @@ -49,6 +49,14 @@ TEST(NewCounterId, SkipsReservedRangeWhenOnlyReservedCountersExist) EXPECT_EQ(f.player.newCounterId(), CounterIds::FirstUserId); } +TEST(NewCounterId, SkipsTaxCounterIds) +{ + PlayerFixture f; + f.player.addCounter(new Server_Counter(CounterIds::CommanderTax, "tax1", color(), 0, 0)); + f.player.addCounter(new Server_Counter(CounterIds::PartnerTax, "tax2", color(), 0, 0)); + EXPECT_EQ(f.player.newCounterId(), CounterIds::FirstUserId); +} + TEST(NewCounterId, ReturnsNextIdAboveHighestUserCounter) { PlayerFixture f;