Adding void types for PHPUnit 8

Author: moufmouf

tests/AlterableResultIteratorTest.php

@@ -8,7 +8,7 @@
 
 class AlterableResultIteratorTest extends TestCase
 {
-    public function testUnalteredResultSet()
+    public function testUnalteredResultSet(): void
     {
         $a = (object) ['a' => 'a'];
         $b = (object) ['b' => 'c'];
@@ -26,7 +26,7 @@ public function testUnalteredResultSet()
         $this->assertEquals($a, $alterableResultIterator->first());
     }
 
-    public function testEmptyResultSet()
+    public function testEmptyResultSet(): void
     {
         $alterableResultIterator = new AlterableResultIterator();
 
@@ -36,7 +36,7 @@ public function testEmptyResultSet()
         $this->assertNull($alterableResultIterator->first());
     }
 
-    public function testAlterEmptyResultSet()
+    public function testAlterEmptyResultSet(): void
     {
         $alterableResultIterator = new AlterableResultIterator();
 
@@ -52,7 +52,7 @@ public function testAlterEmptyResultSet()
         $this->assertEquals([$a], $alterableResultIterator->toArray());
     }
 
-    public function testAlterFilledResultSet()
+    public function testAlterFilledResultSet(): void
     {
         $a = (object) ['a' => 'a'];
         $b = (object) ['b' => 'c'];
@@ -69,7 +69,7 @@ public function testAlterFilledResultSet()
         $this->assertEquals([$c], iterator_to_array($alterableResultIterator->take(1, 1)));
     }
 
-    public function testAddAfterToArray()
+    public function testAddAfterToArray(): void
     {
         $a = (object) ['a' => 'a'];
         $b = (object) ['b' => 'c'];
@@ -87,7 +87,7 @@ public function testAddAfterToArray()
         $this->assertEquals([$a, $c], $alterableResultIterator->toArray());
     }
 
-    public function testGetIterator()
+    public function testGetIterator(): void
     {
         $a = (object) ['a' => 'a'];
         $b = (object) ['b' => 'c'];
@@ -106,21 +106,21 @@ public function testGetIterator()
         $this->assertInstanceOf(\ArrayIterator::class, $alterableResultIterator->getIterator());
     }
 
-    public function testSetException()
+    public function testSetException(): void
     {
         $alterableResultIterator = new AlterableResultIterator();
         $this->expectException(TDBMInvalidOperationException::class);
         $alterableResultIterator[0] = 'foo';
     }
 
-    public function testUnsetException()
+    public function testUnsetException(): void
     {
         $alterableResultIterator = new AlterableResultIterator();
         $this->expectException(TDBMInvalidOperationException::class);
         unset($alterableResultIterator[0]);
     }
 
-    public function testMap()
+    public function testMap(): void
     {
         $a = (object) ['foo' => 'bar'];
 
@@ -135,7 +135,7 @@ public function testMap()
         $this->assertEquals(['bar'], iterator_to_array($map));
     }
 
-    public function testSetIterator()
+    public function testSetIterator(): void
     {
         $alterableResultIterator = new AlterableResultIterator();
 

tests/Commands/AlteredConfigurationTest.php

@@ -14,7 +14,7 @@
 
 class AlteredConfigurationTest extends TestCase
 {
-    public function testAlteredConfiguration()
+    public function testAlteredConfiguration(): void
     {
         $connection = $this->getMockBuilder(Connection::class)->disableOriginalConstructor()->getMock();
         $namingStrategy = $this->getMockBuilder(NamingStrategyInterface::class)->disableOriginalConstructor()->getMock();

tests/Commands/GenerateCommandTest.php

@@ -19,7 +19,7 @@ public static function getInputDefinition()
         ]);
     }
 
-    public function testCall()
+    public function testCall(): void
     {
         $input = new ArrayInput([
         ], self::getInputDefinition());

tests/MapIteratorTest.php

@@ -28,7 +28,7 @@ class MapIteratorTest extends TestCase
     /**
      * @expectedException \TheCodingMachine\TDBM\TDBMException
      */
-    public function testConstructorException1()
+    public function testConstructorException1(): void
     {
         $mapIterator = new MapIterator(new \DateTime(), function ($item) {
             return $item;
@@ -38,14 +38,14 @@ public function testConstructorException1()
     /**
      * @expectedException \TheCodingMachine\TDBM\TDBMException
      */
-    public function testConstructorException2()
+    public function testConstructorException2(): void
     {
         $mapIterator = new MapIterator(array(1, 2, 3), function () {
             return $item;
         });
     }
 
-    public function testJsonSerialize()
+    public function testJsonSerialize(): void
     {
         $value = array(1, 2, 3);
         $mapIterator = new MapIterator($value, function ($item) {

tests/OrderByAnalyzerTest.php

@@ -9,7 +9,7 @@
 
 class OrderByAnalyzerTest extends TestCase
 {
-    public function testAnalyzeOrderBy()
+    public function testAnalyzeOrderBy(): void
     {
         $analyzer = new OrderByAnalyzer(new VoidCache(), '');
         $results = $analyzer->analyzeOrderBy('`a`, b desc, rand() DESC, masc, mytable.mycol');
@@ -46,7 +46,7 @@ public function testAnalyzeOrderBy()
         ], $results[4]);
     }
 
-    public function testExprWithAsc()
+    public function testExprWithAsc(): void
     {
         $analyzer = new OrderByAnalyzer(new VoidCache(), '');
         $results = $analyzer->analyzeOrderBy('foodesc + barasc');
@@ -59,7 +59,7 @@ public function testExprWithAsc()
         ], $results[0]);
     }
 
-    public function testCache()
+    public function testCache(): void
     {
         $analyzer = new OrderByAnalyzer(new ArrayCache(), '');
         $results = $analyzer->analyzeOrderBy('foo');

tests/StandardObjectStorageTest.php

@@ -6,7 +6,7 @@
 
 class StandardObjectStorageTest extends TestCase
 {
-    public function testObjectStorage()
+    public function testObjectStorage(): void
     {
         $objectStorage = new StandardObjectStorage();
         $this->assertNull($objectStorage->get('foo', 42));

tests/TDBMAbstractServiceTest.php

@@ -71,7 +71,7 @@ abstract class TDBMAbstractServiceTest extends TestCase
      */
     private $cache;
 
-    public static function setUpBeforeClass()
+    public static function setUpBeforeClass(): void
     {
         self::resetConnection();
 
@@ -189,7 +189,7 @@ protected function onlyMySql()
         }
     }
 
-    protected function setUp()
+    protected function setUp(): void
     {
         $this->tdbmService = new TDBMService($this->getConfiguration());
     }