Currently the plugins use the static \Drupal class, and should instead use dependency injection, which makes testing easier.

CommentFileSizeAuthor
#2 use_dependency-2846495-2.patch25.43 KBmglaman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mglaman created an issue. See original summary.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman as a volunteer commented
Status: Active » Needs review
FileSize
use_dependency-2846495-2.patch25.43 KB

This fixes the tests

  • mglaman committed 03f5f26 on 8.x-2.x 
    Issue #2846495 by mglaman: Use dependency injecting within plugins
mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman as a volunteer commented
Status: Needs review » Fixed

Fixed, and now we have passing tests.

Status: Fixed » Closed (fixed)

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