You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Following on from #10759, this is an issue to add assertion messages to PHPUnit tests and remove the disabling of the PHPCS rule introduced in the previous issue.
Due to the scope of the changes, we may need to split this into multiple issues.
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
All existing PHPUnit test assertions are updated to include informative failure messages.
Thanks @zutigrm this AC looks good, however, noting that there are >3k assertions in over >200 files, I've added the same time-boxing criteria to the AC as seen in #5795.
Uh oh!
There was an error while loading. Please reload this page.
Feature Description
Following on from #10759, this is an issue to add assertion messages to PHPUnit tests and remove the disabling of the PHPCS rule introduced in the previous issue.
Due to the scope of the changes, we may need to split this into multiple issues.
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation Brief
Test Coverage
QA Brief
Changelog entry
The text was updated successfully, but these errors were encountered: