fix(java-junit): stack trace line can start with whitespaces

Fixes #208
dorny · Nov 19, 2022 · 3a48f6e · 3a48f6e
1 parent e5edb61
commit 3a48f6e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/__tests__/java-stack-trace-element-parser.test.ts b/__tests__/java-stack-trace-element-parser.test.ts
@@ -25,6 +25,15 @@ describe('parseStackTraceLine tests', () => {
         lineStr: '29'
       })
     })
+
+    it('starts with whitespaces', async () => {
+      const line = " \tat org.apache.pulsar.AddMissingPatchVersionTest.testVersionStrings(AddMissingPatchVersionTest.java:29)"
+      expect(parseStackTraceElement(line)).toEqual({
+        tracePath: "org.apache.pulsar.AddMissingPatchVersionTest.testVersionStrings",
+        fileName: "AddMissingPatchVersionTest.java",
+        lineStr: "29"
+      })
+    })
   })
 
   describe('Kotlin class', () => {

diff --git a/src/parsers/java-junit/java-stack-trace-element-parser.ts b/src/parsers/java-junit/java-stack-trace-element-parser.ts
@@ -6,7 +6,7 @@ export interface StackTraceElement {
 
 // simple format:
 // at <FULLY_QUALIFIED_METHOD_NAME>(<FILE_NAME>:<LINE_NUMBER>)
-const re = /^at (.*)\((.*):(\d+)\)$/
+const re = /^\s*at (.*)\((.*):(\d+)\)$/
 
 export function parseStackTraceElement(stackTraceLine: string): StackTraceElement | undefined {
   const match = stackTraceLine.match(re)