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

Issue 10868059: Add a number of convenience methods to test_pub. (Closed)

Created:
8 years, 4 months ago by nweiz
Modified:
8 years, 4 months ago
Reviewers:
Bob Nystrom
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Add a number of convenience methods to test_pub. This changelist has two goals. The first is to simply clean up pub_test by factoring out repeated patterns, removing the use of raw YAML, and making the commands more semantic. The second goal is to set the stage for the use of ephemeral ports for the mock package server, which will allow multiple instances of the tests to be run simultaneously. This was originally submitted as r11261, but that caused test errors due to a race condition that were then fixed. Then it was submitted as r11266, which caused further test errors that are now completely fixed. Committed: https://code.google.com/p/dart/source/detail?r=11273

Patch Set 1 #

Patch Set 2 : Fix some checked-mode issues #

Unified diffs Side-by-side diffs Delta from patch set Stats (+393 lines, -832 lines) Patch
M utils/tests/pub/pub_test.dart View 26 chunks +177 lines, -800 lines 0 comments Download
M utils/tests/pub/test_pub.dart View 1 6 chunks +216 lines, -32 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
nweiz
Let's try this again. Once again, the main difference is in patch set 1.
8 years, 4 months ago (2012-08-23 22:55:32 UTC) #1
Bob Nystrom
8 years, 4 months ago (2012-08-23 23:02:07 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698