Chromium Code Reviews

Issue 1216163002: Fix tests to run in dart-lang/sdk. (Closed)

Created:
5 years, 5 months ago by nweiz
Modified:
5 years, 5 months ago
Reviewers:
Bob Nystrom
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/pub.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix tests to run in dart-lang/sdk. Now that pub uses the new test runner, some of its logic for running in the Dart SDK repository needs to be updated. R=rnystrom@google.com Committed: https://github.com/dart-lang/pub/commit/6ea9840747befeb495fc02ca3492be2032bdfbe3

Patch Set 1 #

Total comments: 4

Patch Set 2 : cr #

Unified diffs Side-by-side diffs Stats (+24 lines, -5 lines)
M lib/src/io.dart View 2 chunks +19 lines, -3 lines 0 comments
M test/test_pub.dart View 1 chunk +5 lines, -2 lines 0 comments

Messages

Total messages: 5 (0 generated)
nweiz
5 years, 5 months ago (2015-06-30 01:22:59 UTC) #1
Bob Nystrom
lgtm https://codereview.chromium.org/1216163002/diff/1/lib/src/io.dart File lib/src/io.dart (right): https://codereview.chromium.org/1216163002/diff/1/lib/src/io.dart#newcode524 lib/src/io.dart:524: // the working directory is "is ..." https://codereview.chromium.org/1216163002/diff/1/lib/src/io.dart#newcode585 ...
5 years, 5 months ago (2015-06-30 18:33:11 UTC) #2
nweiz
https://codereview.chromium.org/1216163002/diff/1/lib/src/io.dart File lib/src/io.dart (right): https://codereview.chromium.org/1216163002/diff/1/lib/src/io.dart#newcode524 lib/src/io.dart:524: // the working directory is On 2015/06/30 18:33:11, Bob ...
5 years, 5 months ago (2015-06-30 19:47:30 UTC) #3
nweiz
Committed patchset #2 (id:20001) manually as 6ea9840747befeb495fc02ca3492be2032bdfbe3 (presubmit successful).
5 years, 5 months ago (2015-06-30 19:48:01 UTC) #4
Bob Nystrom
5 years, 5 months ago (2015-06-30 20:11:10 UTC) #5
Message was sent while issue was closed.
On 2015/06/30 19:47:30, nweiz wrote:
> https://codereview.chromium.org/1216163002/diff/1/lib/src/io.dart
> File lib/src/io.dart (right):
> 
> https://codereview.chromium.org/1216163002/diff/1/lib/src/io.dart#newcode524
> lib/src/io.dart:524: // the working directory is
> On 2015/06/30 18:33:11, Bob Nystrom wrote:
> > "is ..."
> 
> Done.
> 
> https://codereview.chromium.org/1216163002/diff/1/lib/src/io.dart#newcode585
> lib/src/io.dart:585: return
> path.dirname(path.dirname(path.dirname(path.current)));
> On 2015/06/30 18:33:11, Bob Nystrom wrote:
> > Is there a reason to prefer chained dirnames over path.join(path.current,
> > "../../../");?
> 
> Because path.join tries to avoid any unnecessary modification of the inputs,
it
> will return "${path.current}/../../..", which is uglier than actually moving
up
> the directories. Since this path might end up somewhere user-visible, it seems
> better to keep it nice-looking.

Ah, that makes sense. Sometimes, if the chain of dirnames is too long, I'll just
do:

path.normalize(path.join(path.current, "../../../"));

Powered by Google App Engine