tilemap-parser 2.0.0__tar.gz → 2.0.1__tar.gz

{tilemap_parser-2.0.0/src/tilemap_parser.egg-info → tilemap_parser-2.0.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tilemap-parser
-Version: 2.0.0
+Version: 2.0.1
 Summary: Standalone parser/loader for tilemap-editor JSON maps and sprite animation JSON.
 Author: tilemap parser contributors
 Classifier: Programming Language :: Python :: 3

{tilemap_parser-2.0.0 → tilemap_parser-2.0.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "tilemap-parser"
-version = "2.0.0"
+version = "2.0.1"
 description = "Standalone parser/loader for tilemap-editor JSON maps and sprite animation JSON."
 readme = "README.md"
 requires-python = ">=3.10"

{tilemap_parser-2.0.0 → tilemap_parser-2.0.1}/src/tilemap_parser/collision_runner.py

@@ -67,8 +67,8 @@ def point_in_polygon(point: Point, vertices: List[Point]) -> bool:
                 if x <= max(p1x, p2x):
                     if p1y != p2y:
                         xinters = (y - p1y) * (p2x - p1x) / (p2y - p1y) + p1x
-                    if p1x == p2x or x <= xinters:
-                        inside = not inside
+                        if p1x == p2x or x <= xinters:
+                            inside = not inside
         p1x, p1y = p2x, p2y
 
     return inside

{tilemap_parser-2.0.0 → tilemap_parser-2.0.1/src/tilemap_parser.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tilemap-parser
-Version: 2.0.0
+Version: 2.0.1
 Summary: Standalone parser/loader for tilemap-editor JSON maps and sprite animation JSON.
 Author: tilemap parser contributors
 Classifier: Programming Language :: Python :: 3

{tilemap_parser-2.0.0 → tilemap_parser-2.0.1}/src/tilemap_parser.egg-info/SOURCES.txt

@@ -13,4 +13,5 @@ src/tilemap_parser.egg-info/SOURCES.txt
 src/tilemap_parser.egg-info/dependency_links.txt
 src/tilemap_parser.egg-info/requires.txt
 src/tilemap_parser.egg-info/top_level.txt
-tests/test_collision.py
+tests/test_collision.py
+tests/test_collision_runner.py

tilemap_parser-2.0.1/tests/test_collision_runner.py

@@ -0,0 +1,110 @@
+"""
+Tests for tilemap_parser.collision_runner — focusing on correctness of
+geometric utilities, especially the xinters bug fix in point_in_polygon.
+"""
+
+import sys
+from pathlib import Path
+
+import pytest
+
+sys.path.insert(0, str(Path(__file__).parent.parent / "src"))
+
+from tilemap_parser.collision_runner import point_in_polygon, CollisionRunner, MovementMode
+from tilemap_parser.collision import CollisionCache, RectangleShape
+
+
+# ===========================================================================
+# point_in_polygon
+# ===========================================================================
+
+# Simple axis-aligned square: (0,0) -> (10,0) -> (10,10) -> (0,10)
+SQUARE = [(0.0, 0.0), (10.0, 0.0), (10.0, 10.0), (0.0, 10.0)]
+
+# Triangle with a horizontal bottom edge — this is what triggered the xinters bug
+# vertices: (0,0), (10,0), (5,10)
+TRIANGLE_HORIZ_BASE = [(0.0, 0.0), (10.0, 0.0), (5.0, 10.0)]
+
+
+class TestPointInPolygon:
+    def test_inside_square(self):
+        assert point_in_polygon((5.0, 5.0), SQUARE) is True
+
+    def test_outside_square(self):
+        assert point_in_polygon((15.0, 5.0), SQUARE) is False
+        assert point_in_polygon((-1.0, 5.0), SQUARE) is False
+        assert point_in_polygon((5.0, -1.0), SQUARE) is False
+        assert point_in_polygon((5.0, 15.0), SQUARE) is False
+
+    def test_inside_triangle(self):
+        assert point_in_polygon((5.0, 5.0), TRIANGLE_HORIZ_BASE) is True
+
+    def test_outside_triangle(self):
+        assert point_in_polygon((0.5, 9.0), TRIANGLE_HORIZ_BASE) is False
+        assert point_in_polygon((9.5, 9.0), TRIANGLE_HORIZ_BASE) is False
+
+    # --- xinters bug regression tests ---
+    # A polygon with a horizontal edge at y=0. When the ray is cast from a point
+    # whose y equals the horizontal edge's y, the old code would read an
+    # uninitialised / stale xinters and flip `inside` incorrectly.
+
+    def test_horizontal_edge_does_not_use_stale_xinters(self):
+        # Point well inside — must not be flipped by the horizontal base edge
+        assert point_in_polygon((5.0, 5.0), TRIANGLE_HORIZ_BASE) is True
+
+    def test_point_below_horizontal_edge(self):
+        # y=-1 is outside regardless
+        assert point_in_polygon((5.0, -1.0), TRIANGLE_HORIZ_BASE) is False
+
+    def test_polygon_with_multiple_horizontal_edges(self):
+        # Rectangle has horizontal top and bottom edges
+        rect = [(0.0, 0.0), (20.0, 0.0), (20.0, 10.0), (0.0, 10.0)]
+        assert point_in_polygon((10.0, 5.0), rect) is True
+        assert point_in_polygon((10.0, 15.0), rect) is False
+        assert point_in_polygon((-1.0, 5.0), rect) is False
+
+    def test_first_iteration_no_unbound_xinters(self):
+        # Polygon whose very first edge is horizontal — this is the case that
+        # caused an UnboundLocalError before the fix (xinters never assigned
+        # yet when the stale-read branch was reached on i=1).
+        horiz_first = [(0.0, 5.0), (10.0, 5.0), (10.0, 0.0), (0.0, 0.0)]
+        # Point inside
+        assert point_in_polygon((5.0, 2.0), horiz_first) is True
+        # Point outside
+        assert point_in_polygon((5.0, 8.0), horiz_first) is False
+
+
+# ===========================================================================
+# CollisionRunner construction
+# ===========================================================================
+
+class TestCollisionRunnerConstruction:
+    def test_from_game_type_platformer(self):
+        cache = CollisionCache()
+        runner = CollisionRunner.from_game_type("platformer", cache, (32, 32))
+        assert runner.mode == MovementMode.PLATFORMER
+        assert runner.gravity > 0
+
+    def test_from_game_type_topdown(self):
+        cache = CollisionCache()
+        runner = CollisionRunner.from_game_type("topdown", cache, (32, 32))
+        assert runner.mode == MovementMode.SLIDE
+        assert runner.gravity == 0.0
+
+    def test_from_game_type_rpg(self):
+        cache = CollisionCache()
+        runner = CollisionRunner.from_game_type("rpg", cache, (32, 32))
+        assert runner.mode == MovementMode.RPG
+        assert runner.gravity == 0.0
+
+    def test_unknown_game_type_raises(self):
+        cache = CollisionCache()
+        with pytest.raises(ValueError):
+            CollisionRunner.from_game_type("unknown", cache, (32, 32))
+
+    def test_platformer_zero_gravity_raises(self):
+        cache = CollisionCache()
+        runner = CollisionRunner.from_game_type("platformer", cache, (32, 32))
+        runner.gravity = 0.0
+        with pytest.raises(ValueError):
+            runner.validate_config()