Skip to content

refactor: pass --do-not-cache-result to prevent shared cache corruption#10098

Merged
michalsn merged 2 commits intocodeigniter4:developfrom
michalsn:refactor/run-random-tests
Apr 9, 2026
Merged

refactor: pass --do-not-cache-result to prevent shared cache corruption#10098
michalsn merged 2 commits intocodeigniter4:developfrom
michalsn:refactor/run-random-tests

Conversation

@michalsn
Copy link
Copy Markdown
Member

@michalsn michalsn commented Apr 9, 2026

Description
The random test script runs test jobs in parallel. Disabling PHPUnit result caching avoids cache reuse or cache writes interfering across concurrent runs, which can lead to shared cache corruption and make failures harder to trust.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@michalsn michalsn added the refactor Pull requests that refactor code label Apr 9, 2026
@paulbalandan
Copy link
Copy Markdown
Member

In line 615 (originally 614), you need to update the 0:6 part so that the seed is still included in the logs when the test fails.

Co-authored-by: John Paul E Balandan <paulbalandan@gmail.com>
@michalsn
Copy link
Copy Markdown
Member Author

michalsn commented Apr 9, 2026

Thanks!

@michalsn michalsn added the github_actions Pull requests that update Github_actions code label Apr 9, 2026
@michalsn michalsn merged commit 0b33010 into codeigniter4:develop Apr 9, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update Github_actions code refactor Pull requests that refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants