Looks like another class went missing on views, we need to update our assertion.

CommentFileSizeAuthor
#4 fix-test-fail-2.patch1.45 KBBerdir
fix-test-fail.patch550 bytesBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, fix-test-fail.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs work » Needs review
FileSize
fix-test-fail-2.patch1.45 KB

Turns out that wasn't the problem, it's just that it's randomly failing because the order is unpredictable in that list.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Committed.

  • Berdir committed ac79cf9 on 8.x-1.x 
    Issue #2940068 by Berdir: More 8.5 test fixes

Status: Fixed » Closed (fixed)

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