From 144769a14183492a5b73ede4c6e0c0109038416e Mon Sep 17 00:00:00 2001
From: Ixrec <ericrhitchcock@gmail.com>
Date: Tue, 30 Jan 2024 08:00:47 +0000
Subject: [PATCH] Tests: use strict equality in some tests # (#2778)

* Tests: replace .assertLess/GreaterEqual() with .assertEqual() in two tests where strict equality seems more correct
---
 test/general/test_items.py     | 8 ++++----
 test/general/test_locations.py | 8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/test/general/test_items.py b/test/general/test_items.py
index bd6c3fd85305..1612937225f2 100644
--- a/test/general/test_items.py
+++ b/test/general/test_items.py
@@ -43,15 +43,15 @@ def test_item_name_group_conflict(self):
                     with self.subTest(group_name, group_name=group_name):
                         self.assertNotIn(group_name, world_type.item_name_to_id)
 
-    def test_item_count_greater_equal_locations(self):
-        """Test that by the pre_fill step under default settings, each game submits items >= locations"""
+    def test_item_count_equal_locations(self):
+        """Test that by the pre_fill step under default settings, each game submits items == locations"""
         for game_name, world_type in AutoWorldRegister.world_types.items():
             with self.subTest("Game", game=game_name):
                 multiworld = setup_solo_multiworld(world_type)
-                self.assertGreaterEqual(
+                self.assertEqual(
                     len(multiworld.itempool),
                     len(multiworld.get_unfilled_locations()),
-                    f"{game_name} Item count MUST meet or exceed the number of locations",
+                    f"{game_name} Item count MUST match the number of locations",
                 )
 
     def test_items_in_datapackage(self):
diff --git a/test/general/test_locations.py b/test/general/test_locations.py
index 725b48e62f72..2ac059312c17 100644
--- a/test/general/test_locations.py
+++ b/test/general/test_locations.py
@@ -11,14 +11,14 @@ def test_create_duplicate_locations(self):
             multiworld = setup_solo_multiworld(world_type)
             locations = Counter(location.name for location in multiworld.get_locations())
             if locations:
-                self.assertLessEqual(locations.most_common(1)[0][1], 1,
-                                     f"{world_type.game} has duplicate of location name {locations.most_common(1)}")
+                self.assertEqual(locations.most_common(1)[0][1], 1,
+                                 f"{world_type.game} has duplicate of location name {locations.most_common(1)}")
 
             locations = Counter(location.address for location in multiworld.get_locations()
                                 if type(location.address) is int)
             if locations:
-                self.assertLessEqual(locations.most_common(1)[0][1], 1,
-                                     f"{world_type.game} has duplicate of location ID {locations.most_common(1)}")
+                self.assertEqual(locations.most_common(1)[0][1], 1,
+                                 f"{world_type.game} has duplicate of location ID {locations.most_common(1)}")
 
     def test_locations_in_datapackage(self):
         """Tests that created locations not filled before fill starts exist in the datapackage."""