Roave / BackwardCompatibilityCheck

test/unit/Comparator/BackwardsCompatibility/ClassBased/ConstantRemovedTest.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -43,11 +43,11 @@ 
         return [
             'RoaveTestAsset\\ClassWithConstantsBeingRemoved' => [
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/old/ClassWithConstantsBeingRemoved.php',
+                    __DIR__.'/../../../../asset/api/old/ClassWithConstantsBeingRemoved.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithConstantsBeingRemoved'),
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/new/ClassWithConstantsBeingRemoved.php',
+                    __DIR__.'/../../../../asset/api/new/ClassWithConstantsBeingRemoved.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithConstantsBeingRemoved'),
                 [

test/unit/Comparator/BackwardsCompatibility/ClassBased/PropertyRemovedTest.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -45,11 +45,11 @@ 
         return [
             'RoaveTestAsset\\ClassWithPropertiesBeingRemoved' => [
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/old/ClassWithPropertiesBeingRemoved.php',
+                    __DIR__.'/../../../../asset/api/old/ClassWithPropertiesBeingRemoved.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithPropertiesBeingRemoved'),
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/new/ClassWithPropertiesBeingRemoved.php',
+                    __DIR__.'/../../../../asset/api/new/ClassWithPropertiesBeingRemoved.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithPropertiesBeingRemoved'),
                 [

src/Comparator/BackwardsCompatibility/ClassBased/ConstantRemoved.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
             $this->accessibleConstants($toClass)
         );
 
-        return Changes::fromArray(array_values(array_map(function (ReflectionClassConstant $constant) use ($fromClass) : Change {
+        return Changes::fromArray(array_values(array_map(function(ReflectionClassConstant $constant) use ($fromClass) : Change {
             return Change::removed(
                 sprintf('Constant %s::%s was removed', $fromClass->getName(), $constant->getName()),
                 true
@@ -32,7 +32,7 @@ 
     /** @return ReflectionClassConstant[] */
     private function accessibleConstants(ReflectionClass $class) : array
     {
-        return array_filter($class->getReflectionConstants(), function (ReflectionClassConstant $constant) : bool {
+        return array_filter($class->getReflectionConstants(), function(ReflectionClassConstant $constant) : bool {
             return $constant->isPublic() || $constant->isProtected();
         });
     }

bin/api-compare.php

Spacing +3 added lines, -3 removed lines

@@ -20,9 +20,9 @@
 use Symfony\Component\Console\Application;
 use function file_exists;
 
-(function () : void {
-    foreach ([__DIR__ . '/../vendor/autoload.php', __DIR__ . '/../autoload.php'] as $autoload) {
-        if (! file_exists($autoload)) {
+(function() : void {
+    foreach ([__DIR__.'/../vendor/autoload.php', __DIR__.'/../autoload.php'] as $autoload) {
+        if (!file_exists($autoload)) {
             continue;
         }
 

test/unit/ChangeTest.php

Spacing +6 added lines, -6 removed lines

@@ -17,7 +17,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::added($changeText, false);
-        self::assertSame('     ADDED: ' . $changeText, (string) $change);
+        self::assertSame('     ADDED: '.$changeText, (string) $change);
         self::assertTrue($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -27,7 +27,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::added($changeText, true);
-        self::assertSame('[BC] ADDED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] ADDED: '.$changeText, (string) $change);
         self::assertTrue($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -37,7 +37,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::changed($changeText, false);
-        self::assertSame('     CHANGED: ' . $changeText, (string) $change);
+        self::assertSame('     CHANGED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertTrue($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -47,7 +47,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::changed($changeText, true);
-        self::assertSame('[BC] CHANGED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] CHANGED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertTrue($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -57,7 +57,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::removed($changeText, false);
-        self::assertSame('     REMOVED: ' . $changeText, (string) $change);
+        self::assertSame('     REMOVED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertTrue($change->isRemoved());
@@ -67,7 +67,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::removed($changeText, true);
-        self::assertSame('[BC] REMOVED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] REMOVED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertTrue($change->isRemoved());

test/unit/Command/ApiCompareTest.php

Spacing +3 added lines, -3 removed lines

@@ -65,7 +65,7 @@ 
 
     public function setUp() : void
     {
-        $this->sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__ . '/../../../'));
+        $this->sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__.'/../../../'));
         chdir((string) $this->sourceRepository);
 
         $this->input  = $this->createMock(InputInterface::class);
@@ -224,7 +224,7 @@ 
 
         $this->output->expects(self::any())
             ->method('writeln')
-            ->willReturnCallback(function (string $output) use ($changeToExpect) : void {
+            ->willReturnCallback(function(string $output) use ($changeToExpect) : void {
                 self::assertContains($changeToExpect, $output);
             });
     }
@@ -271,7 +271,7 @@ 
 
         $this->getVersions->expects(self::once())
             ->method('fromRepository')
-            ->with(self::callback(function (CheckedOutRepository $checkedOutRepository) : bool {
+            ->with(self::callback(function(CheckedOutRepository $checkedOutRepository) : bool {
                 self::assertEquals($this->sourceRepository, $checkedOutRepository);
                 return true;
             }))

test/unit/Comparator/Variance/TypeIsCovariantTest.php

Spacing +2 added lines, -2 removed lines

@@ -235,7 +235,7 @@ 
         return array_merge(
             [[null]],
             array_merge(...array_map(
-                function (string $type) use ($reflector) : array {
+                function(string $type) use ($reflector) : array {
                     return [
                         [ReflectionType::createFromTypeAndReflector($type, false, $reflector)],
                         [ReflectionType::createFromTypeAndReflector($type, true, $reflector)],
@@ -259,7 +259,7 @@ 
     /** @dataProvider existingNullableTypeStrings */
     public function testCovarianceConsidersNullability(string $type) : void
     {
-        $reflector   = new ClassReflector(new StringSourceLocator(
+        $reflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 

test/unit/Comparator/Variance/TypeIsContravariantTest.php

Spacing +2 added lines, -2 removed lines

@@ -224,7 +224,7 @@ 
         return array_merge(
             [[null]],
             array_merge(...array_map(
-                function (string $type) use ($reflector) : array {
+                function(string $type) use ($reflector) : array {
                     return [
                         [ReflectionType::createFromTypeAndReflector($type, false, $reflector)],
                         [ReflectionType::createFromTypeAndReflector($type, true, $reflector)],
@@ -248,7 +248,7 @@ 
     /** @dataProvider existingNullableTypeStrings */
     public function testContravarianceConsidersNullability(string $type) : void
     {
-        $reflector   = new ClassReflector(new StringSourceLocator(
+        $reflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 

test/unit/Comparator/BackwardsCompatibility/ClassConstantBased/ClassConstantVisibilityReducedTest.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -112,7 +112,7 @@ 
         return array_combine(
             array_keys($properties),
             array_map(
-                function (string $constant, array $errorMessages) use ($fromClass, $toClass) : array {
+                function(string $constant, array $errorMessages) use ($fromClass, $toClass) : array {
                     return [
                         $fromClass->getReflectionConstant($constant),
                         $toClass->getReflectionConstant($constant),