diff --git a/ecs.php b/ecs.php index 3055939..89701b0 100644 --- a/ecs.php +++ b/ecs.php @@ -132,6 +132,7 @@ use PhpCsFixer\Fixer\PhpUnit\PhpUnitDedicateAssertFixer; use PhpCsFixer\Fixer\PhpUnit\PhpUnitDedicateAssertInternalTypeFixer; use PhpCsFixer\Fixer\PhpUnit\PhpUnitExpectationFixer; +use PhpCsFixer\Fixer\PhpUnit\PhpUnitFqcnAnnotationFixer; use PhpCsFixer\Fixer\PhpUnit\PhpUnitMockFixer; use PhpCsFixer\Fixer\PhpUnit\PhpUnitMockShortWillReturnFixer; use PhpCsFixer\Fixer\PhpUnit\PhpUnitNoExpectationAnnotationFixer; @@ -390,6 +391,8 @@ PhpUnitNoExpectationAnnotationFixer::class, // Usages of ->setExpectedException* methods MUST be replaced by ->expectException* methods PhpUnitExpectationFixer::class, + // PHPUnit annotations should be a FQCNs including a root namespace. + PhpUnitFqcnAnnotationFixer::class, // Visibility of setUp() and tearDown() method should be kept protected PhpUnitSetUpTearDownVisibilityFixer::class, // There should not be an empty `return` statement at the end of a function diff --git a/tests/Integration/Fixtures/PhpUnit.correct.php.inc b/tests/Integration/Fixtures/PhpUnit.correct.php.inc index bc52d30..46b3f92 100644 --- a/tests/Integration/Fixtures/PhpUnit.correct.php.inc +++ b/tests/Integration/Fixtures/PhpUnit.correct.php.inc @@ -4,6 +4,9 @@ namespace Lmc\CodingStandard\Integration\Fixtures; use PHPUnit\Framework\TestCase; +/** + * @covers \Lmc\CodingStandard\Integration\Fixtures\MyClass + */ class PhpUnit extends TestCase { // PhpUnitSetUpTearDownVisibilityFixer diff --git a/tests/Integration/Fixtures/PhpUnit.wrong.php.inc b/tests/Integration/Fixtures/PhpUnit.wrong.php.inc index 510e4bb..a9aefd7 100644 --- a/tests/Integration/Fixtures/PhpUnit.wrong.php.inc +++ b/tests/Integration/Fixtures/PhpUnit.wrong.php.inc @@ -4,6 +4,9 @@ namespace Lmc\CodingStandard\Integration\Fixtures; use PHPUnit\Framework\TestCase; +/** + * @covers Lmc\CodingStandard\Integration\Fixtures\MyClass + */ class PhpUnit extends TestCase { // PhpUnitSetUpTearDownVisibilityFixer