- Issue created by @mondrake
- Assigned to mondrake
- Status changed to Needs work
9 months ago 9:40am 28 March 2024 - Issue was unassigned.
- 🇮🇹Italy mondrake 🇮🇹
It' s hard at this point to find a way to group the missing conversions in logical chunks of similar scope, but still can't go for a big change.
So I propose in this step to just limit to 6 conversions:
- core/modules/filter/tests/src/Unit/FilterUninstallValidatorTest.php
- core/modules/forum/tests/src/Unit/Breadcrumb/ForumBreadcrumbBuilderBaseTest.php
- core/tests/Drupal/Tests/Core/Config/Entity/ConfigEntityBaseUnitTest.php
- core/tests/Drupal/Tests/Core/Config/Entity/EntityDisplayBaseTest.php
- core/tests/Drupal/Tests/Core/Test/BrowserTestBaseTest.php
- core/tests/Drupal/Tests/Core/Test/WebDriverTestBaseTest.php
- Status changed to Needs review
9 months ago 1:24pm 4 April 2024 - Status changed to RTBC
8 months ago 4:35pm 28 April 2024 - 🇺🇸United States smustgrave
Refactor seems fine with test coverage not being lost. Leaning on the green tests for this one.
- Status changed to Fixed
8 months ago 9:50am 29 April 2024 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!