Skip to content

Conversation

@rafaeltonholo
Copy link
Member

Part of #9312.

This PR introduces the :feature:notification:testing module, helping to share common fakes between the :feature:notification:api and :feature:notification:impl module.

Additionally, this PR also removes the sealed modifier from AppNotification, SystemNotification and InAppNotification, enabling them to be implemented outside the api module.

Copy link
Member

@wmontwe wmontwe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rafaeltonholo rafaeltonholo merged commit c14fb5d into thunderbird:main Jul 28, 2025
5 checks passed
@rafaeltonholo rafaeltonholo deleted the feat/9312/add-notification-testing-module branch July 28, 2025 13:48
@thunderbird-botmobile thunderbird-botmobile bot added this to the Thunderbird 13 milestone Jul 28, 2025
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.

2 participants