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

Side by Side Diff: samples/tests/dartc/test_config.dart

Issue 10584014: Change test scripts to use Path library in most places, instead of strings. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Take out changes to Path as separate CL. Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | tests/co19/test_config.dart » ('j') | tools/testing/dart/test_suite.dart » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #library("client_dartc_test_config"); 5 #library("client_dartc_test_config");
6 6
7 #import("dart:io");
7 #import("../../../tools/testing/dart/test_suite.dart"); 8 #import("../../../tools/testing/dart/test_suite.dart");
8 9
9 class SamplesDartcTestSuite extends DartcCompilationTestSuite { 10 class SamplesDartcTestSuite extends DartcCompilationTestSuite {
10 SamplesDartcTestSuite(Map configuration) 11 SamplesDartcTestSuite(Map configuration)
11 : super(configuration, 12 : super(configuration,
12 'dartc', 13 'dartc',
13 'samples', 14 'samples',
14 [ 'actors', 15 [ 'actors',
15 'belay', 16 'belay',
16 'calculator', 17 'calculator',
(...skipping 18 matching lines...) Expand all
35 'time', 36 'time',
36 'ui_lib', 37 'ui_lib',
37 ], 38 ],
38 ['samples/tests/dartc/dartc.status']); 39 ['samples/tests/dartc/dartc.status']);
39 40
40 bool isTestFile(String filename) { 41 bool isTestFile(String filename) {
41 if (!filename.endsWith(".dart")) return false; 42 if (!filename.endsWith(".dart")) return false;
42 // Using readOptionsFromFile here causes the file to be read twice, 43 // Using readOptionsFromFile here causes the file to be read twice,
43 // because readOptionsFromFile is called again in the superclass. 44 // because readOptionsFromFile is called again in the superclass.
44 // Avoid this in new code. 45 // Avoid this in new code.
45 return readOptionsFromFile(filename)["containsLeadingHash"]; 46 return readOptionsFromFile(new Path.fromNative(filename))
47 ["containsLeadingHash"];
46 } 48 }
47 49
48 bool listRecursively() => true; 50 bool listRecursively() => true;
49 } 51 }
OLDNEW
« no previous file with comments | « no previous file | tests/co19/test_config.dart » ('j') | tools/testing/dart/test_suite.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698