Skip to content

Event system #67

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
barbosa89 merged 45 commits into from
Sep 25, 2025
Merged

Event system #67

barbosa89 merged 45 commits into from
Sep 25, 2025

Conversation

@barbosa89
Copy link
Contributor

@barbosa89 barbosa89 commented Sep 11, 2025

This PR adds the event system for the framework.

@barbosa89
feat: event system
243dce3
@barbosa89
tests: make event command
12c8b6d
@barbosa89
tests: make listener command
d741479
@barbosa89
refactor: update EventListener and AbstractListener for consistency i…
08a9b11 
…n interface implementation
@barbosa89
refactor: add setPriority method to EventListener interface and Abstr…
2543e7a 
…actListener class
@barbosa89
refactor: streamline handle method in EventListener for improved read…
49682fc 
…ability
@barbosa89
refactor: normalize priority assignment in AbstractListener and Event…
f711cee 
…Listener constructors
@barbosa89
refactor: remove unnecessary docblock from shouldHandle method in lis…
edc1ec6 
…tener stub
@barbosa89
refactor: remove timestamp handling from AbstractEvent and move it to…
657214a 
… Event class
@barbosa89
refactor: clean up whitespace in emitAsync method for improved readab…
819cd0f 
…ility
@barbosa89
refactor: add has method to App class and improve event listener hand…
d5c64a0 
…ling logic
@barbosa89
refactor: remove event constructor
7f57ab5
@barbosa89
refactor: simplify event listener handling and remove unused exceptio…
48590a7 
…n class
@barbosa89
test: add test for registering and emitting events with facade syntax
40005d2
@barbosa89
test: add test for removing non-registered event listeners
eef9ad8
@barbosa89
refactor: remove unused shouldHandle method from listener stub
548ba7e
@barbosa89
test: add test to skip listener when shouldHandle returns false
9237629
@barbosa89
feat: add fake method to extend container with mock implementations
2dfeb97
@barbosa89
feat: add shouldReceive method to Log facade for mocking expectations
89c019c
@barbosa89
test: add test to handle listener error gracefully and log error
872cc28
@barbosa89
feat: add async event registration and emission test
730fbbf
@barbosa89
test: add test for async event propagation handling
1503189
@barbosa89
refactor: improve test description for event handling
10d8d21
@barbosa89
test: add async event handling for listeners that should not be trigg…
eadfd87 
…ered
@barbosa89
test: add async handling for listener errors and ensure proper logging
a31d046
@barbosa89
test: enable warning emission for listener errors in async events
c9cc22d
@barbosa89
test: add async handling for one-time listeners and ensure proper rem…
c3e8cf7 
…oval
@barbosa89
test: add handling for listener errors in async events without warnings
65aefbf
@barbosa89
refactor: remove unnecessary check for event listeners in sortListene…
3e55637 
…rsByPriority method
@barbosa89
test: add warning for exceeding maximum number of listeners for an event
bbd318e
@barbosa89
test: enable warning emission when exceeding maximum number of listeners
4d63e53
@barbosa89
test: add check for warning suppression when exceeding maximum listeners
cfe099d
@barbosa89
refactor: simplify removeListener method by removing unnecessary chec…
7d070ba 
…k for event listeners
@barbosa89
refactor: simplify resolveListener method and improve conditional che…
00f0fb9 
…ck in handle method
@barbosa89
test: add test for registering and emitting events with string-class …
ca5f2d1 
…listeners
@barbosa89
refactor: remove unused has method from App class
cc041ed
@barbosa89
style: php cs
eb814b0
@barbosa89
refactor: remove debug dump from string-class listeners test
0e1444b
@barbosa89
test: add test for handling invalid listeners and create InvalidListe…
3280683 
…ner class
@barbosa89
test: add test for registering and emitting string-class events
887a60a
@barbosa89
refactor: remove __toString method from AbstractEvent class
a4926d2
@barbosa89
test: add test for registering and emitting events with custom listen…
57e974f 
…er priority
@barbosa89
test: add assertion for event timestamp in Event handling test
c0da37a
@barbosa89
style: php cs
3e1a87c
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 25, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
99.1% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@barbosa89 barbosa89 merged commit 3b1d258 into develop Sep 25, 2025
@barbosa89 barbosa89 deleted the feature/event-system branch September 25, 2025 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@barbosa89