Problem/Motivation

type the following from the root directory

  cd core
  phpunit --coverage-html html --verbose --bootstrap tests/bootstrap.php 

Please not that the errors don't show unless the "--coverage-html" is added.
As of today there are two error relating to outdated $cover statements. One is in the entity module.

1) Drupal\Tests\Core\Entity\EntityUrlTest::testGetSystemPath
PHP_CodeCoverage_Exception: Trying to @cover not existing method "\Drupal\Core\Entity\Entity::getPathByAlias".

2) Drupal\path\Tests\Field\PathFieldDefinitionTest::testGetColumns
PHP_CodeCoverage_Exception: Trying to @cover not existing method "\Drupal\path\Plugin\Field\FieldType\PathItem::getSchema".

Proposed resolution

Update the @cover statement to relate to the correct method under test.

Remaining tasks

none

User interface changes

none

API changes

none

CommentFileSizeAuthor
badCovers.patch578 bytesmartin107

Comments

martin107’s picture

Title: @covers statement is out of date. » @cover statement is out of date.
Issue summary: View changes
dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Good catch!

Status: Reviewed & tested by the community » Needs work

The last submitted patch, badCovers.patch, failed testing.

ParisLiakos’s picture

Status: Needs work » Needs review

badCovers.patch queued for re-testing.

ParisLiakos’s picture

Status: Needs review » Reviewed & tested by the community
jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 8.x.

Status: Fixed » Closed (fixed)

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