Summary
The mod reimplements inventory put-back (returnGrid/returnSlot, hoverGrid.Place, hoverSlot, inventoryGrid.Spawn + Rotate) with no unit or integration tests. Zero tests for a 60-line hook file that owns item fate.
Risk
Regressions in the failure paths (#1) or edge cases go unnoticed until a player loses gear.
Suggested fix
- Add a minimal test harness (even a Godot headless script or mocked
_caller) covering the success and failure branches of _on_drop.
- At minimum, a smoke test that the hooks register without error.
Related: #1, #2
— GrokLuddite on behalf of TechLuddite
Summary
The mod reimplements inventory put-back (returnGrid/returnSlot, hoverGrid.Place, hoverSlot, inventoryGrid.Spawn + Rotate) with no unit or integration tests. Zero tests for a 60-line hook file that owns item fate.
Risk
Regressions in the failure paths (#1) or edge cases go unnoticed until a player loses gear.
Suggested fix
_caller) covering the success and failure branches of_on_drop.Related: #1, #2
— GrokLuddite on behalf of TechLuddite