applications/magicciv
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

test(@projects/@magic-civilization): 🐛 set city.owner in city_screen tests (parallel-slot _pi)

Browse source 
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) <noreply@anthropic.com>
This commit is contained in:
Natalie 2026-06-24 05:01:05 -04:00
parent b064f7cd87
commit 4e26e14066
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/game/engine/tests/unit/test_city_screen_p09.gd
View file

@ -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)