Skip to content

✅ Unit-Tests (Job, JobMessage) + CI + symfony/uid-Fix#2

Merged
schmunk42 merged 2 commits into
masterfrom
feature/unit-tests
Jun 14, 2026
Merged

✅ Unit-Tests (Job, JobMessage) + CI + symfony/uid-Fix#2
schmunk42 merged 2 commits into
masterfrom
feature/unit-tests

Conversation

@schmunk42

Copy link
Copy Markdown
Member

Was

Erste Unit-Test-Suite für dieses Paket (es gab bisher keine Tests).

  • Unit-Tests (PHPUnit 12) für die öffentlichen Kern-Klassen
  • CLI-only docker-compose.yml als Test-Runner (docker compose run --rm phpcomposer install && vendor/bin/phpunit), kein MySQL/FPM nötig
  • GitHub-Actions-Workflow (.github/workflows/tests.yml, Matrix PHP 8.2/8.4)
  • phpunit ^12 in require-dev, .gitignore (vendor/lock/cache)

Lokal grün verifiziert.

Außerdem (Bugfix): fehlende Laufzeit-Abhängigkeit symfony/uid zu require ergänzt — die Job-Entity nutzt Symfony\Component\Uid\Uuid im Konstruktor, war aber nicht deklariert. Vom Test aufgedeckt.

🤖 Generated with Claude Code

schmunk42 and others added 2 commits June 14, 2026 02:44
…n [*]

Unit-Tests (PHPUnit 12) für die öffentlichen Kern-Klassen, plus je Paket ein CLI-only docker-compose-Runner (composer install + vendor/bin/phpunit), ein GitHub-Actions-Workflow und .gitignore. phpunit ^12 in require-dev.

Außerdem: fehlende Laufzeit-Abhängigkeit symfony/uid zu require ergänzt (Job-Entity nutzt Symfony\Component\Uid\Uuid; vom Test aufgedeckt).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
8.2 fiel raus: PHPUnit 12 verlangt PHP >=8.3, daher schlug composer install im 8.2-Job fehl. Runtime-Constraint des Pakets (>=8.2) bleibt unverändert.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@schmunk42 schmunk42 merged commit 82d33d7 into master Jun 14, 2026
4 checks passed
@schmunk42 schmunk42 deleted the feature/unit-tests branch June 14, 2026 01:32
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