alter parser for tap.py output

Signed-off-by: shmck <[email protected]>

Author: ShMcK

Diff for: src/services/testRunner/parser.test.ts

@@ -225,13 +225,13 @@ not ok 1 test_add_no_numbers (tests.math_test.MathTest)
 # AssertionError: 42 != 0 : Should return 0 with no params
 1..1`
       expect(parser(example)).toEqual({
-        ok: true,
+        ok: false,
         passed: [],
         failed: [
           {
             message: 'add no numbers',
             details:
-              'Traceback (most recent call last):\n   Fail Message\nAssertionError: 42 != 0 : Should return 0 with no params',
+              'Traceback (most recent call last):\nFail Message\nAssertionError: 42 != 0 : Should return 0 with no params',
           },
         ],
         logs: [],
@@ -251,13 +251,13 @@ not ok 2 test_add_one_number (tests.math_test.MathTest)
 1..2
 `
       expect(parser(example)).toEqual({
-        ok: true,
+        ok: false,
         passed: [{ message: 'add no numbers' }],
         failed: [
           {
             message: 'add one number',
             details:
-              'Traceback (most recent call last):\n   Fail Message\nAssertionError: 2 != 1 : Should add one number to 0',
+              'Traceback (most recent call last):\nFail Message\nAssertionError: 2 != 1 : Should add one number to 0',
           },
         ],
         logs: [],

Diff for: src/services/testRunner/parser.ts

@@ -23,7 +23,7 @@ const r = {
   start: /^(not ok)|(ok)/,
   fail: /^not ok (?<index>\d+)\s(\-\s)?(?<message>.+)$/,
   pass: /^ok (?<index>\d+)\s(\-\s)?(?<message>.+)$/,
-  details: /^#\s{2}(?<message>.+)$/,
+  details: /^#\s{1,2}(?<message>.+)$/,
   ignore: /^(1\.\.[0-9]+)|(#\s+(tests|pass|fail|skip)\s+[0-9]+)$/,
 }
 
@@ -75,7 +75,7 @@ const parser = (text: string): ParserOutput => {
   }
 
   for (const line of lines) {
-    if (!line.length) {
+    if (!line.length || !!r.ignore.exec(line)) {
       continue
     }
     // be optimistic! check for success first
@@ -121,11 +121,9 @@ const parser = (text: string): ParserOutput => {
       continue
     }
 
-    if (!r.ignore.exec(line)) {
-      // must be a log, associate with the next test
-      logs.push(line)
-      result.logs.push(line)
-    }
+    // must be a log, associate with the next test
+    logs.push(line)
+    result.logs.push(line)
   }
   addCurrentDetails()
   return result