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

CommentFileSizeAuthor
#2 3001644-2.patch3.86 KBLendude
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Lendude created an issue. See original summary.

Lendude’s picture

Issue summary: View changes
Status: Active » Needs review
FileSize
3.86 KB

We might as well clean it up since it is only one short test. Took care of all the deprecated methods and uses of t().

longwave’s picture

Status: Needs review » Reviewed & tested by the community

Checked the conversion, everything seems fine.

AdamPS’s picture

Sorry 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.

Lendude’s picture

@AdamPS yeah that's what I figured too, no worries, easily fixed!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed dc23efb and pushed to 8.7.x. Thanks!

  • alexpott committed dc23efb on 8.7.x
    Issue #3001644 by Lendude: Convert \Drupal\user\Tests\UserSubAdminTest...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.