Completed
Pull Request — master (#38)
by Marco
02:14
created
src/Comparator/BackwardsCompatibility/ClassBased/FinalClassChanged.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
6 6
 
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
 
20 20
     public function compare(ReflectionClass $fromClass, ReflectionClass $toClass) : Changes
21 21
     {
22
-        if (! $fromClass->isFinal()) {
22
+        if (!$fromClass->isFinal()) {
23 23
             return Changes::new();
24 24
         }
25 25
 
Please login to merge, or discard this patch.
Comparator/BackwardsCompatibility/FunctionBased/ParameterTypeChanged.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\FunctionBased;
6 6
 
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 
79 79
     private function typeToString(?ReflectionType $type) : string
80 80
     {
81
-        if (! $type) {
81
+        if (!$type) {
82 82
             return 'no type';
83 83
         }
84 84
 
Please login to merge, or discard this patch.
src/Comparator/BackwardsCompatibility/FunctionBased/ReturnTypeChanged.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\FunctionBased;
6 6
 
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 
79 79
     private function typeToString(?ReflectionType $type) : string
80 80
     {
81
-        if (! $type) {
81
+        if (!$type) {
82 82
             return 'no type';
83 83
         }
84 84
 
Please login to merge, or discard this patch.
test/unit/Formatter/MarkdownPipedToSymfonyConsoleFormatterTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace RoaveTest\ApiCompare\Formatter;
6 6
 
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 
37 37
         $output->expects(self::any())
38 38
             ->method('writeln')
39
-            ->willReturnCallback(function (string $output) use ($changeToExpect) : void {
39
+            ->willReturnCallback(function(string $output) use ($changeToExpect) : void {
40 40
                 self::assertContains($changeToExpect, $output);
41 41
             });
42 42
 
Please login to merge, or discard this patch.
test/unit/Command/ApiCompareTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace RoaveTest\ApiCompare\Command;
6 6
 
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 
66 66
     public function setUp() : void
67 67
     {
68
-        $this->sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__ . '/../../../'));
68
+        $this->sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__.'/../../../'));
69 69
         chdir((string) $this->sourceRepository);
70 70
 
71 71
         $this->input  = $this->createMock(InputInterface::class);
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
 
226 226
         $this->output->expects(self::any())
227 227
             ->method('writeln')
228
-            ->willReturnCallback(function (string $output) use ($changeToExpect) : void {
228
+            ->willReturnCallback(function(string $output) use ($changeToExpect) : void {
229 229
                 self::assertContains($changeToExpect, $output);
230 230
             });
231 231
     }
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
 
273 273
         $this->getVersions->expects(self::once())
274 274
             ->method('fromRepository')
275
-            ->with(self::callback(function (CheckedOutRepository $checkedOutRepository) : bool {
275
+            ->with(self::callback(function(CheckedOutRepository $checkedOutRepository) : bool {
276 276
                 self::assertEquals($this->sourceRepository, $checkedOutRepository);
277 277
                 return true;
278 278
             }))
Please login to merge, or discard this patch.
test/unit/ChangeTest.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace RoaveTest\ApiCompare;
6 6
 
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
     {
18 18
         $changeText = uniqid('changeText', true);
19 19
         $change     = Change::added($changeText, false);
20
-        self::assertSame('     ADDED: ' . $changeText, (string) $change);
20
+        self::assertSame('     ADDED: '.$changeText, (string) $change);
21 21
         self::assertTrue($change->isAdded());
22 22
         self::assertFalse($change->isChanged());
23 23
         self::assertFalse($change->isRemoved());
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
     {
28 28
         $changeText = uniqid('changeText', true);
29 29
         $change     = Change::added($changeText, true);
30
-        self::assertSame('[BC] ADDED: ' . $changeText, (string) $change);
30
+        self::assertSame('[BC] ADDED: '.$changeText, (string) $change);
31 31
         self::assertTrue($change->isAdded());
32 32
         self::assertFalse($change->isChanged());
33 33
         self::assertFalse($change->isRemoved());
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
     {
38 38
         $changeText = uniqid('changeText', true);
39 39
         $change     = Change::changed($changeText, false);
40
-        self::assertSame('     CHANGED: ' . $changeText, (string) $change);
40
+        self::assertSame('     CHANGED: '.$changeText, (string) $change);
41 41
         self::assertFalse($change->isAdded());
42 42
         self::assertTrue($change->isChanged());
43 43
         self::assertFalse($change->isRemoved());
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
     {
48 48
         $changeText = uniqid('changeText', true);
49 49
         $change     = Change::changed($changeText, true);
50
-        self::assertSame('[BC] CHANGED: ' . $changeText, (string) $change);
50
+        self::assertSame('[BC] CHANGED: '.$changeText, (string) $change);
51 51
         self::assertFalse($change->isAdded());
52 52
         self::assertTrue($change->isChanged());
53 53
         self::assertFalse($change->isRemoved());
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
     {
58 58
         $changeText = uniqid('changeText', true);
59 59
         $change     = Change::removed($changeText, false);
60
-        self::assertSame('     REMOVED: ' . $changeText, (string) $change);
60
+        self::assertSame('     REMOVED: '.$changeText, (string) $change);
61 61
         self::assertFalse($change->isAdded());
62 62
         self::assertFalse($change->isChanged());
63 63
         self::assertTrue($change->isRemoved());
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
     {
68 68
         $changeText = uniqid('changeText', true);
69 69
         $change     = Change::removed($changeText, true);
70
-        self::assertSame('[BC] REMOVED: ' . $changeText, (string) $change);
70
+        self::assertSame('[BC] REMOVED: '.$changeText, (string) $change);
71 71
         self::assertFalse($change->isAdded());
72 72
         self::assertFalse($change->isChanged());
73 73
         self::assertTrue($change->isRemoved());
Please login to merge, or discard this patch.
unit/Comparator/BackwardsCompatibility/ClassBased/ClassBecameFinalTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
6 6
 
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 
32 32
         self::assertSame(
33 33
             $expectedMessages,
34
-            array_map(function (Change $change) : string {
34
+            array_map(function(Change $change) : string {
35 35
                 return $change->__toString();
36 36
             }, iterator_to_array($changes))
37 37
         );
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
             ,
54 54
             $locator
55 55
         ));
56
-        $toReflector   = new ClassReflector(new StringSourceLocator(
56
+        $toReflector = new ClassReflector(new StringSourceLocator(
57 57
             <<<'PHP'
58 58
 <?php
59 59
 
Please login to merge, or discard this patch.
test/unit/Comparator/Variance/TypeIsContravariantTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
6 6
 
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
224 224
         return array_merge(
225 225
             [[null]],
226 226
             array_merge(...array_map(
227
-                function (string $type) use ($reflector) : array {
227
+                function(string $type) use ($reflector) : array {
228 228
                     return [
229 229
                         [ReflectionType::createFromTypeAndReflector($type, false, $reflector)],
230 230
                         [ReflectionType::createFromTypeAndReflector($type, true, $reflector)],
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
     /** @dataProvider existingNullableTypeStrings */
249 249
     public function testContravarianceConsidersNullability(string $type) : void
250 250
     {
251
-        $reflector   = new ClassReflector(new StringSourceLocator(
251
+        $reflector = new ClassReflector(new StringSourceLocator(
252 252
             <<<'PHP'
253 253
 <?php
254 254
 
Please login to merge, or discard this patch.
test/unit/Comparator/Variance/TypeIsCovariantTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
6 6
 
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
         return array_merge(
226 226
             [[null]],
227 227
             array_merge(...array_map(
228
-                function (string $type) use ($reflector) : array {
228
+                function(string $type) use ($reflector) : array {
229 229
                     return [
230 230
                         [ReflectionType::createFromTypeAndReflector($type, false, $reflector)],
231 231
                         [ReflectionType::createFromTypeAndReflector($type, true, $reflector)],
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
     /** @dataProvider existingNullableTypeStrings */
250 250
     public function testCovarianceConsidersNullability(string $type) : void
251 251
     {
252
-        $reflector   = new ClassReflector(new StringSourceLocator(
252
+        $reflector = new ClassReflector(new StringSourceLocator(
253 253
             <<<'PHP'
254 254
 <?php
255 255
 
Please login to merge, or discard this patch.