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

Issue 12787011: - Make test code fail when run from unexpected location. (Closed)

Created:
7 years, 9 months ago by Ivan Posva
Modified:
7 years, 9 months ago
Reviewers:
Alan Knight, kustermann
CC:
reviews_dartlang.org
Visibility:
Public.

Description

- Make test code fail when run from unexpected location. Committed: https://code.google.com/p/dart/source/detail?r=20122

Patch Set 1 #

Total comments: 6

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -1 line) Patch
M intl/test/data_directory.dart View 1 2 chunks +10 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
Ivan Posva
7 years, 9 months ago (2013-03-15 22:01:53 UTC) #1
kustermann
lgtm https://codereview.chromium.org/12787011/diff/1/intl/test/data_directory.dart File intl/test/data_directory.dart (right): https://codereview.chromium.org/12787011/diff/1/intl/test/data_directory.dart#newcode25 intl/test/data_directory.dart:25: if (new Directory('${dir.path}${_sep}pkg').existsSync()) { Use current instead of ...
7 years, 9 months ago (2013-03-15 22:21:36 UTC) #2
Alan Knight
https://codereview.chromium.org/12787011/diff/1/intl/test/data_directory.dart File intl/test/data_directory.dart (right): https://codereview.chromium.org/12787011/diff/1/intl/test/data_directory.dart#newcode3 intl/test/data_directory.dart:3: // BSD-style license that can be found in the ...
7 years, 9 months ago (2013-03-15 22:24:50 UTC) #3
Ivan Posva
https://codereview.chromium.org/12787011/diff/1/intl/test/data_directory.dart File intl/test/data_directory.dart (right): https://codereview.chromium.org/12787011/diff/1/intl/test/data_directory.dart#newcode25 intl/test/data_directory.dart:25: if (new Directory('${dir.path}${_sep}pkg').existsSync()) { On 2013/03/15 22:21:36, kustermann wrote: ...
7 years, 9 months ago (2013-03-15 22:25:57 UTC) #4
Ivan Posva
https://codereview.chromium.org/12787011/diff/1/intl/test/data_directory.dart File intl/test/data_directory.dart (right): https://codereview.chromium.org/12787011/diff/1/intl/test/data_directory.dart#newcode3 intl/test/data_directory.dart:3: // BSD-style license that can be found in the ...
7 years, 9 months ago (2013-03-15 22:47:36 UTC) #5
Ivan Posva
Resolved conflicts. -Ivan
7 years, 9 months ago (2013-03-15 23:09:01 UTC) #6
Ivan Posva
7 years, 9 months ago (2013-03-15 23:14:13 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 manually as r20122 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698