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

Issue 11777003: Make the "pub" and "pkg" test suites run Dart from the built SDK. (Closed)

Created:
7 years, 11 months ago by Bob Nystrom
Modified:
7 years, 11 months ago
Reviewers:
ricow1, Emily Fortuna
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Make the "pub" and "pkg" test suites run Dart from the built SDK. I'm not using a test.dart command-line argument for this because I want to enforce that they *always* run out of the built SDK directory. This will be needed in a later patch so that the pub test code can find the SDK directory easily relative to the currently running Dart executable. Committed: https://code.google.com/p/dart/source/detail?r=16652

Patch Set 1 #

Total comments: 2

Patch Set 2 : Add comment. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -19 lines) Patch
M tools/testing/dart/test_suite.dart View 1 11 chunks +32 lines, -19 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Bob Nystrom
This is a step along the path to get everything in "pkg" using "package:" imports. ...
7 years, 11 months ago (2013-01-04 20:56:00 UTC) #1
Emily Fortuna
lgtm, with one comment added, please. https://codereview.chromium.org/11777003/diff/1/tools/testing/dart/test_suite.dart File tools/testing/dart/test_suite.dart (right): https://codereview.chromium.org/11777003/diff/1/tools/testing/dart/test_suite.dart#newcode110 tools/testing/dart/test_suite.dart:110: bool get useSdk ...
7 years, 11 months ago (2013-01-04 21:39:10 UTC) #2
Bob Nystrom
7 years, 11 months ago (2013-01-04 22:29:45 UTC) #3
Thanks!

https://codereview.chromium.org/11777003/diff/1/tools/testing/dart/test_suite...
File tools/testing/dart/test_suite.dart (right):

https://codereview.chromium.org/11777003/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:110: bool get useSdk {
On 2013/01/04 21:39:10, Emily Fortuna wrote:
> can we add a little TODO here that says that we plan to switch all tests to
run
> with the SDK eventually, so this code should go away? 

Done.

Powered by Google App Engine
This is Rietveld 408576698