Skip to content
Snippets Groups Projects
Commit 559543ec authored by David Klempner's avatar David Klempner
Browse files

Unconditionally add a newline if a test times out or fails.

Failure should generally result in output unless something goes wrong,
but timeout currently doesn't, but should be printed 100% of the time.
parent 73701364
No related branches found
No related tags found
No related merge requests found
...@@ -192,7 +192,7 @@ class Job(object): ...@@ -192,7 +192,7 @@ class Job(object):
self._tempfile.seek(0) self._tempfile.seek(0)
stdout = self._tempfile.read() stdout = self._tempfile.read()
message('FAILED', '%s [ret=%d]' % ( message('FAILED', '%s [ret=%d]' % (
self._spec.shortname, self._process.returncode), stdout) self._spec.shortname, self._process.returncode), stdout, do_newline=True)
else: else:
self._state = _SUCCESS self._state = _SUCCESS
message('PASSED', '%s [time=%.1fsec]' % (self._spec.shortname, elapsed), message('PASSED', '%s [time=%.1fsec]' % (self._spec.shortname, elapsed),
...@@ -200,7 +200,7 @@ class Job(object): ...@@ -200,7 +200,7 @@ class Job(object):
if self._bin_hash: if self._bin_hash:
update_cache.finished(self._spec.identity(), self._bin_hash) update_cache.finished(self._spec.identity(), self._bin_hash)
elif self._state == _RUNNING and time.time() - self._start > 300: elif self._state == _RUNNING and time.time() - self._start > 300:
message('TIMEOUT', self._spec.shortname, do_newline=self._travis) message('TIMEOUT', self._spec.shortname, do_newline=True)
self.kill() self.kill()
return self._state return self._state
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment