Skip to content

Commit

Permalink
tools: remove hyphen in TAP result
Browse files Browse the repository at this point in the history
As it is, the TAP result shows an extra hyphen in front of test names.
Sample:
ci.nodejs.org/job/node-test-commit-osx/nodes=osx1010/454/tapResults/
This patch removes the extra hyphen.

PR-URL: #2718
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
  • Loading branch information
thefourtheye committed Sep 12, 2015
1 parent 8e14135 commit de05175
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tools/test.py
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,7 @@ def HasRun(self, output):
self._done += 1
command = basename(output.command[-1])
if output.UnexpectedOutput():
status_line = 'not ok %i - %s' % (self._done, command)
status_line = 'not ok %i %s' % (self._done, command)
if FLAKY in output.test.outcomes and self.flaky_tests_mode == DONTCARE:
status_line = status_line + ' # TODO : Fix flaky test'
logger.info(status_line)
Expand All @@ -271,9 +271,9 @@ def HasRun(self, output):
skip = skip_regex.search(output.output.stdout)
if skip:
logger.info(
'ok %i - %s # skip %s' % (self._done, command, skip.group(1)))
'ok %i %s # skip %s' % (self._done, command, skip.group(1)))
else:
status_line = 'ok %i - %s' % (self._done, command)
status_line = 'ok %i %s' % (self._done, command)
if FLAKY in output.test.outcomes:
status_line = status_line + ' # TODO : Fix flaky test'
logger.info(status_line)
Expand Down

0 comments on commit de05175

Please sign in to comment.