Rector type coverage level 2

Author: devnix

rector.php

@@ -11,7 +11,7 @@
         __DIR__ . '/tests',
     ])
     ->withPhpSets()
-    ->withTypeCoverageLevel(1)
+    ->withTypeCoverageLevel(2)
     ->withDeadCodeLevel(2)
     ->withCodeQualityLevel(10)
     ->withCodingStyleLevel(0)

tests/BasePoLoaderTestCase.php

@@ -11,7 +11,7 @@ abstract class BasePoLoaderTestCase extends TestCase
 {
     abstract protected function createPoLoader(): Loader;
 
-    public function testPoLoader()
+    public function testPoLoader(): void
     {
         $loader = $this->createPoLoader();
         $translations = $loader->loadFile(__DIR__.'/assets/translations.po');
@@ -260,7 +260,7 @@ public function stringDecodeProvider()
      * @param mixed $source
      * @param mixed $decoded
      */
-    public function testStringDecode($source, $decoded)
+    public function testStringDecode($source, $decoded): void
     {
         $po = <<<EOT
 msgid "source"
@@ -270,7 +270,7 @@ public function testStringDecode($source, $decoded)
         $this->assertSame($decoded, $translations->find(null, 'source')->getTranslation());
     }
 
-    public function testMultilineDisabledTranslations()
+    public function testMultilineDisabledTranslations(): void
     {
         $po = <<<'EOT'
 #~ msgid "Last agent hours-description"

tests/CommentsTest.php

@@ -8,7 +8,7 @@
 
 class CommentsTest extends TestCase
 {
-    public function testComments()
+    public function testComments(): void
     {
         $comments = new Comments();
 
@@ -36,7 +36,7 @@ public function testComments()
         $this->assertCount(1, $comments);
     }
 
-    public function testMergeComments()
+    public function testMergeComments(): void
     {
         $comments1 = new Comments('one', 'two', 'three');
         $comments2 = new Comments('three', 'four', 'five');
@@ -50,7 +50,7 @@ public function testMergeComments()
         $this->assertNotSame($merged, $comments2);
     }
 
-    public function testCreateFromState()
+    public function testCreateFromState(): void
     {
         $state = ['comments' => ['First comment', 'Second comment']];
         $comments = Comments::__set_state($state);

tests/FlagsTest.php

@@ -8,7 +8,7 @@
 
 class FlagsTest extends TestCase
 {
-    public function testFlags()
+    public function testFlags(): void
     {
         $flags = new Flags();
 
@@ -41,7 +41,7 @@ public function testFlags()
         $this->assertCount(2, $flags);
     }
 
-    public function testMergeFlags()
+    public function testMergeFlags(): void
     {
         $flags1 = new Flags('one', 'two', 'three');
         $flags2 = new Flags('three', 'four', 'five');
@@ -61,7 +61,7 @@ public function testMergeFlags()
         $this->assertNotSame($merged, $flags2);
     }
 
-    public function testCreateFromState()
+    public function testCreateFromState(): void
     {
         $state = ['flags' => ['one', 'two']];
         $flags = Flags::__set_state($state);

tests/HeadersTest.php

@@ -9,7 +9,7 @@
 
 class HeadersTest extends TestCase
 {
-    public function testHeaders()
+    public function testHeaders(): void
     {
         $headers = new Headers();
 
@@ -41,7 +41,7 @@ public function testHeaders()
         $this->assertCount(0, $headers);
     }
 
-    public function testDomain()
+    public function testDomain(): void
     {
         $headers = new Headers();
         $headers->setDomain('foo');
@@ -52,7 +52,7 @@ public function testDomain()
         $this->assertSame('foo', $headers->getDomain());
     }
 
-    public function testLanguage()
+    public function testLanguage(): void
     {
         $headers = new Headers();
         $headers->setLanguage('gl_ES');
@@ -63,15 +63,15 @@ public function testLanguage()
         $this->assertSame('gl_ES', $headers->getLanguage());
     }
 
-    public function testInvalidLanguage()
+    public function testInvalidLanguage(): void
     {
         $this->expectException(InvalidArgumentException::class);
 
         $headers = new Headers();
         $headers->setPluralForm(1, 'foo');
     }
 
-    public function testPluralForm()
+    public function testPluralForm(): void
     {
         $headers = new Headers();
         $headers->setPluralForm(2, '(n=1)');
@@ -82,7 +82,7 @@ public function testPluralForm()
         $this->assertSame([2, '(n=1)'], $headers->getPluralForm());
     }
 
-    public function testMergeHeaders()
+    public function testMergeHeaders(): void
     {
         $headers1 = new Headers(['X-Domain' => 'foo', 'Language' => 'gl_ES']);
         $headers2 = new Headers(['Translator' => 'Oscar Otero', 'Language' => 'ru']);
@@ -97,7 +97,7 @@ public function testMergeHeaders()
         $this->assertNotSame($merged, $headers2);
     }
 
-    public function testCreateFromState()
+    public function testCreateFromState(): void
     {
         $state = ['headers' => ['X-Domain' => 'foo']];
         $headers = Headers::__set_state($state);

tests/MergeTest.php

@@ -93,7 +93,7 @@ private static function createPO(): Translations
         return $translations;
     }
 
-    public function testNoStrategy()
+    public function testNoStrategy(): void
     {
         $pot = self::createPOT();
         $po = self::createPO();
@@ -106,7 +106,7 @@ public function testNoStrategy()
     /**
      * We want to use the scanner to fetch new entries and complete them with PO files.
      */
-    public function testScanAndLoadStrategy()
+    public function testScanAndLoadStrategy(): void
     {
         $pot = self::createPOT();
         $po = self::createPO();

tests/MoGeneratorTest.php

@@ -11,7 +11,7 @@
 
 class MoGeneratorTest extends TestCase
 {
-    public function testMoGenerator()
+    public function testMoGenerator(): void
     {
         $generator = (new MoGenerator())->includeHeaders();
         $loader = new MoLoader();

tests/MoLoaderTest.php

@@ -9,7 +9,7 @@
 
 class MoLoaderTest extends TestCase
 {
-    public function testMoLoader()
+    public function testMoLoader(): void
     {
         $loader = new MoLoader();
         $translations = $loader->loadFile(__DIR__.'/assets/translations.mo');

tests/ParsedFunctionsAndCommentsTest.php

@@ -8,7 +8,7 @@
 
 class ParsedFunctionsAndCommentsTest extends TestCase
 {
-    public function testParsedFunction()
+    public function testParsedFunction(): void
     {
         $function = new ParsedFunction('__', 'template.php', 45);
 

tests/PoGeneratorTest.php

@@ -10,7 +10,7 @@
 
 class PoGeneratorTest extends TestCase
 {
-    public function testPoLoader()
+    public function testPoLoader(): void
     {
         $generator = new PoGenerator();
         $translations = Translations::create('my-domain');
@@ -112,7 +112,7 @@ public function stringEncodeProvider()
      * @param mixed $encoded
      * @param mixed $decoded
      */
-    public function testStringEncode($encoded, $decoded)
+    public function testStringEncode($encoded, $decoded): void
     {
         $this->assertSame($encoded, PoGenerator::encode($decoded));
     }