Problem/Motivation

Trying to generate a coverage report for Drupal 8, I encountered some error results.

Turns out they're because of incorrect @covers/@coversDefaultClass annotation.

Proposed resolution

Fix them. :-)

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#1 2219913.patch2.56 KBMile23
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 commented
FileSize
2219913.patch2.56 KB

The fix. Super-simple.

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 commented
Status: Active » Needs review
tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Reviewed & tested by the community

Yup, looks good.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Nice catch!

Committed and pushed to 8.x. Thanks!

  • Commit 72341d4 on 8.x by webchick: 
    Issue #2219913 by Mile23: Incorrect @covers/@coversDefaultClass breaks...

Status: Fixed » Closed (fixed)

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