Merge pull request #24 from svenluijten/numbers-in-test-methods

Support numbers in PHPUnit method names

Author: ventrec

src/PrettyPrinter.php

@@ -33,20 +33,22 @@ public function endTest(Test $test, float $time): void
 
         // Convert non-breaking method name to camelCase
         $testMethodName[1] = str_replace(' ', '', ucwords($testMethodName[1], ' '));
-        
+
         // Convert snakeCase method name to camelCase
         $testMethodName[1] = str_replace('_', '', ucwords($testMethodName[1], '_'));
-        
-        preg_match_all('/((?:^|[A-Z])[a-z]+)/', $testMethodName[1], $matches);
-        $testNameArray = array_map('strtolower', $matches[0]);
 
-        // check if prefix is test remove it
-        if ($testNameArray[0] === 'test') {
-            array_shift($testNameArray);
-        }
+        preg_match_all('/((?:^|[A-Z])[a-z0-9]+)/', $testMethodName[1], $matches);
+
+        // Prepend all numbers with a space
+        $replaced = preg_replace('/(\d+)/', ' $1', $matches[0]);
+
+        $testNameArray = array_map('strtolower', $replaced);
 
         $name = implode(' ', $testNameArray);
 
+        // check if prefix is test remove it
+        $name = preg_replace('/^test /', '', $name, 1);
+
         // Get the data set name
         $name = $this->handleDataSetName($name, $testMethodName[1]);
 

tests/Output.php

@@ -45,4 +45,14 @@ public function test should convert non breaking spaces to lowercased wo
     {
         $this->assertTrue(true);
     }
+
+    public function testCanContain1Or99Numbers()
+    {
+        $this->assertTrue(true);
+    }
+
+    public function test123CanStartOrEndWithNumbers456()
+    {
+        $this->assertTrue(true);
+    }
 }

tests/PrinterTest.php

@@ -60,6 +60,20 @@ public function testTestNameCanBeNonBreakingSpaced()
         $this->assertStringContainsString('✓ should convert non breaking spaces to lowercased words', $lines[11]);
     }
 
+    public function testTestNameCanContainNumbers()
+    {
+        $lines = $this->getOutput();
+
+        $this->assertStringContainsString('✓ can contain 1 or 99 numbers', $lines[12]);
+    }
+
+    public function testTestNameCanStartOrEndWithANumber()
+    {
+        $lines = $this->getOutput();
+
+        $this->assertStringContainsString('✓ 123 can start or end with numbers 456', $lines[13]);
+    }
+
     private function getOutput(): array
     {
         $command = [