The test block appears in the custom category.
Other
DisplayBlockTest.php
102
Drupal\block\Tests\Views\DisplayBlockTest->testBlockCategory()	


The second cloned test block appears in the custom category.
Other
DisplayBlockTest.php
120
Drupal\block\Tests\Views\DisplayBlockTest->testBlockCategory()
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Issue tags: +Random test failure

Tagging.

tim.plunkett’s picture

FileSize
1.03 KB

From form_builder():

  if (!isset($element['#id'])) {
    $element['#id'] = drupal_html_id('edit-' . implode('-', $element['#parents']));
  }

The implode is causes any leading or trailing hyphens to be stripped off before making the ID.

tim.plunkett’s picture

Priority: Normal » Critical
Status: Active » Needs review

Marking this critical because it just "broke" HEAD.

tim.plunkett’s picture

FileSize
897 bytes

Nevermind, that last assumption was completely wrong, and idk why I thought that. It's just that drupal_html_id() removes consecutive hyphens.

pwolanin’s picture

per IRC discussion - that's the wrong fix, instead move 'edit-' . inside the call to drupal_html_id($id)

pwolanin’s picture

cross-post - yes, #4 looks like the right fix.

pwolanin’s picture

I'm not sure that the String::checkPlain() is actually needed, but clearly it's already in the test.

pwolanin’s picture

Status: Needs review » Reviewed & tested by the community

RTBC (assuming the tests pass with the corrected logic).

jibran’s picture

Issue tags: +VDC

Finally @tim.plunkett created the issue :P. RTBC +1

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 34f638b and pushed to 8.x. Thanks!

tim.plunkett’s picture

Priority: Critical » Major
Status: Fixed » Active

Berdir found yet another random fail in the same place, but much less common...

olli’s picture

Status: Active » Needs review
FileSize
3.13 KB
3.6 KB
816 bytes

Categories like "#random" are not visible on the block placement page.

Issue tags: +VDC, +Random test failure

The last submitted patch, drupal-2097537-12.patch, failed testing.

Status: Needs review » Needs work
Issue tags: -VDC, -Random test failure

The last submitted patch, drupal-2097537-12.patch, failed testing.

olli’s picture

Status: Needs work » Needs review
FileSize
1.65 KB
4.78 KB
tim.plunkett’s picture

Status: Needs review » Reviewed & tested by the community

Thanks... This has been awful :)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Wow, great find!

Committed and pushed to 8.x. Thanks!

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