Add type declarations to DateIntervalTest

Author: lcobucci

tests/Doctrine/Tests/DBAL/Types/DateIntervalTest.php

@@ -2,10 +2,12 @@
 
 namespace Doctrine\Tests\DBAL\Types;
 
+use Doctrine\DBAL\Types\ConversionException;
+use Doctrine\DBAL\Types\DateIntervalType;
 use Doctrine\DBAL\Types\Type;
 use Doctrine\Tests\DBAL\Mocks\MockPlatform;
 
-class DateIntervalTest  extends \Doctrine\Tests\DbalTestCase
+final class DateIntervalTest extends \Doctrine\Tests\DbalTestCase
 {
     /**
      * @var MockPlatform
@@ -20,15 +22,15 @@ class DateIntervalTest  extends \Doctrine\Tests\DbalTestCase
     /**
      * {@inheritDoc}
      */
-    protected function setUp()
+    protected function setUp() : void
     {
         $this->platform = new MockPlatform();
         $this->type     = Type::getType('dateinterval');
 
-        self::assertInstanceOf('Doctrine\DBAL\Types\DateIntervalType', $this->type);
+        self::assertInstanceOf(DateIntervalType::class, $this->type);
     }
 
-    public function testDateIntervalConvertsToDatabaseValue()
+    public function testDateIntervalConvertsToDatabaseValue() : void
     {
         $interval = new \DateInterval('P2Y1DT1H2M3S');
 
@@ -38,68 +40,66 @@ public function testDateIntervalConvertsToDatabaseValue()
         self::assertEquals($expected, $actual);
     }
 
-    public function testDateIntervalConvertsToPHPValue()
+    public function testDateIntervalConvertsToPHPValue() : void
     {
         $interval = $this->type->convertToPHPValue('+P02Y00M01DT01H02M03S', $this->platform);
 
-        self::assertInstanceOf('DateInterval', $interval);
-        self::assertEquals('+P02Y00M01DT01H02M03S', $interval->format('%RP%YY%MM%DDT%HH%IM%SS'));
+        self::assertInstanceOf(\DateInterval::class, $interval);
+        self::assertEquals('+P02Y00M01DT01H02M03S', $interval->format(DateIntervalType::FORMAT));
     }
 
-    public function testNegativeDateIntervalConvertsToDatabaseValue()
+    public function testNegativeDateIntervalConvertsToDatabaseValue() : void
     {
         $interval = new \DateInterval('P2Y1DT1H2M3S');
         $interval->invert = 1;
 
-        $expected = '-P02Y00M01DT01H02M03S';
         $actual = $this->type->convertToDatabaseValue($interval, $this->platform);
 
-        self::assertEquals($expected, $actual);
+        self::assertEquals('-P02Y00M01DT01H02M03S', $actual);
     }
 
-    public function testNegativeDateIntervalConvertsToPHPValue()
+    public function testNegativeDateIntervalConvertsToPHPValue() : void
     {
         $interval = $this->type->convertToPHPValue('-P02Y00M01DT01H02M03S', $this->platform);
 
-        self::assertInstanceOf('DateInterval', $interval);
-        self::assertEquals('-P02Y00M01DT01H02M03S', $interval->format('%RP%YY%MM%DDT%HH%IM%SS'));
+        self::assertInstanceOf(\DateInterval::class, $interval);
+        self::assertEquals('-P02Y00M01DT01H02M03S', $interval->format(DateIntervalType::FORMAT));
     }
 
-    public function testInvalidDateIntervalFormatConversion()
+    public function testInvalidDateIntervalFormatConversion() : void
     {
-        $this->expectException('Doctrine\DBAL\Types\ConversionException');
+        $this->expectException(ConversionException::class);
+
         $this->type->convertToPHPValue('abcdefg', $this->platform);
     }
 
-    public function testDateIntervalNullConversion()
+    public function testDateIntervalNullConversion() : void
     {
         self::assertNull($this->type->convertToPHPValue(null, $this->platform));
     }
 
     /**
      * @group DBAL-1288
      */
-    public function testRequiresSQLCommentHint()
+    public function testRequiresSQLCommentHint() : void
     {
         self::assertTrue($this->type->requiresSQLCommentHint($this->platform));
     }
 
     /**
      * @dataProvider invalidPHPValuesProvider
-     *
-     * @param mixed $value
      */
-    public function testInvalidTypeConversionToDatabaseValue($value)
+    public function testInvalidTypeConversionToDatabaseValue($value) : void
     {
-        $this->expectException('Doctrine\DBAL\Types\ConversionException');
+        $this->expectException(ConversionException::class);
 
         $this->type->convertToDatabaseValue($value, $this->platform);
     }
 
     /**
      * @return mixed[][]
      */
-    public function invalidPHPValuesProvider()
+    public function invalidPHPValuesProvider() : array
     {
         return [
             [0],