- Issue created by @BramDriesen
- First commit to issue fork.
- Merge request !8Add unit tests for ReminderService covering all scenarios → (Merged) created by Unnamed author
- 🇸🇮Slovenia miroslavstankov
This test includes all main scenarios for ReminderService:
Untranslatable entity → no warning
No translations → no warning
Disabled bundle → no warning
Enabled bundle with custom message → warning shown
Enabled bundle with default message → warning shown - 🇮🇳India ankitv18
Test look fine but there should be a proper documentation for this test class and methods which is currently missing.
Pending: docBlock for class and methods.
- 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Thank you so much @miroslavstankov
Happy to merge this once the PHPCS and PHPSTAN errors are fixed :-). Assigning credits.
In case you don't know, you can see the reports in the GitLab runner results, and some are visible on the merge request diff.
-
bramdriesen →
committed 168ecaea on 1.0.x authored by
miroslavstankov →
[#3514986] feat: Add PHPUnit tests By: bramdriesen By: miroslavstankov...
-
bramdriesen →
committed 168ecaea on 1.0.x authored by
miroslavstankov →