diff options
| author | Sergey Avseyev <sergey.avseyev@gmail.com> | 2018-01-01 02:39:55 +0300 |
|---|---|---|
| committer | Andreas Rumpf <rumpf_a@web.de> | 2018-01-01 00:39:55 +0100 |
| commit | 3f2636c76580c8a086df4255a17b077622434d7d (patch) | |
| tree | 21720e4a0803677898d8790a0ae9a8fc45307e67 | |
| parent | c87d61771835bad9acc91c64c0aefa27104731cf (diff) | |
| download | Nim-3f2636c76580c8a086df4255a17b077622434d7d.tar.gz | |
Remove obsolete method analyzeAndConsolidateOutput (#6998)
After the change, when stacktraces rendered in reversed order (most recent call first), this method removed all stacktraces in the test failures.
| -rw-r--r-- | tests/testament/tester.nim | 17 |
1 files changed, 1 insertions, 16 deletions
diff --git a/tests/testament/tester.nim b/tests/testament/tester.nim index 69b640fa2..870f9f865 100644 --- a/tests/testament/tester.nim +++ b/tests/testament/tester.nim @@ -290,20 +290,6 @@ proc compilerOutputTests(test: TTest, target: TTarget, given: var TSpec, if given.err == reSuccess: inc(r.passed) r.addResult(test, target, expectedmsg, givenmsg, given.err) -proc analyzeAndConsolidateOutput(s: string): string = - result = "" - let rows = s.splitLines - for i in 0 ..< rows.len: - if (let pos = find(rows[i], "Traceback (most recent call last)"); pos != -1): - result = substr(rows[i], pos) & "\n" - for i in i+1 ..< rows.len: - result.add rows[i] & "\n" - if not (rows[i] =~ peg"['(']+ '(' \d+ ')' \s+"): - return - elif (let pos = find(rows[i], "SIGSEGV: Illegal storage access."); pos != -1): - result = substr(rows[i], pos) - return - proc testSpec(r: var TResults, test: TTest, target = targetC) = let tname = test.name.addFileExt(".nim") #echo "TESTING ", tname @@ -376,8 +362,7 @@ proc testSpec(r: var TResults, test: TTest, target = targetC) = if exitCode != expected.exitCode: r.addResult(test, target, "exitcode: " & $expected.exitCode, "exitcode: " & $exitCode & "\n\nOutput:\n" & - analyzeAndConsolidateOutput(bufB), - reExitCodesDiffer) + bufB, reExitCodesDiffer) continue if bufB != expectedOut and expected.action != actionRunNoSpec: |