CS

Author: AydinHassan

phpstan.neon

@@ -0,0 +1,9 @@
+parameters:
+    treatPhpDocTypesAsCertain: false
+    ignoreErrors:
+        - '#Cannot access property \$args on PhpParser\\Node\|null#'
+        - '#Call to an undefined method PhpParser\\Node\\Expr\|PhpParser\\Node\\Name\:\:toString\(\)#'
+        - '#Parameter \#1 \$array of function array_flip expects array, array\<int, int\|string\>\|int\|string given\.#'
+
+    excludes_analyse:
+        - src/TestUtils/WorkshopExerciseTest.php

src/Exercise/HaveTheLastSay.php

@@ -62,17 +62,18 @@ public function getArgs(): array
 
         file_put_contents(
             $file,
-            implode("\n", array_map(
-                fn ($row) => implode("|", $row),
-                $this->getRandomCountries($countries))
-            )
+            implode("\n", array_map(fn ($row) => implode("|", $row), $this->getRandomCountries($countries)))
         );
 
         return [
             [$file]
         ];
     }
 
+    /**
+     * @param array<array{0: string, 1: string}> $countries
+     * @return array<array{0: string, 1: string}> $countries
+     */
     private function getRandomCountries(array $countries): array
     {
         return array_intersect_key(
@@ -140,7 +141,10 @@ public function check(Input $input): ResultInterface
         }
 
         if (!$funcCall->args[1]->name) {
-            return Failure::fromNameAndReason($this->getName(), 'The second positional argument should not be specified');
+            return Failure::fromNameAndReason(
+                $this->getName(),
+                'The second positional argument should not be specified'
+            );
         }
 
         if ($funcCall->args[1]->name->name !== 'separator') {

test/Exercise/HaveTheLastSayTest.php

@@ -38,7 +38,10 @@ public function testWithNoStreamArgument(): void
 
         $this->assertVerifyWasNotSuccessful();
 
-        $this->assertResultsHasFailure(Failure::class, 'The stream argument must be specified using a positional parameter');
+        $this->assertResultsHasFailure(
+            Failure::class,
+            'The stream argument must be specified using a positional parameter'
+        );
     }
 
     public function testWithStreamAsNamedArgument(): void
@@ -47,7 +50,10 @@ public function testWithStreamAsNamedArgument(): void
 
         $this->assertVerifyWasNotSuccessful();
 
-        $this->assertResultsHasFailure(Failure::class, 'The stream argument must be specified using a positional parameter');
+        $this->assertResultsHasFailure(
+            Failure::class,
+            'The stream argument must be specified using a positional parameter'
+        );
     }
 
     public function testWithSecondArgumentSpecified(): void
@@ -56,7 +62,10 @@ public function testWithSecondArgumentSpecified(): void
 
         $this->assertVerifyWasNotSuccessful();
 
-        $this->assertResultsHasFailure(Failure::class, 'The second positional argument should not be specified');
+        $this->assertResultsHasFailure(
+            Failure::class,
+            'The second positional argument should not be specified'
+        );
     }
 
     public function testWithMoreThanTwoArguments(): void
@@ -65,7 +74,10 @@ public function testWithMoreThanTwoArguments(): void
 
         $this->assertVerifyWasNotSuccessful();
 
-        $this->assertResultsHasFailure(Failure::class, 'You should only specify the stream and separator arguments, no others');
+        $this->assertResultsHasFailure(
+            Failure::class,
+            'You should only specify the stream and separator arguments, no others'
+        );
     }
 
     public function testWithNoSeparatorArgumentSpecified(): void
@@ -74,7 +86,10 @@ public function testWithNoSeparatorArgumentSpecified(): void
 
         $this->assertVerifyWasNotSuccessful();
 
-        $this->assertResultsHasFailure(Failure::class, 'The separator argument has not been specified');
+        $this->assertResultsHasFailure(
+            Failure::class,
+            'The separator argument has not been specified'
+        );
     }
 
     public function testWithWrongNamedArgument(): void
@@ -83,7 +98,10 @@ public function testWithWrongNamedArgument(): void
 
         $this->assertVerifyWasNotSuccessful();
 
-        $this->assertResultsHasFailure(Failure::class, 'A named argument has been used, but not for the separator argument');
+        $this->assertResultsHasFailure(
+            Failure::class,
+            'A named argument has been used, but not for the separator argument'
+        );
     }
 
     public function testWithNoFgetCsv(): void