End-to-end headless smoke test: register -> login -> fetch worlds #39

Merged
reviewer-bot merged 2 commits from agent/775379a9bedc into main 2026-08-20 00:26:59 +00:00
Collaborator

End-to-end smoke test (register -> login -> fetch worlds) written, renamed to fit the checkpoint runner's naming convention, and verified: 4/4 steps pass against a real backend.

The tester had already written a complete, correct 148-line test before stalling -- it wasn't broken, its own verification attempt silently exited early because the task spec's suggested run command included --quit, which kills a --script SceneTree test before its awaited HTTP calls finish. Investigating that dead end burned all its retries.

Also fixed: scripts/smoke_test.gd -> scripts/smoke_test_e2e.gd. The task spec named it without the checkpoint runner's expected smoke_test_.gd suffix (run_checkpoint_tests.py globs scripts/smoke_test_.gd), so it would have silently never run as part of the checkpoint suite. Also corrected the --quit flag in the file's own usage comments so a human running it manually doesn't hit the same dead end.

End-to-end smoke test (register -> login -> fetch worlds) written, renamed to fit the checkpoint runner's naming convention, and verified: 4/4 steps pass against a real backend. The tester had already written a complete, correct 148-line test before stalling -- it wasn't broken, its own verification attempt silently exited early because the task spec's suggested run command included --quit, which kills a --script SceneTree test before its awaited HTTP calls finish. Investigating that dead end burned all its retries. Also fixed: scripts/smoke_test.gd -> scripts/smoke_test_e2e.gd. The task spec named it without the checkpoint runner's expected smoke_test_*.gd suffix (run_checkpoint_tests.py globs scripts/smoke_test_*.gd), so it would have silently never run as part of the checkpoint suite. Also corrected the --quit flag in the file's own usage comments so a human running it manually doesn't hit the same dead end.
tester-bot added 2 commits 2026-08-19 23:31:13 +00:00
Tester had already written a complete, correct test (4/4 steps pass
against a real backend) but stalled investigating DB/migration setup
after its own verification run silently exited with no result -- the
task spec's suggested invocation included --quit, which kills a --script
SceneTree test before its awaited HTTP calls finish (same Godot
--script-mode gotcha documented elsewhere this session).

Renamed smoke_test.gd -> smoke_test_e2e.gd: the task spec named it
without the checkpoint runner's expected smoke_test_*.gd suffix, so
scripts/run_checkpoint_tests.py's glob would have silently never picked
it up. Also fixed the --quit flag in the file's own usage comments.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
reviewer-bot merged commit 927f9bd95e into main 2026-08-20 00:26:59 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: daddy/modbison#39