Roave / BackwardCompatibilityCheck

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()));
                 }
             )
         );

src/DetectChanges/BCBreak/ClassBased/PropertyRemoved.php

Spacing +3 added lines, -3 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,10 +45,10 @@ 
     /** @return ReflectionProperty[] */
     private function accessibleProperties(ReflectionClass $class) : array
     {
-        return array_filter($class->getProperties(), function (ReflectionProperty $property) : bool {
+        return array_filter($class->getProperties(), function(ReflectionProperty $property) : bool {
             return ($property->isPublic()
                 || $property->isProtected())
-                && ! $this->isInternalDocComment($property->getDocComment());
+                && !$this->isInternalDocComment($property->getDocComment());
         });
     }
 

src/DetectChanges/BCBreak/ClassBased/MethodRemoved.php

Spacing +4 added lines, -4 removed lines

@@ -36,7 +36,7 @@ 
             array_change_key_case($this->accessibleMethods($toClass), CASE_UPPER)
         );
 
-        return Changes::fromList(...array_values(array_map(function (ReflectionMethod $method) : Change {
+        return Changes::fromList(...array_values(array_map(function(ReflectionMethod $method) : Change {
             return Change::removed(
                 sprintf('Method %s was removed', $this->formatFunction->__invoke($method)),
                 true
@@ -47,14 +47,14 @@ 
     /** @return ReflectionMethod[] */
     private function accessibleMethods(ReflectionClass $class) : array
     {
-        $methods = array_filter($class->getMethods(), function (ReflectionMethod $method) : bool {
+        $methods = array_filter($class->getMethods(), function(ReflectionMethod $method) : bool {
             return ($method->isPublic()
                 || $method->isProtected())
-                && ! $this->isInternalDocComment($method->getDocComment());
+                && !$this->isInternalDocComment($method->getDocComment());
         });
 
         return array_combine(
-            array_map(static function (ReflectionMethod $method) : string {
+            array_map(static function(ReflectionMethod $method) : string {
                 return $method->getName();
             }, $methods),
             $methods

src/DetectChanges/BCBreak/ClassBased/ClassBecameInternal.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 {
     public function __invoke(ReflectionClass $fromClass, ReflectionClass $toClass) : Changes
     {
-        if (! $this->isInternalDocComment($fromClass->getDocComment())
+        if (!$this->isInternalDocComment($fromClass->getDocComment())
             && $this->isInternalDocComment($toClass->getDocComment())
         ) {
             return Changes::fromList(Change::changed(

src/DetectChanges/BCBreak/InterfaceBased/InterfaceBecameClass.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
 {
     public function __invoke(ReflectionClass $fromInterface, ReflectionClass $toInterface) : Changes
     {
-        if (! $this->isClass($toInterface) || ! $fromInterface->isInterface()) {
+        if (!$this->isClass($toInterface) || !$fromInterface->isInterface()) {
             // checking whether a class became an interface is done in `ClassBecameInterface`
             return Changes::empty();
         }
@@ -33,6 +33,6 @@ 
      */
     private function isClass(ReflectionClass $class) : bool
     {
-        return ! ($class->isTrait() || $class->isInterface());
+        return !($class->isTrait() || $class->isInterface());
     }
 }