Skip to content

test: optimize AutoReview tests#10127

Merged
paulbalandan merged 1 commit intocodeigniter4:developfrom
paulbalandan:optimize-autoreview
Apr 21, 2026
Merged

test: optimize AutoReview tests#10127
paulbalandan merged 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
@paulbalandan paulbalandan merged commit 74982b1 into codeigniter4:develop Apr 21, 2026
53 checks passed
@paulbalandan paulbalandan deleted the optimize-autoreview branch April 21, 2026 19:29
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.

2 participants