Add some more tests around exception messages

Author: ciaranmcnulty

php/tests/unit/ParserException/CompositeParserExceptionTest.php

@@ -4,11 +4,26 @@
 
 namespace Cucumber\Gherkin\ParserException;
 
+use Cucumber\Gherkin\Location;
 use Cucumber\Gherkin\ParserException;
 use PHPUnit\Framework\TestCase;
 
 final class CompositeParserExceptionTest extends TestCase
 {
+    public function testMessageContainsAHeaderAndALineForEachException(): void
+    {
+        $compositeParserException = new CompositeParserException([
+            new class ('Message 1', null) extends ParserException {},
+            new class ('Message 2', null) extends ParserException {},
+        ]);
+
+        $message = $compositeParserException->getMessage();
+
+        self::assertStringStartsWith("(-1:-1): Parser errors:\n", $message);
+        self::assertCount(3, explode("\n", $message));
+    }
+
+
     public function testMessageContainsAllTheChildMessages(): void
     {
         $compositeParserException = new CompositeParserException([
@@ -21,4 +36,17 @@ public function testMessageContainsAllTheChildMessages(): void
         self::assertStringContainsString('Message 1', $message);
         self::assertStringContainsString('Message 2', $message);
     }
+
+    public function testMessageContainsAllTheChildLocations(): void
+    {
+        $compositeParserException = new CompositeParserException([
+            new class ('', new Location(100, 200)) extends ParserException {},
+            new class ('', new Location(300, 400)) extends ParserException {},
+        ]);
+
+        $message = $compositeParserException->getMessage();
+
+        self::assertStringContainsString('(100:200)', $message);
+        self::assertStringContainsString('(300:400)', $message);
+    }
 }

php/tests/unit/ParserException/UnexpectedTokenExceptionTest.php

@@ -11,10 +11,10 @@
 
 final class UnexpectedTokenExceptionTest extends TestCase
 {
-    public function testMessageSpecifiesWhichTokenWasUnexpected(): void
+    public function testMessageSpecifiesAndTrimsWhichTokenWasUnexpected(): void
     {
         $line = $this->createMock(GherkinLine::class);
-        $line->method('getLineText')->willReturn('Baz');
+        $line->method('getLineText')->willReturn('  Baz  ');
 
         $exception = new UnexpectedTokenException(
             new Token($line, new Location(10, 10)),