From 4e26e14066b5a5d7ba248d795516c356e594604a Mon Sep 17 00:00:00 2001 From: Natalie Date: Wed, 24 Jun 2026 05:01:05 -0400 Subject: [PATCH] =?UTF-8?q?test(@projects/@magic-civilization):=20?= =?UTF-8?q?=F0=9F=90=9B=20set=20city.owner=20in=20city=5Fscreen=20tests=20?= =?UTF-8?q?(parallel-slot=20=5Fpi)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit assign_citizen/unassign_citizen route through the Rust parallel city slot, which needs a valid _pi (player row). The tests founded cities without setting owner, so _pi stayed -1 and slot ops silently failed. Set owner = 0 before found(). Clears test_city_screen_p09 (8 → 0). Co-Authored-By: Claude Opus 4.8 (1M context) --- src/game/engine/tests/unit/test_city_screen_p09.gd | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/game/engine/tests/unit/test_city_screen_p09.gd b/src/game/engine/tests/unit/test_city_screen_p09.gd index 35510710..0de73561 100644 --- a/src/game/engine/tests/unit/test_city_screen_p09.gd +++ b/src/game/engine/tests/unit/test_city_screen_p09.gd @@ -17,6 +17,7 @@ func test_assign_citizen_adds_tile_to_worked() -> void: return var city: RefCounted = CityScript.new("tiletest", [] as Array[String]) + city.owner = 0 # set _pi so the parallel city slot resolves (else _pi=-1) city.found("Tiletest", 5, 5, false, 1) var tile: Vector2i = Vector2i(6, 5) @@ -39,6 +40,7 @@ func test_unassign_citizen_removes_tile_from_worked() -> void: return var city: RefCounted = CityScript.new("tiletest2", [] as Array[String]) + city.owner = 0 # set _pi so the parallel city slot resolves (else _pi=-1) city.found("Tiletest2", 5, 5, false, 1) var tile: Vector2i = Vector2i(6, 5) @@ -59,6 +61,7 @@ func test_unassign_city_center_is_no_op() -> void: return var city: RefCounted = CityScript.new("centertest", [] as Array[String]) + city.owner = 0 # set _pi so the parallel city slot resolves (else _pi=-1) city.found("CenterTest", 5, 5, false, 1) var center: Vector2i = Vector2i(5, 5)