This code needs some tests.

CommentFileSizeAuthor
#2 2788165-02.patch3.5 KBjhedstrom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhedstrom created an issue. See original summary.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom at Phase2 for Workday, Inc. commented
Status: Active » Needs review
FileSize
2788165-02.patch3.5 KB

Status: Needs review » Needs work

The last submitted patch, 2: 2788165-02.patch, failed testing.

  • jhedstrom committed b57a667 on 8.x-1.x 
    Issue #2788165 by jhedstrom: Add some tests
jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom at Phase2 for Workday, Inc. commented
Status: Needs work » Fixed

Apparently the test runners to not run composer install. These are passing locally.

Status: Fixed » Closed (fixed)

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