Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Issue 26654006: Emits additional output when test_runner_test fails. (Closed)

Created:
7 years, 2 months ago by zra
Modified:
7 years, 2 months ago
Reviewers:
regis, floitsch, kustermann
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Emits additional output when test_runner_test fails. On SIMARM and SIMMIPS, test_runner_test is flaky. Further, we have been unable to reproduce the failure case locally. The test fails when the "Unexpected fail" exception is thrown in processCompletedTest. This change prints the stdout and stderr of the completed test. With this output in the flakylogs, we'll hopefully be able to diagnose the problem. R=regis@google.com Committed: https://code.google.com/p/dart/source/detail?r=28420

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -1 line) Patch
M tests/standalone/io/test_runner_test.dart View 1 chunk +13 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
zra
7 years, 2 months ago (2013-10-09 21:12:49 UTC) #1
zra
7 years, 2 months ago (2013-10-09 21:23:40 UTC) #2
regis
LGTM
7 years, 2 months ago (2013-10-09 21:25:54 UTC) #3
zra
7 years, 2 months ago (2013-10-09 21:26:28 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r28420 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698