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

Issue 10968040: Eliminate some of the flakiness in testing: (Closed)

Created:
8 years, 3 months ago by gram
Modified:
8 years, 3 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Eliminate some of the flakiness in testing: - errors in test_controller.js get posted to the Dart configs - Dart configs will receive these and set uncaughtError - Selenium driver will output the full HTML body, not just up to the first script tag (else the errors above could be lost). Committed: https://code.google.com/p/dart/source/detail?r=12735

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -10 lines) Patch
M pkg/unittest/html_config.dart View 2 chunks +20 lines, -4 lines 0 comments Download
M pkg/unittest/html_enhanced_config.dart View 2 chunks +14 lines, -4 lines 0 comments Download
M pkg/unittest/test_controller.js View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/unittest/unittest.dart View 1 chunk +3 lines, -1 line 0 comments Download
M tools/testing/run_selenium.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
gram
8 years, 3 months ago (2012-09-21 20:01:14 UTC) #1
Emily Fortuna
lgtm. thanks for catching this.
8 years, 3 months ago (2012-09-21 20:18:49 UTC) #2
Siggi Cherem (dart-lang)
lgtm
8 years, 3 months ago (2012-09-21 20:20:39 UTC) #3
ahe
8 years, 3 months ago (2012-09-22 07:39:46 UTC) #4
LGTM!

Powered by Google App Engine
This is Rietveld 408576698