Skip to content

Enable parallel testing in CI and add per-worker test setup#1799

Merged
kristiankunc merged 4 commits intoVATSIM-UK:mainfrom
MrAdder:issue-479
Feb 15, 2026
Merged

Enable parallel testing in CI and add per-worker test setup#1799
kristiankunc merged 4 commits intoVATSIM-UK:mainfrom
MrAdder:issue-479

Conversation

@MrAdder
Copy link
Copy Markdown
Contributor

@MrAdder MrAdder commented Feb 2, 2026

fixes #479
also includes PR #1798
Summary:

  • Add CI step to generate Passport keys (php artisan passport:keys --force) so tests can run reliably in CI.
  • Switch CI test run to parallel: php artisan test --parallel --processes=4 --recreate-databases --coverage-clover=coverage.xml.
  • Add a ParallelTestingServiceProvider to set up per-worker databases and create a personal Passport client for each worker.
  • Add .nvmrc (Node 24) and add brianium/paratest + bump PHPUnit in composer.json.
  • Composer lock and JS lockfiles updated to reflect dependency upgrades.

Files touched (high level):

  • .github/workflows/test.yml
  • app/Providers/ParallelTestingServiceProvider.php (new)
  • .nvmrc (new)
  • composer.json
  • composer.lock
  • (lockfile updates for JS deps)

Testing / Notes:

  1. CI now generates Passport keys once (prevents file races across workers).
  2. Local run to try parallel tests:
    composer install
    php artisan passport:keys --force
    php artisan test --parallel --processes=4 --recreate-databases
    
    
    
    
    
    

New
image

Old

image

@kristiankunc
Copy link
Copy Markdown
Contributor

I've merged #1798 before I saw this - merge conflicts need fixing

@MrAdder
Copy link
Copy Markdown
Contributor Author

MrAdder commented Feb 9, 2026

Give me 20 mins and it will be fixed

@MrAdder
Copy link
Copy Markdown
Contributor Author

MrAdder commented Feb 9, 2026

@kristiankunc Fixed

kristiankunc
kristiankunc previously approved these changes Feb 9, 2026
@kristiankunc
Copy link
Copy Markdown
Contributor

@MrAdder sorry I've merged a copule of dependency updates and forgot this was still open - mind fixing the lockfile again please?

@sonarqubecloud
Copy link
Copy Markdown

@MrAdder
Copy link
Copy Markdown
Contributor Author

MrAdder commented Feb 15, 2026

@kristiankunc Fixed all 38 dependency updates in the lock file

@kristiankunc kristiankunc merged commit 6fee680 into VATSIM-UK:main Feb 15, 2026
6 checks passed
@VATSIMUK
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 6.62.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

Add parallel testing - Laravel 8.25

3 participants