Line 330 of the test, the hash is the same when expected to be different

Pass      Other      ToolbarAdminMenuT  330 Drupal\toolbar\Tests\ToolbarAdminMe
    A valid hash value for the admin menu subtrees was created.
Fail      Other      ToolbarAdminMenuT  331 Drupal\toolbar\Tests\ToolbarAdminMe
    The user-specific subtree menu hash has been updated.
    Value 'ADugRbLRN_vkrN7NMUt2omJIRRm_d0XK9R1cDMtBrrI' is not equal to value
    'ADugRbLRN_vkrN7NMUt2omJIRRm_d0XK9R1cDMtBrrI'.

Comments

pwolanin created an issue. See original summary.

neclimdul’s picture

StatusFileSize
new1 byte

Just to confirm.

neclimdul’s picture

Status: Active » Needs review

bah

kgoel’s picture

pwolanin’s picture

looks like the translation may not be getting saved based on the verbose test output

kgoel’s picture

Status: Needs review » Needs work

The last submitted patch, 2: test_fails_on_sqlite-2580485-2.patch, failed testing.

pwolanin’s picture

running a git bisect assuming 7ed830c is good

pwolanin’s picture

1cdbe865bc0b9ef0f26d6db302a252c63062b1bf is the first bad commit

commit 1cdbe865bc0b9ef0f26d6db302a252c63062b1bf
Author: webchick <drupal@webchick.net>
Date:   Sun Oct 4 23:48:45 2015 -0700

    Issue #2503755 by Wim Leers, bobrov1989, webchick, dcrocks, andypost, emma.maria, catch, dawehner, Bojhan, tstoeckler, alexpott, yoroy: Switch from user login block to login menu link and search block in standard profile

pwolanin’s picture

Status: Needs work » Needs review
StatusFileSize
new392 bytes

dummy patch to show HEAD fail

wim leers’s picture

Progress report.

pwolanin: WimLeers:  I'm testing a revert of just the changes to DefaultMenuLinkTreeManipulators and MenuLinkBase
WimLeers: pwolanin++
WimLeers: pwolanin++
pwolanin: WimLeers:  and that passes
WimLeers: pwolanin: thanks!
WimLeers: pwolanin: but still WTFFFFF
pwolanin’s picture

Assigned: Unassigned » wim leers

Wim is tracking this down

martin107’s picture

Just providing information.....

Another issue has been having similar problems today

a document only patch ... with unexpected failures that look similar...

a simplytest.me sandbox was used to show test passing but failing on our test infrastructure.

martin107’s picture

double post.

pwolanin’s picture

It's something in this commit: #2503755: Switch from user login block to login menu link and search block in standard profile

involving changes to these 2 files, since reverting just that part of the commit makes the test pass

core/lib/Drupal/Core/Menu/DefaultMenuLinkTreeManipulators.php
core/lib/Drupal/Core/Menu/MenuLinkBase.php

pwolanin’s picture

wim leers’s picture

Status: Fixed » Closed (fixed)

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