@covers is out of date in the path module [#2275145]

Problem/Motivation

type the following from the root directory

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

Please note the error only shows with the --coverage-html option enabled.
As of today there are two error relating to outdated @cover statements. One is in the path 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 @covers statement to relate to the correct method under test.

Remaining tasks

none

User interface changes

none

API changes

none

Comment	File	Size	Author
#3	interdiff-0-3.txt	718 bytes	martin107
#3	pathCovers-2275145-3.patch	721 bytes	martin107
#3
	pathCovers.patch	721 bytes	martin107

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

martin107 CreditAttribution: martin107 commented 31 May 2014 at 12:42

Issue summary:

View changes

Comment #2

dawehner

German

CreditAttribution: dawehner commented 31 May 2014 at 20:59

Status:

Needs review

» Needs work

+++ b/core/modules/path/tests/src/Field/PathFieldDefinitionTest.php
@@ -49,7 +49,7 @@ protected function getModuleAndPath() {
-   * @covers \Drupal\path\Plugin\Field\FieldType\PathItem::getSchema
+   * @covers \Drupal\path\Plugin\Field\FieldType\PathItem::Schema

this should be lowercase