add line number to TestParseException exception message (#4446)

Codeception · Aug 24, 2017 · 215bab0 · 215bab0
1 parent c59f541
commit 215bab0
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/src/Codeception/Exception/TestParseException.php b/src/Codeception/Exception/TestParseException.php
@@ -3,9 +3,13 @@
 
 class TestParseException extends \Exception
 {
-    public function __construct($fileName, $errors = null)
+    public function __construct($fileName, $errors = null, $line = null)
     {
-        $this->message = "Couldn't parse test '$fileName'";
+        if ($line) {
+            $this->message = "Couldn't parse test '$fileName' on line $line";
+        } else {
+            $this->message = "Couldn't parse test '$fileName'";
+        }
         if ($errors) {
             $this->message .= "\n$errors";
         }

diff --git a/src/Codeception/Lib/Parser.php b/src/Codeception/Lib/Parser.php
@@ -125,7 +125,7 @@ public static function load($file)
         try {
             self::includeFile($file);
         } catch (\ParseError $e) {
-            throw new TestParseException($file, $e->getMessage());
+            throw new TestParseException($file, $e->getMessage(), $e->getLine());
         } catch (\Exception $e) {
             // file is valid otherwise
         }

diff --git a/src/Codeception/Test/Cept.php b/src/Codeception/Test/Cept.php
@@ -41,7 +41,7 @@ public function test()
         try {
             require $testFile;
         } catch (\ParseError $e) {
-            throw new TestParseException($testFile, $e->getMessage());
+            throw new TestParseException($testFile, $e->getMessage(), $e->getLine());
         }
     }