Merge pull request #497 from pestphp/feature/style

style: apply fixes from PHP-CS-Fixer

src/Concerns/Pipeable.php

@@ -43,7 +43,7 @@ trait Pipeable
         $this->pipe($name, function ($next, ...$arguments) use ($handler, $filter) {
             /* @phpstan-ignore-next-line */
             if ($filter($this->value, ...$arguments)) {
-                //@phpstan-ignore-next-line
+                // @phpstan-ignore-next-line
                 $handler->bindTo($this, get_class($this))(...$arguments);
 
                 return;

src/Expectation.php

@@ -307,7 +307,7 @@ final class Expectation
     private function getExpectationClosure(string $name): Closure
     {
         if (method_exists(Mixins\Expectation::class, $name)) {
-            //@phpstan-ignore-next-line
+            // @phpstan-ignore-next-line
             return Closure::fromCallable([new Mixins\Expectation($this->value), $name]);
         }
 

src/Kernel.php

@@ -37,7 +37,7 @@ final class Kernel
     public static function boot(): self
     {
         foreach (self::$bootstrappers as $bootstrapper) {
-            //@phpstan-ignore-next-line
+            // @phpstan-ignore-next-line
             (new $bootstrapper())->__invoke();
         }
 

src/Mixins/Expectation.php

@@ -283,7 +283,7 @@ final class Expectation
     {
         $this->toBeObject();
 
-        //@phpstan-ignore-next-line
+        // @phpstan-ignore-next-line
         Assert::assertTrue(property_exists($this->value, $name));
 
         if (func_num_args() > 1) {
@@ -534,7 +534,7 @@ final class Expectation
     {
         Assert::assertIsString($this->value);
 
-        //@phpstan-ignore-next-line
+        // @phpstan-ignore-next-line
         Assert::assertJson($this->value);
 
         return $this;
@@ -580,7 +580,7 @@ final class Expectation
         try {
             Assert::assertTrue(Arr::has($array, $key));
 
-            /* @phpstan-ignore-next-line  */
+            /* @phpstan-ignore-next-line */
         } catch (ExpectationFailedException $exception) {
             throw new ExpectationFailedException("Failed asserting that an array has the key '$key'", $exception->getComparisonFailure());
         }

src/Repositories/DatasetsRepository.php

@@ -156,11 +156,11 @@ final class DatasetsRepository
                 $datasets[$index] = iterator_to_array($datasets[$index]);
             }
 
-            //@phpstan-ignore-next-line
+            // @phpstan-ignore-next-line
             foreach ($datasets[$index] as $key => $values) {
                 $values             = is_array($values) ? $values : [$values];
                 $processedDataset[] = [
-                    'label'  => self::getDatasetDescription($key, $values), //@phpstan-ignore-line
+                    'label'  => self::getDatasetDescription($key, $values), // @phpstan-ignore-line
                     'values' => $values,
                 ];
             }
@@ -189,7 +189,7 @@ final class DatasetsRepository
             $result = $tmp;
         }
 
-        //@phpstan-ignore-next-line
+        // @phpstan-ignore-next-line
         return $result;
     }
 

src/Support/Container.php

@@ -85,7 +85,7 @@ final class Container
                             }
                         }
 
-                        //@phpstan-ignore-next-line
+                        // @phpstan-ignore-next-line
                         return $this->get($candidate);
                     },
                     $constructor->getParameters()

src/Support/HigherOrderMessageCollection.php

@@ -40,7 +40,7 @@ final class HigherOrderMessageCollection
     public function chain(object $target): void
     {
         foreach ($this->messages as $message) {
-            //@phpstan-ignore-next-line
+            // @phpstan-ignore-next-line
             $target = $message->call($target) ?? $target;
         }
     }

tests/Features/Coverage.php

@@ -8,7 +8,7 @@ use Symfony\Component\Console\Output\ConsoleOutput;
 it('has plugin')->assertTrue(class_exists(CoveragePlugin::class));
 
 it('adds coverage if --coverage exist', function () {
-    $plugin = new CoveragePlugin(new ConsoleOutput());
+    $plugin    = new CoveragePlugin(new ConsoleOutput());
     $testSuite = TestSuite::getInstance();
 
     expect($plugin->coverage)->toBeFalse();

tests/Features/Expect/pipes.php

@@ -12,7 +12,7 @@ class Number
     public function __construct(
         public int $value
     ) {
-        //..
+        // ..
     }
 }
 
@@ -21,7 +21,7 @@ class Char
     public function __construct(
         public string $value
     ) {
-        //..
+        // ..
     }
 }
 
@@ -30,7 +30,7 @@ class Symbol
     public function __construct(
         public string $value
     ) {
-        //..
+        // ..
     }
 }
 
@@ -69,11 +69,11 @@ expect()->pipe('toBe', function ($next, $expected) use ($state) {
         assertInstanceOf(Char::class, $expected);
         assertEquals($this->value->value, $expected->value);
 
-        //returning nothing stops pipeline execution
+        // returning nothing stops pipeline execution
         return;
     }
 
-    //calling $next(); let the pipeline to keep running
+    // calling $next(); let the pipeline to keep running
     $next();
 });
 

tests/Features/Expect/toBe.php

@@ -5,7 +5,7 @@ use PHPUnit\Framework\ExpectationFailedException;
 expect(true)->toBeTrue()->and(false)->toBeFalse();
 
 test('strict comparisons', function () {
-    $nuno = new stdClass();
+    $nuno  = new stdClass();
     $dries = new stdClass();
 
     expect($nuno)->toBe($nuno)->not->toBe($dries);

tests/Features/Expect/toHaveProperties.php

@@ -3,24 +3,24 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 test('pass', function () {
-    $object = new stdClass();
+    $object       = new stdClass();
     $object->name = 'Jhon';
-    $object->age = 21;
+    $object->age  = 21;
 
     expect($object)->toHaveProperties(['name', 'age']);
 });
 
 test('failures', function () {
-    $object = new stdClass();
+    $object       = new stdClass();
     $object->name = 'Jhon';
 
     expect($object)->toHaveProperties(['name', 'age']);
 })->throws(ExpectationFailedException::class);
 
 test('not failures', function () {
-    $object = new stdClass();
+    $object       = new stdClass();
     $object->name = 'Jhon';
-    $object->age = 21;
+    $object->age  = 21;
 
     expect($object)->not->toHaveProperties(['name', 'age']);
 })->throws(ExpectationFailedException::class);

tests/Features/Expect/unless.php

@@ -3,9 +3,9 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 beforeEach(function () {
-    $this->unlessObject = new stdClass();
+    $this->unlessObject            = new stdClass();
     $this->unlessObject->trueValue = true;
-    $this->unlessObject->foo = 'foo';
+    $this->unlessObject->foo       = 'foo';
 });
 
 it('pass', function () {

tests/Features/Expect/when.php

@@ -3,9 +3,9 @@
 use PHPUnit\Framework\ExpectationFailedException;
 
 beforeEach(function () {
-    $this->whenObject = new stdClass();
+    $this->whenObject            = new stdClass();
     $this->whenObject->trueValue = true;
-    $this->whenObject->foo = 'foo';
+    $this->whenObject->foo       = 'foo';
 });
 
 it('pass', function () {

tests/Unit/Support/Reflection.php

@@ -10,7 +10,7 @@ it('gets file name from closure', function () {
 });
 
 it('gets property values', function () {
-    $class = new class() {
+    $class           = new class() {
         private $foo = 'bar';
     };
 

tests/Unit/TestSuite.php

@@ -8,7 +8,7 @@ use Pest\TestSuite;
 
 it('does not allow to add the same test description twice', function () {
     $testSuite = new TestSuite(getcwd(), 'tests');
-    $method = new TestCaseMethodFactory('foo', 'bar', null);
+    $method    = new TestCaseMethodFactory('foo', 'bar', null);
 
     $testSuite->tests->set($method);
     $testSuite->tests->set($method);
@@ -43,7 +43,7 @@ it('can return an array of all test suite filenames', function () {
 it('can filter the test suite filenames to those with the only method', function () {
     $testSuite = new TestSuite(getcwd(), 'tests');
 
-    $testWithOnly = new TestCaseMethodFactory('a', 'b', null);
+    $testWithOnly       = new TestCaseMethodFactory('a', 'b', null);
     $testWithOnly->only = true;
     $testSuite->tests->set($testWithOnly);
 
@@ -61,7 +61,7 @@ it('does not filter the test suite filenames to those with the only method when
 
     $test = function () {};
 
-    $testWithOnly = new TestCaseMethodFactory('a', 'b', null);
+    $testWithOnly       = new TestCaseMethodFactory('a', 'b', null);
     $testWithOnly->only = true;
     $testSuite->tests->set($testWithOnly);
 

tests/Visual/Help.php

@@ -8,7 +8,7 @@ test('visual snapshot of help command output', function () {
 
     if (getenv('REBUILD_SNAPSHOTS')) {
         $outputBuffer = new BufferedOutput();
-        $plugin = new Help($outputBuffer);
+        $plugin       = new Help($outputBuffer);
 
         $plugin();
 

tests/Visual/Success.php

@@ -2,7 +2,7 @@
 
 test('visual snapshot of test suite on success', function () {
     $testsPath = dirname(__DIR__);
-    $snapshot = implode(DIRECTORY_SEPARATOR, [
+    $snapshot  = implode(DIRECTORY_SEPARATOR, [
         $testsPath,
         '.snapshots',
         'success.txt',