Test timeout issues

bin/test.sh

@@ -58,7 +58,17 @@ function test {
     cp "${exercise_dir}/.meta/${example_file}" "${outdir}/${exercise_file}.${file_ext}"
   fi
 
-  eval "${PHPUNIT_BIN}" --no-configuration "${outdir}/${test_file}"
+  # The time-limit `54` is based on an approximation of the performance.
+  # Online Test Runner is appr. 3x faster than GitHub CI on Ubuntu.
+  #
+  # The total runtime limit of the Online Test Runner is 20 seconds including
+  # Docker container launch and processing the results. That leaves appr. 18
+  # seconds for all tests of an exercise.
+  #
+  # Putting that together gives a max. of 18 seconds x3 = 54 seconds for any
+  # test class in CI.
+  # See: https://forum.exercism.org/t/test-tracks-for-the-20-seconds-limit-on-test-runners/10536/8
+  timeout 54s "${PHPUNIT_BIN}" --no-configuration "${outdir}/${test_file}"
 }
 
 function installed {

exercises/practice/robot-name/RobotNameTest.php

@@ -73,26 +73,42 @@ public function testResetName(): void
         $this->assertMatchesRegularExpression('/\w{2}\d{3}/', $name2);
     }
 
+    /**
+     * PHPUnit ^10.5 has an issue with
+     *   $this->assertArrayNotHasKey($name, $names, sprintf...
+     * that leads to test timeouts. This is fixed in PHPUnit ^11.
+     * Revert workaround
+     *   $this->assertFalse(isset($names[$name]), sprintf...
+     * when upgrading.
+     */
     public function testNameArentRecycled(): void
     {
         $names = [];
 
         for ($i = 0; $i < 10000; $i++) {
             $name = $this->robot->getName();
-            $this->assertArrayNotHasKey($name, $names, sprintf('Name %s reissued after Reset.', $name));
+            $this->assertFalse(isset($names[$name]), sprintf('Name %s reissued after Reset.', $name));
             $names[$name] = true;
             $this->robot->reset();
         }
     }
 
+    /**
+     * PHPUnit ^10.5 has an issue with
+     *   $this->assertArrayNotHasKey($name, $names, sprintf...
+     * that leads to test timeouts. This is fixed in PHPUnit ^11.
+     * Revert workaround
+     *   $this->assertFalse(isset($names[$name]), sprintf...
+     * when upgrading.
+     */
     // This test is optional.
     public function testNameUniquenessManyRobots(): void
     {
         $names = [];
 
         for ($i = 0; $i < 10000; $i++) {
             $name = (new Robot())->getName();
-            $this->assertArrayNotHasKey($name, $names, sprintf('Name %s reissued after %d robots', $name, $i));
+            $this->assertFalse(isset($names[$name]), sprintf('Name %s reissued after %d robots', $name, $i));
             $names[$name] = true;
         }
     }