fix(map): require single-character legend keys
This commit is contained in:
@@ -183,6 +183,13 @@ def load_map(path: Path) -> MapSpec:
|
||||
|
||||
terrain = _split_rows(raw.terrain, "terrain", path)
|
||||
height, width = len(terrain), len(terrain[0])
|
||||
multi_char = sorted(key for key in raw.legend if len(key) != 1)
|
||||
if multi_char:
|
||||
msg = (
|
||||
"legend keys must be exactly one character (one char = one "
|
||||
f"square), got: {multi_char}"
|
||||
)
|
||||
raise MapValidationError(path, msg)
|
||||
unknown = sorted({ch for row in terrain for ch in row if ch not in raw.legend})
|
||||
if unknown:
|
||||
msg = f"terrain chars missing from legend: {unknown}"
|
||||
|
||||
@@ -93,6 +93,26 @@ def test_rejects_char_missing_from_legend(tmp_path: Path) -> None:
|
||||
load_map(_write(tmp_path, body))
|
||||
|
||||
|
||||
def test_rejects_multi_character_legend_key(tmp_path: Path) -> None:
|
||||
"""One char = one square: a legend key like "ZZ" can never match a cell
|
||||
and is dead data -- reject it at load."""
|
||||
body = _minimal_map().replace(
|
||||
' ".": { type: floor, move_cost: 1 }\n',
|
||||
' ".": { type: floor, move_cost: 1 }\n "ZZ": { type: floor, move_cost: 1 }\n',
|
||||
)
|
||||
with pytest.raises(MapValidationError, match="legend keys must be exactly one character"):
|
||||
load_map(_write(tmp_path, body))
|
||||
|
||||
|
||||
def test_rejects_empty_legend_key(tmp_path: Path) -> None:
|
||||
body = _minimal_map().replace(
|
||||
' ".": { type: floor, move_cost: 1 }\n',
|
||||
' "": { type: floor, move_cost: 1 }\n',
|
||||
)
|
||||
with pytest.raises(MapValidationError, match="legend keys must be exactly one character"):
|
||||
load_map(_write(tmp_path, body))
|
||||
|
||||
|
||||
def test_rejects_elevation_dimension_mismatch(tmp_path: Path) -> None:
|
||||
body = _minimal_map() + "elevation: |\n 000\n 00\n"
|
||||
with pytest.raises(MapValidationError):
|
||||
|
||||
Reference in New Issue
Block a user