Fix: Run 'php-cs-fixer fix'

sebastianbergmann · Aug 27, 2021 · a8f8f8f · a8f8f8f
1 parent 137e72f
commit a8f8f8f
Show file tree

Hide file tree

Showing 24 changed files with 92 additions and 46 deletions.
diff --git a/src/Framework/MockObject/Builder/InvocationMocker.php b/src/Framework/MockObject/Builder/InvocationMocker.php
@@ -217,7 +217,8 @@ public function method($constraint): self
         }
 
         $configurableMethodNames = array_map(
-            static function (ConfigurableMethod $configurable) {
+            static function (ConfigurableMethod $configurable)
+            {
                 return strtolower($configurable->getName());
             },
             $this->configurableMethods

diff --git a/src/Framework/MockObject/Generator.php b/src/Framework/MockObject/Generator.php
@@ -114,7 +114,8 @@ public function getMock($type, $methods = [], array $arguments = [], string $moc
         if (is_array($type)) {
             $type = array_unique(
                 array_map(
-                    static function ($type) {
+                    static function ($type)
+                    {
                         if ($type === 'Traversable' ||
                             $type === '\\Traversable' ||
                             $type === '\\Iterator') {

diff --git a/src/Framework/MockObject/Invocation.php b/src/Framework/MockObject/Invocation.php
@@ -145,13 +145,15 @@ public function generateReturnValue()
 
             case 'callable':
             case 'closure':
-                return static function (): void {
+                return static function (): void
+                {
                 };
 
             case 'traversable':
             case 'generator':
             case 'iterable':
-                $generator = static function () {
+                $generator = static function ()
+                {
                     yield from [];
                 };
 

diff --git a/src/Framework/TestCase.php b/src/Framework/TestCase.php
@@ -1708,7 +1708,8 @@ protected function createPartialMock($originalClassName, array $methods): MockOb
 
             $mockedMethodsThatDontExist = array_filter(
                 $methods,
-                static function (string $method) use ($reflection) {
+                static function (string $method) use ($reflection)
+                {
                     return !$reflection->hasMethod($method);
                 }
             );

diff --git a/src/Runner/TestSuiteSorter.php b/src/Runner/TestSuiteSorter.php
@@ -246,7 +246,8 @@ private function suiteOnlyContainsTests(TestSuite $suite): bool
     {
         return array_reduce(
             $suite->tests(),
-            static function ($carry, $test) {
+            static function ($carry, $test)
+            {
                 return $carry && ($test instanceof TestCase || $test instanceof DataProviderTestSuite);
             },
             true
@@ -272,7 +273,8 @@ private function sortDefectsFirst(array $tests): array
             /**
              * @throws \SebastianBergmann\RecursionContext\InvalidArgumentException
              */
-            function ($left, $right) {
+            function ($left, $right)
+            {
                 return $this->cmpDefectPriorityAndTime($left, $right);
             }
         );
@@ -287,7 +289,8 @@ private function sortByDuration(array $tests): array
             /**
              * @throws \SebastianBergmann\RecursionContext\InvalidArgumentException
              */
-            function ($left, $right) {
+            function ($left, $right)
+            {
                 return $this->cmpDuration($left, $right);
             }
         );
@@ -302,7 +305,8 @@ private function sortBySize(array $tests): array
             /**
              * @throws \SebastianBergmann\RecursionContext\InvalidArgumentException
              */
-            function ($left, $right) {
+            function ($left, $right)
+            {
                 return $this->cmpSize($left, $right);
             }
         );
@@ -386,7 +390,8 @@ private function resolveDependencies(array $tests): array
                 /**
                  * @throws \SebastianBergmann\RecursionContext\InvalidArgumentException
                  */
-                static function ($test) {
+                static function ($test)
+                {
                     return self::getTestSorterUID($test);
                 },
                 $tests
@@ -417,7 +422,8 @@ private function getNormalizedDependencyNames($test): array
         }
 
         $names = array_map(
-            static function ($name) use ($testClass) {
+            static function ($name) use ($testClass)
+            {
                 return strpos($name, '::') === false ? $testClass . '::' . $name : $name;
             },
             $test->getDependencies()

diff --git a/src/TextUI/Help.php b/src/TextUI/Help.php
@@ -234,7 +234,8 @@ private function writeWithColor(): void
                     $arg = Color::colorize('fg-green', str_pad($option['arg'], $this->maxArgLength));
                     $arg = preg_replace_callback(
                         '/(<[^>]+>)/',
-                        static function ($matches) {
+                        static function ($matches)
+                        {
                             return Color::colorize('fg-cyan', $matches[0]);
                         },
                         $arg

diff --git a/src/Util/Annotation/DocBlock.php b/src/Util/Annotation/DocBlock.php
@@ -604,7 +604,8 @@ private static function extractAnnotationsFromReflector(Reflector $reflector): a
             $annotations = array_merge(
                 $annotations,
                 ...array_map(
-                    static function (ReflectionClass $trait): array {
+                    static function (ReflectionClass $trait): array
+                    {
                         return self::parseDocBlock((string) $trait->getDocComment());
                     },
                     array_values($reflector->getTraits())

diff --git a/src/Util/Color.php b/src/Util/Color.php
@@ -116,7 +116,8 @@ public static function colorizePath(string $path, ?string $prevPath = null, bool
             $last        = count($path) - 1;
             $path[$last] = preg_replace_callback(
                 '/([\-_\.]+|phpt$)/',
-                static function ($matches) {
+                static function ($matches)
+                {
                     return self::dim($matches[0]);
                 },
                 $path[$last]
@@ -139,7 +140,8 @@ public static function visualizeWhitespace(string $buffer, bool $visualizeEOL =
     {
         $replaceMap = $visualizeEOL ? self::WHITESPACE_EOL_MAP : self::WHITESPACE_MAP;
 
-        return preg_replace_callback('/\s+/', static function ($matches) use ($replaceMap) {
+        return preg_replace_callback('/\s+/', static function ($matches) use ($replaceMap)
+        {
             return self::dim(strtr($matches[0], $replaceMap));
         }, $buffer);
     }

diff --git a/src/Util/ErrorHandler.php b/src/Util/ErrorHandler.php
@@ -57,7 +57,8 @@ final class ErrorHandler
     public static function invokeIgnoringWarnings(callable $callable)
     {
         set_error_handler(
-            static function ($errorNumber, $errorString) {
+            static function ($errorNumber, $errorString)
+            {
                 if ($errorNumber === E_WARNING) {
                     return;
                 }

diff --git a/src/Util/PHP/AbstractPhpProcess.php b/src/Util/PHP/AbstractPhpProcess.php
@@ -275,7 +275,8 @@ private function processChildResult(Test $test, TestResult $result, string $stdo
                 /**
                  * @throws ErrorException
                  */
-                static function ($errno, $errstr, $errfile, $errline): void {
+                static function ($errno, $errstr, $errfile, $errline): void
+                {
                     throw new ErrorException($errstr, $errno, $errno, $errfile, $errline);
                 }
             );

diff --git a/src/Util/RegularExpression.php b/src/Util/RegularExpression.php
@@ -22,7 +22,8 @@ final class RegularExpression
     public static function safeMatch(string $pattern, string $subject)
     {
         return ErrorHandler::invokeIgnoringWarnings(
-            static function () use ($pattern, $subject) {
+            static function () use ($pattern, $subject)
+            {
                 return preg_match($pattern, $subject);
             }
         );

diff --git a/src/Util/TestDox/CliTestDoxPrinter.php b/src/Util/TestDox/CliTestDoxPrinter.php
@@ -286,7 +286,8 @@ protected function formatTestResultMessage(Throwable $t, array $result, ?string
 
         if ($this->colors) {
             $color  = self::STATUS_STYLES[$result['status']]['color'] ?? '';
-            $prefix = array_map(static function ($p) use ($color) {
+            $prefix = array_map(static function ($p) use ($color)
+            {
                 return Color::colorize($color, $p);
             }, self::PREFIX_DECORATED);
         }

diff --git a/src/Util/TestDox/NamePrettifier.php b/src/Util/TestDox/NamePrettifier.php
@@ -143,7 +143,8 @@ public function prettifyTestCase(TestCase $test): string
         $annotations                = $test->getAnnotations();
         $annotationWithPlaceholders = false;
 
-        $callback = static function (string $variable): string {
+        $callback = static function (string $variable): string
+        {
             return sprintf('/%s(?=\b)/', preg_quote($variable, '/'));
         };
 
@@ -314,7 +315,8 @@ private function mapTestMethodParameterNamesToProvidedDataValues(TestCase $test)
         }
 
         if ($this->useColor) {
-            $providedData = array_map(static function ($value) {
+            $providedData = array_map(static function ($value)
+            {
                 return Color::colorize('fg-cyan', Color::visualizeWhitespace((string) $value, true));
             }, $providedData);
         }

diff --git a/src/Util/TestDox/TestDoxPrinter.php b/src/Util/TestDox/TestDoxPrinter.php
@@ -374,7 +374,8 @@ protected function prefixLines(string $prefix, string $message): string
         return implode(
             PHP_EOL,
             array_map(
-                static function (string $text) use ($prefix) {
+                static function (string $text) use ($prefix)
+                {
                     return '   ' . $prefix . ($text ? ' ' . $text : '');
                 },
                 preg_split('/\r\n|\r|\n/', $message)

diff --git a/src/Util/TestDox/XmlResultPrinter.php b/src/Util/TestDox/XmlResultPrinter.php
@@ -158,7 +158,8 @@ public function endTest(Test $test, float $time): void
 
         $groups = array_filter(
             $test->getGroups(),
-            static function ($group) {
+            static function ($group)
+            {
                 return !($group === 'small' || $group === 'medium' || $group === 'large');
             }
         );

diff --git a/src/Util/XdebugFilterScriptGenerator.php b/src/Util/XdebugFilterScriptGenerator.php
@@ -27,7 +27,8 @@ public function generate(array $filterData): string
         $items = $this->getWhitelistItems($filterData);
 
         $files = array_map(
-            static function ($item) {
+            static function ($item)
+            {
                 return sprintf(
                     "        '%s'",
                     $item

diff --git a/tests/unit/Framework/Assert/FunctionsTest.php b/tests/unit/Framework/Assert/FunctionsTest.php
@@ -54,7 +54,8 @@ public function provideStaticAssertionMethodNames(): array
 
         return array_reduce(
             $matches[1],
-            static function (array $functionNames, string $functionName) {
+            static function (array $functionNames, string $functionName)
+            {
                 $functionNames[$functionName] = [$functionName];
 
                 return $functionNames;

diff --git a/tests/unit/Framework/AssertTest.php b/tests/unit/Framework/AssertTest.php
@@ -1368,7 +1368,8 @@ public function testAssertThatCallback(): void
     {
         $this->assertThat(
             null,
-            $this->callback(static function ($other) {
+            $this->callback(static function ($other)
+            {
                 return true;
             })
         );
@@ -2041,7 +2042,8 @@ public function testScalarTypeCanBeAsserted(): void
 
     public function testCallableTypeCanBeAsserted(): void
     {
-        $this->assertIsCallable(static function (): void {
+        $this->assertIsCallable(static function (): void
+        {
         });
 
         try {
@@ -2188,7 +2190,8 @@ public function testNotCallableTypeCanBeAsserted(): void
         $this->assertIsNotCallable(null);
 
         try {
-            $this->assertIsNotCallable(static function (): void {
+            $this->assertIsNotCallable(static function (): void
+            {
             });
         } catch (AssertionFailedError $e) {
             return;

diff --git a/tests/unit/Framework/Constraint/CallbackTest.php b/tests/unit/Framework/Constraint/CallbackTest.php
@@ -28,11 +28,13 @@ public function callbackReturningTrue()
 
     public function testConstraintCallback(): void
     {
-        $closureReflect = static function ($parameter) {
+        $closureReflect = static function ($parameter)
+        {
             return $parameter;
         };
 
-        $closureWithoutParameter = static function () {
+        $closureWithoutParameter = static function ()
+        {
             return true;
         };
 
@@ -56,7 +58,8 @@ public function testConstraintCallback(): void
 
     public function testConstraintCallbackFailure(): void
     {
-        $constraint = new Callback(static function () {
+        $constraint = new Callback(static function ()
+        {
             return false;
         });
 

diff --git a/tests/unit/Framework/Constraint/LogicalAndTest.php b/tests/unit/Framework/Constraint/LogicalAndTest.php
@@ -55,7 +55,8 @@ public function testCountReturnsCountOfComposedConstraints(): void
             8,
         ];
 
-        $constraints = array_map(static function (int $count) {
+        $constraints = array_map(static function (int $count)
+        {
             return CountConstraint::fromCount($count);
         }, $counts);
 
@@ -76,7 +77,8 @@ public function testToStringReturnsImplodedStringRepresentationOfComposedConstra
             'is rich in unsaturated fats',
         ];
 
-        $constraints = array_map(static function (string $name) {
+        $constraints = array_map(static function (string $name)
+        {
             return NamedConstraint::fromName($name);
         }, $names);
 
@@ -240,7 +242,8 @@ private function stringify(array $constraints): string
     {
         return implode(
             ' and ',
-            array_map(static function (Constraint $constraint) {
+            array_map(static function (Constraint $constraint)
+            {
                 return $constraint->toString();
             }, $constraints)
         );

diff --git a/tests/unit/Framework/Constraint/LogicalOrTest.php b/tests/unit/Framework/Constraint/LogicalOrTest.php
@@ -47,7 +47,8 @@ public function testCountReturnsCountOfComposedConstraints(): void
             8,
         ];
 
-        $constraints = array_map(static function (int $count) {
+        $constraints = array_map(static function (int $count)
+        {
             return CountConstraint::fromCount($count);
         }, $counts);
 
@@ -68,7 +69,8 @@ public function testToStringReturnsImplodedStringRepresentationOfComposedConstra
             'is rich in unsaturated fats',
         ];
 
-        $constraints = array_map(static function (string $name) {
+        $constraints = array_map(static function (string $name)
+        {
             return NamedConstraint::fromName($name);
         }, $names);
 
@@ -234,7 +236,8 @@ private function stringify(array $constraints): string
     {
         return implode(
             ' or ',
-            array_map(static function (Constraint $constraint) {
+            array_map(static function (Constraint $constraint)
+            {
                 return $constraint->toString();
             }, $constraints)
         );

diff --git a/tests/unit/Framework/Constraint/LogicalXorTest.php b/tests/unit/Framework/Constraint/LogicalXorTest.php
@@ -25,7 +25,8 @@ public function testFromConstraintsReturnsConstraint(): void
         $other = 'Foo';
         $count = 5;
 
-        $constraints = array_map(function () use ($other) {
+        $constraints = array_map(function () use ($other)
+        {
             static $count = 0;
 
             $constraint = $this->getMockBuilder(Constraint::class)->getMock();