Merge pull request #249 from kubawerlos/add_php_unit_fqcn_annotation

PHPUnit annotations should be a FQCNs including a root namespace
cocur · Dec 2, 2019 · dc19dbb · dc19dbb
2 parents 6c05172 + c271a18
commit dc19dbb
Show file tree

Hide file tree

Showing 15 changed files with 42 additions and 41 deletions.
diff --git a/.php_cs b/.php_cs
@@ -14,6 +14,7 @@ return PhpCsFixer\Config::create()
     ->setRules([
         '@PSR2' => true,
         'no_empty_phpdoc' => true,
+        'php_unit_fqcn_annotation' => true,
         'php_unit_test_annotation' => true,
         'phpdoc_trim' => true,
     ]);
diff --git a/tests/Bridge/Laravel/SlugifyProviderTest.php b/tests/Bridge/Laravel/SlugifyProviderTest.php
@@ -42,7 +42,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Laravel\SlugifyServiceProvider::register()
+     * @covers \Cocur\Slugify\Bridge\Laravel\SlugifyServiceProvider::register()
      */
     public function testRegisterRegistersTheServiceProvider()
     {
@@ -56,7 +56,7 @@ public function testRegisterRegistersTheServiceProvider()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Laravel\SlugifyServiceProvider::provides()
+     * @covers \Cocur\Slugify\Bridge\Laravel\SlugifyServiceProvider::provides()
      */
     public function testContainsReturnsTheNameOfThProvider()
     {

diff --git a/tests/Bridge/Latte/SlugifyHelperTest.php b/tests/Bridge/Latte/SlugifyHelperTest.php
@@ -25,7 +25,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Latte\SlugifyHelper::slugify()
+     * @covers \Cocur\Slugify\Bridge\Latte\SlugifyHelper::slugify()
      */
     public function testSlugify()
     {

diff --git a/tests/Bridge/Nette/SlugifyExtensionTest.php b/tests/Bridge/Nette/SlugifyExtensionTest.php
@@ -24,7 +24,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Nette\SlugifyExtension::loadConfiguration()
+     * @covers \Cocur\Slugify\Bridge\Nette\SlugifyExtension::loadConfiguration()
      */
     public function testLoadConfiguration()
     {
@@ -75,7 +75,7 @@ public function testLoadConfiguration()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Nette\SlugifyExtension::beforeCompile()
+     * @covers \Cocur\Slugify\Bridge\Nette\SlugifyExtension::beforeCompile()
      */
     public function testBeforeCompile()
     {

diff --git a/tests/Bridge/Plum/SlugifyConverterTest.php b/tests/Bridge/Plum/SlugifyConverterTest.php
@@ -26,8 +26,8 @@
 class SlugifyConverterTest extends MockeryTestCase
 {
     /**
-     * @covers Cocur\Slugify\Bridge\Plum\SlugifyConverter::__construct()
-     * @covers Cocur\Slugify\Bridge\Plum\SlugifyConverter::convert()
+     * @covers \Cocur\Slugify\Bridge\Plum\SlugifyConverter::__construct()
+     * @covers \Cocur\Slugify\Bridge\Plum\SlugifyConverter::convert()
      */
     public function testConvertSlugifiesString()
     {
@@ -39,8 +39,8 @@ public function testConvertSlugifiesString()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Plum\SlugifyConverter::__construct()
-     * @covers Cocur\Slugify\Bridge\Plum\SlugifyConverter::convert()
+     * @covers \Cocur\Slugify\Bridge\Plum\SlugifyConverter::__construct()
+     * @covers \Cocur\Slugify\Bridge\Plum\SlugifyConverter::convert()
      */
     public function testConstructorCreatesSlugifyIfNoneIsProvided()
     {

diff --git a/tests/Bridge/Silex/SlugifySilexProviderTest.php b/tests/Bridge/Silex/SlugifySilexProviderTest.php
@@ -31,7 +31,7 @@
 class SlugifySilexProviderTest extends MockeryTestCase
 {
     /**
-     * @covers Cocur\Slugify\Bridge\Silex\SlugifyServiceProvider
+     * @covers \Cocur\Slugify\Bridge\Silex\SlugifyServiceProvider
      */
     public function testRegister()
     {

diff --git a/tests/Bridge/Symfony/CocurSlugifyBundleTest.php b/tests/Bridge/Symfony/CocurSlugifyBundleTest.php
@@ -29,7 +29,7 @@
 class CocurSlugifyBundleTest extends MockeryTestCase
 {
     /**
-     * @covers Cocur\Slugify\Bridge\Symfony\CocurSlugifyBundle::getContainerExtension()
+     * @covers \Cocur\Slugify\Bridge\Symfony\CocurSlugifyBundle::getContainerExtension()
      */
     public function testGetContainerExtension()
     {

diff --git a/tests/Bridge/Symfony/CocurSlugifyExtensionTest.php b/tests/Bridge/Symfony/CocurSlugifyExtensionTest.php
@@ -34,7 +34,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Symfony\CocurSlugifyExtension::load()
+     * @covers \Cocur\Slugify\Bridge\Symfony\CocurSlugifyExtension::load()
      */
     public function testLoad()
     {

diff --git a/tests/Bridge/Twig/SlugifyExtensionTest.php b/tests/Bridge/Twig/SlugifyExtensionTest.php
@@ -45,7 +45,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Twig\SlugifyExtension::getFilters()
+     * @covers \Cocur\Slugify\Bridge\Twig\SlugifyExtension::getFilters()
      */
     public function testGetFilters()
     {
@@ -56,7 +56,7 @@ public function testGetFilters()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\Twig\SlugifyExtension::slugifyFilter()
+     * @covers \Cocur\Slugify\Bridge\Twig\SlugifyExtension::slugifyFilter()
      */
     public function testSlugifyFilter()
     {

diff --git a/tests/Bridge/ZF2/ModuleTest.php b/tests/Bridge/ZF2/ModuleTest.php
@@ -23,7 +23,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\ZF2\Module::getServiceConfig()
+     * @covers \Cocur\Slugify\Bridge\ZF2\Module::getServiceConfig()
      */
     public function testGetServiceConfig()
     {
@@ -36,7 +36,7 @@ public function testGetServiceConfig()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\ZF2\Module::getViewHelperConfig()
+     * @covers \Cocur\Slugify\Bridge\ZF2\Module::getViewHelperConfig()
      */
     public function testGetViewHelperConfig()
     {

diff --git a/tests/Bridge/ZF2/SlugifyServiceTest.php b/tests/Bridge/ZF2/SlugifyServiceTest.php
@@ -25,7 +25,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\ZF2\SlugifyService::__invoke()
+     * @covers \Cocur\Slugify\Bridge\ZF2\SlugifyService::__invoke()
      */
     public function testInvokeWithoutCustomConfig()
     {
@@ -40,7 +40,7 @@ public function testInvokeWithoutCustomConfig()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\ZF2\SlugifyService::__invoke()
+     * @covers \Cocur\Slugify\Bridge\ZF2\SlugifyService::__invoke()
      */
     public function testInvokeWithCustomConfig()
     {

diff --git a/tests/Bridge/ZF2/SlugifyViewHelperFactoryTest.php b/tests/Bridge/ZF2/SlugifyViewHelperFactoryTest.php
@@ -26,7 +26,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\ZF2\SlugifyViewHelperFactory::__invoke()
+     * @covers \Cocur\Slugify\Bridge\ZF2\SlugifyViewHelperFactory::__invoke()
      */
     public function testCreateService()
     {

diff --git a/tests/Bridge/ZF2/SlugifyViewHelperTest.php b/tests/Bridge/ZF2/SlugifyViewHelperTest.php
@@ -23,7 +23,7 @@ class SlugifyViewHelperTest extends MockeryTestCase
     private $slugify;
 
     /**
-     * @covers Cocur\Slugify\Bridge\ZF2\SlugifyViewHelper::__construct()
+     * @covers \Cocur\Slugify\Bridge\ZF2\SlugifyViewHelper::__construct()
      */
     protected function setUp()
     {
@@ -32,7 +32,7 @@ protected function setUp()
     }
 
     /**
-     * @covers Cocur\Slugify\Bridge\ZF2\SlugifyViewHelper::__invoke()
+     * @covers \Cocur\Slugify\Bridge\ZF2\SlugifyViewHelper::__invoke()
      */
     public function testInvoke()
     {

diff --git a/tests/RuleProvider/FileRuleProviderTest.php b/tests/RuleProvider/FileRuleProviderTest.php
@@ -26,8 +26,8 @@
 class FileRuleProviderTest extends MockeryTestCase
 {
     /**
-     * @covers Cocur\Slugify\RuleProvider\FileRuleProvider::__construct()
-     * @covers Cocur\Slugify\RuleProvider\FileRuleProvider::getRules()
+     * @covers \Cocur\Slugify\RuleProvider\FileRuleProvider::__construct()
+     * @covers \Cocur\Slugify\RuleProvider\FileRuleProvider::getRules()
      */
     public function testGetRulesReturnsRulesReadFromJsonFile()
     {

diff --git a/tests/SlugifyTest.php b/tests/SlugifyTest.php
@@ -48,7 +48,7 @@ protected function setUp()
 
     /**
      * @dataProvider defaultRuleProvider
-     * @covers       Cocur\Slugify\Slugify::slugify()
+     * @covers       \Cocur\Slugify\Slugify::slugify()
      */
     public function testSlugifyReturnsSlugifiedStringUsingDefaultProvider($string, $result)
     {
@@ -58,8 +58,8 @@ public function testSlugifyReturnsSlugifiedStringUsingDefaultProvider($string, $
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::addRule()
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::addRule()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testAddRuleAddsRule()
     {
@@ -71,8 +71,8 @@ public function testAddRuleAddsRule()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::addRules()
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::addRules()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testAddRulesAddsMultipleRules()
     {
@@ -84,7 +84,7 @@ public function testAddRulesAddsMultipleRules()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::activateRuleset()
+     * @covers \Cocur\Slugify\Slugify::activateRuleset()
      */
     public function testActivateRulesetActivatesTheGivenRuleset()
     {
@@ -101,15 +101,15 @@ public function testActivateRulesetActivatesTheGivenRuleset()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::create()
+     * @covers \Cocur\Slugify\Slugify::create()
      */
     public function testCreateReturnsAnInstance()
     {
         $this->assertInstanceOf('Cocur\\Slugify\\SlugifyInterface', Slugify::create());
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::__construct()
+     * @covers \Cocur\Slugify\Slugify::__construct()
      */
     public function testConstructWithOtherRegexp()
     {
@@ -119,8 +119,8 @@ public function testConstructWithOtherRegexp()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::__construct()
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::__construct()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testDoNotConvertToLowercase()
     {
@@ -162,8 +162,8 @@ public function customRulesProvider()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::__construct()
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::__construct()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testSlugifyDefaultsToSeparatorOption()
     {
@@ -175,8 +175,8 @@ public function testSlugifyDefaultsToSeparatorOption()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::__construct()
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::__construct()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testSlugifyHonorsSeparatorArgument()
     {
@@ -188,7 +188,7 @@ public function testSlugifyHonorsSeparatorArgument()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testSlugifyOptionsArray()
     {
@@ -210,7 +210,7 @@ public function testSlugifyOptionsArray()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testSlugifyCustomRuleSet()
     {
@@ -263,7 +263,7 @@ public function defaultRuleProvider()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testSlugifyLowercaseNotAfterRegexp()
     {
@@ -282,7 +282,7 @@ public function testSlugifyLowercaseNotAfterRegexp()
     }
 
     /**
-     * @covers Cocur\Slugify\Slugify::slugify()
+     * @covers \Cocur\Slugify\Slugify::slugify()
      */
     public function testSlugifyLowercaseAfterRegexp()
     {