Somehow an extra little period slipped by some patch review, and now we have two 'Routing' test groups, 'Routing' and 'Routing.' (with a period). Patch in comment below fixes the issue.

(as seen on the admin/config/development/testing page).

CommentFileSizeAuthor
#1 fix_errant_routing-2343097-1.patch521 bytesgeerlingguy
routing-madness.png8.47 KBgeerlingguy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy commented
Status: Active » Needs review
FileSize
fix_errant_routing-2343097-1.patch521 bytes

  • webchick committed 9b16aeb on 
    Issue #2343097 by geerlingguy: Fixed errant Routing. test group.
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Needs review » Fixed

Nice catch.

Committed and pushed to 8.x. Thanks!

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

It would be great it we could add one issue which fixes all the other ones.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Please!

geerlingguy’s picture

geerlingguy CreditAttribution: geerlingguy commented
Related issues: +#2343455: Clean up test @group names

Added #2343455: Clean up test @group names.

Status: Fixed » Closed (fixed)

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