Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It appears some update tests were added after #2870009: Update: Convert system functional tests to phpunit landed or we just plain missed them for some reason.
todo:
Covert remaining update tests extending \Drupal\system\Tests\Update\UpdatePathTestBase
Comment | File | Size | Author |
---|---|---|---|
#2 | 2985734-2.patch | 5.52 KB | Lendude |
Comments
Comment #2
LendudeHere we go. Lets see what this does.
Comment #3
borisson_I think this is a good improvement, I had one remark on this patch, but it's not worth setting this back to NW over it.
Do we need the class name in here? Because the class name here doesn't make it reusable and prone to get outdated.
It is just a test conversion, so we probably shouldn't do that in this issue. We could open a followup to do that?
Comment #4
alexpottCommitted and pushed eb5d499d31 to 8.7.x and 9a2592f7ed to 8.6.x. Thanks!
re #3 - I'm not sure it matters. Tests don't move often.