Skip to content

Conversation

alexshamrai
Copy link
Owner

No description provided.

@alexshamrai alexshamrai force-pushed the feature/investigate-listener branch 18 times, most recently from bfb8d5e to 9dc92b3 Compare October 9, 2025 08:26
Introduce CtrfListener as an alternative to CtrfExtension for generating
CTRF reports, providing flexibility in how users integrate the library.
Both implementations now delegate to a shared CtrfReportManager singleton,
eliminating code duplication and ensuring consistent behavior.

BREAKING CHANGES:
- TestProcessor.createTest() now accepts displayName as a parameter instead
  of ExtensionContext, making it framework-agnostic
- TestDetails model enhanced with uniqueId and displayName fields

Changes:
- Add CtrfListener implementing JUnit Platform TestExecutionListener
- Extract shared logic into CtrfReportManager singleton pattern
- Refactor CtrfExtension to delegate to CtrfReportManager
- Make TestProcessor framework-agnostic for reusability
- Reorganize integration tests into three modules:
  * integration-ctrf-validator: validates generated CTRF reports
  * integration-tests-extension: tests Extension-based reporting
  * integration-tests-listener: tests Listener-based reporting
- Add separate CI workflows for Extension and Listener integration tests
- Update documentation to reflect new architecture

This change enables users to choose between @ExtendWith annotation
(Extension) or service provider configuration (Listener) based on their
project needs, while maintaining a single, well-tested core implementation.
@alexshamrai alexshamrai force-pushed the feature/investigate-listener branch from 5a8a4cb to 506587d Compare October 9, 2025 09:26
@alexshamrai alexshamrai marked this pull request as ready for review October 9, 2025 09:30
@alexshamrai alexshamrai changed the title check listener Introduce CtrfListener feature Oct 9, 2025
@alexshamrai alexshamrai merged commit 7abdbf1 into master Oct 9, 2025
3 checks passed
@alexshamrai alexshamrai deleted the feature/investigate-listener branch October 9, 2025 09:31
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