Skip to content

test: optimize AutoReview tests#10127

Open
paulbalandan wants to merge 1 commit intocodeigniter4:developfrom
paulbalandan:optimize-autoreview
Open

test: optimize AutoReview tests#10127
paulbalandan wants to merge 1 commit intocodeigniter4:developfrom
paulbalandan:optimize-autoreview

Conversation

@paulbalandan
Copy link
Copy Markdown
Member

@paulbalandan paulbalandan commented Apr 21, 2026

Description

  • In random tests, AutoReview tests currently take about 30s on its first run and normalizes to ~1s on subsequent runs. This optimizes the runs using a filename-based check instead of the expensive is_subclass_of() check.
  • In CreateNewChangelogTest, moves the fetching of the whole repo to the workflow level.

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

@github-actions github-actions bot added the testing Pull requests that changes tests only label Apr 21, 2026
@github-actions github-actions bot added github_actions Pull requests that update Github_actions code and removed testing Pull requests that changes tests only labels Apr 21, 2026
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant