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.
follow up to #2854252: User forms broken for admin without 'administer users'.
A new WebTestBase got added, we need to convert this to BrowserTestBase.
Scope:
- \Drupal\user\Tests\UserSubAdminTest
Comment | File | Size | Author |
---|---|---|---|
#2 | 3001644-2.patch | 3.86 KB | Lendude |
Comments
Comment #2
LendudeWe might as well clean it up since it is only one short test. Took care of all the deprecated methods and uses of t().
Comment #3
longwaveChecked the conversion, everything seems fine.
Comment #4
AdamPS CreditAttribution: AdamPS at AlbanyWeb commentedSorry about that! I guess it probably happened because the patch that was committed in the original issue had been active for a long time so it started life as WebTestBase and no one noticed it needed to be converted.
Comment #5
Lendude@AdamPS yeah that's what I figured too, no worries, easily fixed!
Comment #6
alexpottCommitted dc23efb and pushed to 8.7.x. Thanks!