This is the twin sibling of #2426729: NodeSaveTest hardwires SQL, exact same problem, exact same solution. For that reason I am skipping protocol and file it as RTBC.

On the other hand, #2079647: Convert UserSaveTest to a kernel test. Remove useless test is related, the test has not changed since that issue was filed (in fact, git log -p --follow **/UserSaveTest.php shows it hasn't been changed since 2012).

CommentFileSizeAuthor
ust.patch741 byteschx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Needs review

Marking your own patches RTBC is against the community process.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Reviewed & tested by the community

Looks good, thx!

  • alexpott committed 8d4e412 on 8.0.x 
    Issue #2431525 by chx: UserSaveTest hardwires SQL
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Test code is not frozen in beta. Committed 8d4e412 and pushed to 8.0.x. Thanks!

Status: Fixed » Closed (fixed)

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