Skip to content

Add a TODO for the intermittent functional test failures in CI - #42

Closed
bonelifer wants to merge 1 commit into
masterfrom
todo-flaky-functional-tests
Closed

bonelifer wants to merge 1 commit into
masterfrom
todo-flaky-functional-tests

Conversation

@bonelifer

Copy link
Copy Markdown
Contributor

Summary

  • Adds docs/TODO.md with an entry for the intermittent functional test failures in CI ("Output found before DOCTYPE"):
    • where they actually happen: phpBB's own test login and logout steps, on MSSQL 2019, MSSQL 2022 and SQLite in different runs
    • why they look environmental: identical code both passed and failed
    • the next step: collect the PHP and nginx error logs from a failing run to find the cause
  • Adds the standard README TODO section linking to it.

This corrects the earlier explanation in #37, which blamed the cancelled request in confirm_bypass_test; #37's description has been updated to match.

Test plan

  • Documentation only.

Investigated and written by Claude on behalf of William Jacoby (bonelifer).

Record where the empty-page failures actually happen (phpBB's own test
login/logout steps, on different databases, with identical code both
passing and failing), and the next step: collect the PHP error log from
a failing run. Link docs/TODO.md from the README.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bonelifer

Copy link
Copy Markdown
Contributor Author

Closing: the cause turned out to be a real bug (an unnamed cron task), fixed in #43, so this TODO no longer applies.


Investigated and written by Claude on behalf of William Jacoby (bonelifer).

@bonelifer bonelifer closed this Sep 28, 2026
@bonelifer
bonelifer deleted the todo-flaky-functional-tests branch September 28, 2026 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant