Roave / BackwardCompatibilityCheck

test/unit/Git/GetVersionCollectionFromGitRepositoryTest.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Git;
 
@@ -26,10 +26,10 @@ 
 
     public function setUp() : void
     {
-        $tmpGitRepo = sys_get_temp_dir() . '/api-compare-' . uniqid('tmpGitRepo', true);
+        $tmpGitRepo = sys_get_temp_dir().'/api-compare-'.uniqid('tmpGitRepo', true);
         mkdir($tmpGitRepo, 0777, true);
         (new Process(['git', 'init']))->setWorkingDirectory($tmpGitRepo)->mustRun();
-        file_put_contents($tmpGitRepo . '/test', uniqid('testContent', true));
+        file_put_contents($tmpGitRepo.'/test', uniqid('testContent', true));
         (new Process(['git', 'add', '.']))->setWorkingDirectory($tmpGitRepo)->mustRun();
         (new Process(['git', 'commit', '-m', '"whatever"']))->setWorkingDirectory($tmpGitRepo)->mustRun();
         $this->repoPath = CheckedOutRepository::fromPath($tmpGitRepo);
@@ -52,7 +52,7 @@ 
         self::assertSame(
             ['1.0.0'],
             array_map(
-                function (Version $version) {
+                function(Version $version) {
                     return $version->getVersionString();
                 },
                 iterator_to_array((new GetVersionCollectionFromGitRepository())->fromRepository($this->repoPath))

test/unit/ChangeTest.php

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare;
 
@@ -17,7 +17,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         self::assertSame(
-            '     ADDED: ' . $changeText,
+            '     ADDED: '.$changeText,
             (string) Change::added($changeText, false)
         );
     }
@@ -26,7 +26,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         self::assertSame(
-            '[BC] ADDED: ' . $changeText,
+            '[BC] ADDED: '.$changeText,
             (string) Change::added($changeText, true)
         );
     }
@@ -35,7 +35,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         self::assertSame(
-            '     CHANGED: ' . $changeText,
+            '     CHANGED: '.$changeText,
             (string) Change::changed($changeText, false)
         );
     }
@@ -44,7 +44,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         self::assertSame(
-            '[BC] CHANGED: ' . $changeText,
+            '[BC] CHANGED: '.$changeText,
             (string) Change::changed($changeText, true)
         );
     }
@@ -53,7 +53,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         self::assertSame(
-            '     REMOVED: ' . $changeText,
+            '     REMOVED: '.$changeText,
             (string) Change::removed($changeText, false)
         );
     }
@@ -62,7 +62,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         self::assertSame(
-            '[BC] REMOVED: ' . $changeText,
+            '[BC] REMOVED: '.$changeText,
             (string) Change::removed($changeText, true)
         );
     }

src/Comparator/BackwardsCompatibility/ClassBased/ConstantVisibilityReduced.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -30,19 +30,19 @@ 
             array_combine(
                 $sharedKeys,
                 array_map(
-                    function (string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
+                    function(string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
                         return [$visibilitiesFrom[$propertyName], $visibilitiesTo[$propertyName]];
                     },
                     $sharedKeys
                 )
             ),
-            function (array $visibilities) : bool {
+            function(array $visibilities) : bool {
                 // Note: works because public, protected and private are (luckily) sortable
                 return $visibilities[0] > $visibilities[1];
             }
         );
 
-        return Changes::fromArray(array_values(array_map(function (
+        return Changes::fromArray(array_values(array_map(function(
             string $propertyName,
             array $visibilities
         ) use (
@@ -64,7 +64,7 @@ 
     /** @return string[] indexed by constant name */
     private function constantVisibilities(ReflectionClass $class) : array
     {
-        return array_map(function (ReflectionClassConstant $constant) : string {
+        return array_map(function(ReflectionClassConstant $constant) : string {
             if ($constant->isPublic()) {
                 return self::VISIBILITY_PUBLIC;
             }

src/Comparator/BackwardsCompatibility/ClassBased/ConstantValueChanged.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -19,7 +19,7 @@ 
         $fromValues = $this->accessibleConstantValues($fromClass);
         $toValues   = $this->accessibleConstantValues($toClass);
 
-        $changedConstants = array_keys(array_filter($fromValues, function (
+        $changedConstants = array_keys(array_filter($fromValues, function(
             $constantValue,
             string $constantName
         ) use (
@@ -28,7 +28,7 @@ 
             return array_key_exists($constantName, $toValues) && $constantValue !== $toValues[$constantName];
         }, \ARRAY_FILTER_USE_BOTH));
 
-        return Changes::fromArray(array_values(array_map(function (string $constantName) use ($fromClass) : Change {
+        return Changes::fromArray(array_values(array_map(function(string $constantName) use ($fromClass) : Change {
             return Change::changed(
                 sprintf('Value of constant %s::%s changed', $fromClass->getName(), $constantName),
                 true
@@ -41,12 +41,12 @@ 
     {
         $accessibleConstants = array_filter(
             $class->getReflectionConstants(),
-            function (ReflectionClassConstant $constant) : bool {
+            function(ReflectionClassConstant $constant) : bool {
                 return $constant->isPublic() || $constant->isProtected();
             }
         );
 
-        return array_map(function (ReflectionClassConstant $constant) {
+        return array_map(function(ReflectionClassConstant $constant) {
             return $constant->getValue();
         }, $accessibleConstants);
     }

test/unit/Comparator/Variance/TypeIsContravariantTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -220,7 +220,7 @@ 
         return array_merge(
             [[null]],
             array_merge(...array_map(
-                function (string $type) : array {
+                function(string $type) : array {
                     return [
                         [ReflectionType::createFromType($type, false)],
                         [ReflectionType::createFromType($type, true)],

src/Comparator/Variance/TypeIsCovariant.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\Variance;
 
@@ -33,7 +33,7 @@ 
             return false;
         }
 
-        if ($comparedType->allowsNull() && ! $type->allowsNull()) {
+        if ($comparedType->allowsNull() && !$type->allowsNull()) {
             return false;
         }
 
@@ -49,7 +49,7 @@ 
             return false;
         }
 
-        if (strtolower($typeAsString) === 'object' && ! $comparedType->isBuiltin()) {
+        if (strtolower($typeAsString) === 'object' && !$comparedType->isBuiltin()) {
             // `object` is not covariant to a defined class type
             return true;
         }
@@ -59,7 +59,7 @@ 
             return true;
         }
 
-        if (strtolower($typeAsString) === 'iterable' && ! $comparedType->isBuiltin()) {
+        if (strtolower($typeAsString) === 'iterable' && !$comparedType->isBuiltin()) {
             /** @var ReflectionClass $comparedTypeReflectionClass */
             $comparedTypeReflectionClass = $reflector->reflect($comparedTypeAsString);
 

src/Comparator/Variance/TypeIsContravariant.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\Variance;
 
@@ -32,7 +32,7 @@ 
             return false;
         }
 
-        if ($type->allowsNull() && ! $comparedType->allowsNull()) {
+        if ($type->allowsNull() && !$comparedType->allowsNull()) {
             return false;
         }
 
@@ -48,7 +48,7 @@ 
             return true;
         }
 
-        if (strtolower($comparedTypeAsString) === 'object' && ! $type->isBuiltin()) {
+        if (strtolower($comparedTypeAsString) === 'object' && !$type->isBuiltin()) {
             // `object` is always contravariant to any object type
             return true;
         }

test/unit/Comparator/Variance/TypeIsCovariantTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -220,7 +220,7 @@ 
         return array_merge(
             [[null]],
             array_merge(...array_map(
-                function (string $type) : array {
+                function(string $type) : array {
                     return [
                         [ReflectionType::createFromType($type, false)],
                         [ReflectionType::createFromType($type, true)],