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

Issue 2683403003: Improve error message in flaky test (Closed)

Created:
3 years, 10 months ago by Siggi Cherem (dart-lang)
Modified:
3 years, 7 months ago
Reviewers:
Bill Hesse
CC:
reviews_dartlang.org, nweiz
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Improve error message in flaky test This test is flaky (see #28712), but there are many test cases that hit this line. This should help narrow things down when this fails in minified mode (where it is hard to extract what the stack trace looks like). R=whesse@google.com Committed: https://github.com/dart-lang/sdk/commit/5615f72247855a63c27b1f42695e3b2626cade6e

Patch Set 1 #

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

Messages

Total messages: 5 (2 generated)
Siggi Cherem (dart-lang)
3 years, 10 months ago (2017-02-10 22:34:40 UTC) #2
Bill Hesse
lgtm
3 years, 10 months ago (2017-02-10 22:52:48 UTC) #3
Siggi Cherem (dart-lang)
3 years, 7 months ago (2017-04-28 22:35:03 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
5615f72247855a63c27b1f42695e3b2626cade6e (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698