Skip to content

ci: run PHPUnit Unit suite on push/PR (PHP 8.1–8.4) - #59

Merged
PrestaEdit merged 2 commits into
mainfrom
ci/unit-tests
Aug 17, 2026
Merged

ci: run PHPUnit Unit suite on push/PR (PHP 8.1–8.4)#59
PrestaEdit merged 2 commits into
mainfrom
ci/unit-tests

Conversation

@PrestaEdit

Copy link
Copy Markdown
Contributor

Contexte

Le repo n'a actuellement aucun workflow GitHub Actions — les tests unitaires (117 tests, 442 assertions) ne sont donc jamais joués sur les push ni les PR. Seul le Copilot review tourne.

Changement

Ajoute .github/workflows/tests.yml :

  • Se déclenche sur push main et toute pull request.
  • Matrice PHP 8.1 → 8.4, fail-fast: false pour voir toutes les versions en un run.
  • composer install avec cache par version PHP.
  • Lance uniquement vendor/bin/phpunit --testsuite Unit.

Pourquoi seulement le suite Unit ?

Le phpunit.xml.dist déclare aussi une suite Feature, qui exercice probablement Chromium via chrome-php/chrome. Ça mérite son propre workflow (installation du navigateur, gestion des artefacts d'échec, timeout plus large) et je préfère l'ajouter séparément une fois qu'on aura statué sur la stratégie. La suite Unit couvre déjà les 117 tests actuels et n'a aucune dépendance système.

🤖 Generated with Claude Code

PrestaEdit and others added 2 commits August 17, 2026 10:14
Adds a GitHub Actions workflow that installs Composer deps and runs
the Unit testsuite across a PHP 8.1 → 8.4 matrix on every push to main
and every pull request. Only the Unit suite runs here — the Feature
suite needs a Chromium instance and belongs in a separate workflow.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Library requires PHP 8.2+ (per docs and the resolved symfony/* 7.x
lock), so 8.1 fails at composer install. Keep 8.2 → 8.4.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@PrestaEdit
PrestaEdit merged commit 038bf7a into main Aug 17, 2026
3 checks passed
@PrestaEdit
PrestaEdit deleted the ci/unit-tests branch August 17, 2026 08:36
@PrestaEdit PrestaEdit mentioned this pull request Aug 17, 2026
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