Conversation
Merge pull request #5 from arkdevuk/dev
…ublished, closed, assigned)
arkdevuk
added a commit
that referenced
this pull request
Jul 22, 2026
Merge pull request #7 from arkdevuk/dev
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request implements email notifications for testers about test plan lifecycle events (published, closed, or new assignment) and refines the logic for dispatching these notifications. It also introduces dedicated email templates for each notification type, improves configuration for local development, and adds unit tests for the new listener.
Test Plan Notification System
TesterNotificationListenerthat listens for test plan lifecycle events and triggers the appropriate notification methods inTesterManager(src/EventListener/TesterNotificationListener.php).TesterManagerto email testers when a plan is published, closed, or when a tester is newly assigned to an already-published plan, using dedicated Twig templates (src/Services/Entities/TesterManager.php).Email Templates
templates/test-plan-published.email.twig,templates/test-plan-closed.email.twig,templates/test-plan-invitation.email.twig). [1] [2] [3]Test Plan State Transition Logic
TestPlanStateProcessorto ensure notifications are sent only for the correct state transitions (e.g., testers added during a publish transition only get the "plan published" email, not an additional invitation) (src/State/TestPlanStateProcessor.php).Testing
TesterNotificationListenerto verify correct notification dispatching for each event type (tests/Unit/EventListener/TesterNotificationListenerTest.php).Development Environment Configuration
SERVER_NAMEandTRUSTED_HOSTSdefaults incompose.ymlto make local development easier, especially when using tunnels or accessing from other devices (compose.yml). [1] [2]