I've seen this a few times now, about time to open an issue about it. Seen it myself twice (last in #1804380: convert color variables to config/state) and heard of a third case, so I'm quite sure that this happened even more.

Fallback page title for user pages is 'User account' for anonymous users.	Other	UserAccountLinksTests.php	118	Drupal\user\Tests\UserAccountLinksTests->testAccountPageTitles()

Haven't looked into it yet, I've no idea how an assertion like this could randomly fail.

CommentFileSizeAuthor
#2 account-links-random-test-failure-1833928.patch904 bytesBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

Got it. As I assumed, the problem is that we use randomString() for an URL but I wasn't sure what exactly of that is the problem.

This fails:

http://d8/user/password/.I_%7B%40%60I5

Will upload a patch soon.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Active » Needs review
FileSize
account-links-random-test-failure-1833928.patch904 bytes

And here is the patch. Tests were introduced in #1251188: Set unique titles for user/register, user/password, and user/login menu items

boombatower’s picture

boombatower CreditAttribution: boombatower commented
Status: Needs review » Reviewed & tested by the community

Nice catch.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Should we just remove the randomString() function, I wonder? These things seem to bite us a lot. :\

Anyway, great catch. Committed and pushed to 8.x. Thanks!

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