Roave / BackwardCompatibilityCheck

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

Indentation +3 added lines, -3 removed lines

@@ -35,9 +35,9 @@
         /** @var ClassBased&MockObject $check */
         $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/DetectChanges/BCBreak/ClassBased/ClassBecameInternalTest.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,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))
         );
@@ -53,7 +53,7 @@ 
             ,
             $locator
         ));
-        $toReflector   = new ClassReflector(new StringSourceLocator(
+        $toReflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 

test/unit/DetectChanges/BCBreak/InterfaceBased/ExcludeInternalInterfaceTest.php

Indentation +3 added lines, -3 removed lines

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

test/unit/DetectChanges/BCBreak/FunctionBased/FunctionBecameInternalTest.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,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))
         );
@@ -96,7 +96,7 @@ 
         return array_combine(
             array_keys($functions),
             array_map(
-                static function (string $function, array $errorMessages) use ($fromReflector, $toReflector) : array {
+                static function(string $function, array $errorMessages) use ($fromReflector, $toReflector) : array {
                     return [
                         $fromReflector->reflect($function),
                         $toReflector->reflect($function),

test/unit/DetectChanges/BCBreak/FunctionBased/ExcludeInternalFunctionTest.php

Indentation +4 added lines, -4 removed lines

@@ -33,9 +33,9 @@ 
 
         $check = $this->createMock(FunctionBased::class);
         $check->expects(self::once())
-              ->method('__invoke')
-              ->with($function, $function)
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
+                ->method('__invoke')
+                ->with($function, $function)
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
 
         self::assertEquals(
             Changes::fromList(Change::removed('foo', true)),
@@ -61,7 +61,7 @@ 
 
         $check = $this->createMock(FunctionBased::class);
         $check->expects(self::never())
-              ->method('__invoke');
+                ->method('__invoke');
 
         self::assertEquals(
             Changes::empty(),

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 {
     public function testNormalFunctionsAreNotExcluded() : void
     {
-        $source   = new StringSourceLocator(
+        $source = new StringSourceLocator(
             <<<'PHP'
 <?php
 
@@ -46,7 +46,7 @@ 
 
     public function testInternalFunctionsAreExcluded() : void
     {
-        $source   = new StringSourceLocator(
+        $source = new StringSourceLocator(
             <<<'PHP'
 <?php
 

test/unit/DetectChanges/BCBreak/PropertyBased/ExcludeInternalPropertyTest.php

Indentation +4 added lines, -4 removed lines

@@ -36,9 +36,9 @@ 
 
         $check = $this->createMock(PropertyBased::class);
         $check->expects(self::once())
-              ->method('__invoke')
-              ->with($property, $property)
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
+                ->method('__invoke')
+                ->with($property, $property)
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
 
         self::assertEquals(
             Changes::fromList(Change::removed('foo', true)),
@@ -68,7 +68,7 @@ 
 
         $check = $this->createMock(PropertyBased::class);
         $check->expects(self::never())
-              ->method('__invoke');
+                ->method('__invoke');
 
         self::assertEquals(
             Changes::empty(),

test/unit/DetectChanges/BCBreak/PropertyBased/PropertyBecameInternalTest.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,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))
         );
@@ -108,7 +108,7 @@ 
         return array_combine(
             array_keys($properties),
             array_map(
-                static function (string $property, array $errorMessages) use ($fromClass, $toClass) : array {
+                static function(string $property, array $errorMessages) use ($fromClass, $toClass) : array {
                     return [
                         $fromClass->getProperty($property),
                         $toClass->getProperty($property),

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

Indentation +4 added lines, -4 removed lines

@@ -34,9 +34,9 @@ 
 
         $check = $this->createMock(MethodBased::class);
         $check->expects(self::once())
-              ->method('__invoke')
-              ->with($method, $method)
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
+                ->method('__invoke')
+                ->with($method, $method)
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
 
         self::assertEquals(
             Changes::fromList(Change::removed('foo', true)),
@@ -64,7 +64,7 @@ 
 
         $check = $this->createMock(MethodBased::class);
         $check->expects(self::never())
-              ->method('__invoke');
+                ->method('__invoke');
 
         self::assertEquals(
             Changes::empty(),

src/CompareClasses.php

Spacing +3 added lines, -3 removed lines

@@ -46,13 +46,13 @@
         ClassReflector $newSourcesWithDependencies
     ) : Changes {
         $definedApiClassNames = array_map(
-            static function (ReflectionClass $class) : string {
+            static function(ReflectionClass $class) : string {
                 return $class->getName();
             },
             array_filter(
                 $definedSymbols->getAllClasses(),
-                function (ReflectionClass $class) : bool {
-                    return ! ($class->isAnonymous() || $this->isInternalDocComment($class->getDocComment()));
+                function(ReflectionClass $class) : bool {
+                    return !($class->isAnonymous() || $this->isInternalDocComment($class->getDocComment()));
                 }
             )
         );