Roave / BackwardCompatibilityCheck

test/unit/DetectChanges/BCBreak/TraitBased/TraitBecameInterfaceTest.php

Spacing +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(static function (Change $change) : string {
+            array_map(static function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -68,7 +68,7 @@ 
             ,
             $locator
         ));
-        $toReflector   = new ClassReflector(new StringSourceLocator(
+        $toReflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 
@@ -102,7 +102,7 @@ 
             array_keys($classes),
             array_map(
                 /** @psalm-param list<string> $errors https://github.com/vimeo/psalm/issues/2772 */
-                static function (string $className, array $errors) use ($fromReflector, $toReflector) : array {
+                static function(string $className, array $errors) use ($fromReflector, $toReflector) : array {
                     return [
                         $fromReflector->reflect($className),
                         $toReflector->reflect($className),

test/unit/DetectChanges/BCBreak/TraitBased/TraitBecameClassTest.php

Spacing +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(static function (Change $change) : string {
+            array_map(static function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -68,7 +68,7 @@ 
             ,
             $locator
         ));
-        $toReflector   = new ClassReflector(new StringSourceLocator(
+        $toReflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 
@@ -102,7 +102,7 @@ 
             array_keys($classes),
             array_map(
                 /** @psalm-param list<string> $errors https://github.com/vimeo/psalm/issues/2772 */
-                static function (string $className, array $errors) use ($fromReflector, $toReflector) : array {
+                static function(string $className, array $errors) use ($fromReflector, $toReflector) : array {
                     return [
                         $fromReflector->reflect($className),
                         $toReflector->reflect($className),

test/unit/DetectChanges/BCBreak/MethodBased/MethodScopeChangedTest.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(static function (Change $change) : string {
+            array_map(static function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -128,7 +128,7 @@ 
             array_keys($properties),
             array_map(
                 /** @psalm-param list<string> $errorMessages https://github.com/vimeo/psalm/issues/2772 */
-                static function (string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
+                static function(string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
                     return [
                         $fromClass->getMethod($methodName),
                         $toClass->getMethod($methodName),

test/unit/DetectChanges/BCBreak/MethodBased/MethodConcretenessChangedTest.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(static function (Change $change) : string {
+            array_map(static function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -128,7 +128,7 @@ 
             array_keys($properties),
             array_map(
                 /** @psalm-param list<string> $errorMessages https://github.com/vimeo/psalm/issues/2772 */
-                static function (string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
+                static function(string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
                     return [
                         $fromClass->getMethod($methodName),
                         $toClass->getMethod($methodName),

test/unit/DetectChanges/BCBreak/ClassBased/ExcludeInternalClassTest.php

Indentation +3 added lines, -3 removed lines

@@ -33,9 +33,9 @@
 
         $check = $this->createMock(ClassBased::class);
         $check->expects(self::once())
-              ->method('__invoke')
-              ->with($fromReflection, $toReflection)
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
+                ->method('__invoke')
+                ->with($fromReflection, $toReflection)
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
 
         self::assertEquals(
             Changes::fromList(Change::removed('foo', true)),

test/unit/Git/GitCheckoutRevisionToTemporaryPathTest.php

Spacing +5 added lines, -5 removed lines

@@ -82,7 +82,7 @@ 
                 ->getOutput()
         );
 
-        file_put_contents($repoPath . '/a-file.txt', 'file contents');
+        file_put_contents($repoPath.'/a-file.txt', 'file contents');
 
         (new Process(['git', 'add', 'a-file.txt'], $repoPath))
             ->mustRun();
@@ -102,8 +102,8 @@ 
         $first            = $git->checkout($sourceRepository, $firstCommit);
         $second           = $git->checkout($sourceRepository, $secondCommit);
 
-        self::assertFileDoesNotExist($first->__toString() . '/a-file.txt');
-        self::assertFileExists($second->__toString() . '/a-file.txt');
+        self::assertFileDoesNotExist($first->__toString().'/a-file.txt');
+        self::assertFileExists($second->__toString().'/a-file.txt');
 
         $git->remove($first);
         $git->remove($second);
@@ -113,7 +113,7 @@ 
 
     public function testExceptionIsThrownWhenTwoPathsCollide() : void
     {
-        $git              = new GitCheckoutRevisionToTemporaryPath(static function () : string {
+        $git              = new GitCheckoutRevisionToTemporaryPath(static function() : string {
             return 'foo';
         });
         $sourceRepository = $this->sourceRepository();
@@ -141,6 +141,6 @@ 
 
     private function sourceRepository() : CheckedOutRepository
     {
-        return CheckedOutRepository::fromPath(realpath(__DIR__ . '/../../..'));
+        return CheckedOutRepository::fromPath(realpath(__DIR__.'/../../..'));
     }
 }

src/DetectChanges/BCBreak/ClassBased/PropertyRemoved.php

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
             array_keys($this->accessibleProperties($toClass))
         );
 
-        return Changes::fromList(...array_map(function (string $property) use ($fromProperties) : Change {
+        return Changes::fromList(...array_map(function(string $property) use ($fromProperties) : Change {
             return Change::removed(
                 sprintf('Property %s was removed', $this->formatProperty->__invoke($fromProperties[$property])),
                 true
@@ -45,12 +45,12 @@ 
     /** @return ReflectionProperty[] */
     private function accessibleProperties(ReflectionClass $class) : array
     {
-        $classIsOpen = ! $class->isFinal();
+        $classIsOpen = !$class->isFinal();
 
-        return array_filter($class->getProperties(), function (ReflectionProperty $property) use ($classIsOpen) : bool {
+        return array_filter($class->getProperties(), function(ReflectionProperty $property) use ($classIsOpen) : bool {
             return ($property->isPublic()
                 || ($classIsOpen && $property->isProtected()))
-                && ! $this->isInternalDocComment($property->getDocComment());
+                && !$this->isInternalDocComment($property->getDocComment());
         });
     }