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

Side by Side Diff: test/io.dart

Issue 934413002: Replace the existing unittest APIs with the new runner infrastructure. (Closed) Base URL: git@github.com:dart-lang/unittest@master
Patch Set: Code review changes Created 5 years, 10 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
« no previous file with comments | « test/invalid_ops_test.dart ('k') | test/late_exception_test.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, 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 unittest.test.io; 5 library unittest.test.io;
6 6
7 import 'dart:io'; 7 import 'dart:io';
8 import 'dart:mirrors'; 8 import 'dart:mirrors';
9 9
10 import 'package:path/path.dart' as p; 10 import 'package:path/path.dart' as p;
11 import 'package:stack_trace/stack_trace.dart'; 11 import 'package:stack_trace/stack_trace.dart';
12 12
13 /// The root directory of the `unittest` package. 13 /// The root directory of the `unittest` package.
14 final String packageDir = _computePackageDir(); 14 final String packageDir = _computePackageDir();
15 String _computePackageDir() => 15 String _computePackageDir() =>
16 p.dirname(p.dirname(_libraryPath(#unittest.test.io))); 16 p.dirname(p.dirname(_libraryPath(#unittest.test.io)));
17 17
18 /// Runs the unittest executable with the package root set properly. 18 /// Runs the unittest executable with the package root set properly.
19 ProcessResult runUnittest(List<String> args, {String workingDirectory}) { 19 ProcessResult runUnittest(List<String> args, {String workingDirectory}) {
20 var allArgs = Platform.executableArguments.toList() 20 var allArgs = [
21 ..add(p.join(packageDir, 'bin/unittest.dart')) 21 p.join(packageDir, 'bin/unittest.dart'),
22 ..add("--package-root=${p.join(packageDir, 'packages')}") 22 "--package-root=${p.join(packageDir, 'packages')}"
23 ..addAll(args); 23 ]..addAll(args);
24
25 // TODO(nweiz): Use ScheduledProcess once it's compatible.
26 return runDart(allArgs, workingDirectory: workingDirectory);
27 }
28
29 /// Runs Dart.
30 ProcessResult runDart(List<String> args, {String workingDirectory}) {
31 var allArgs = Platform.executableArguments.toList()..addAll(args);
24 32
25 // TODO(nweiz): Use ScheduledProcess once it's compatible. 33 // TODO(nweiz): Use ScheduledProcess once it's compatible.
26 return Process.runSync(Platform.executable, allArgs, 34 return Process.runSync(Platform.executable, allArgs,
27 workingDirectory: workingDirectory); 35 workingDirectory: workingDirectory);
28 } 36 }
29 37
30 /// Returns the path to the library named [libraryName]. 38 /// Returns the path to the library named [libraryName].
31 /// 39 ///
32 /// The library name must be globally unique, or the wrong library path may be 40 /// The library name must be globally unique, or the wrong library path may be
33 /// returned. 41 /// returned.
34 String _libraryPath(Symbol libraryName) { 42 String _libraryPath(Symbol libraryName) {
35 var lib = currentMirrorSystem().findLibrary(libraryName); 43 var lib = currentMirrorSystem().findLibrary(libraryName);
36 return p.fromUri(lib.uri); 44 return p.fromUri(lib.uri);
37 } 45 }
38 46
39 47
OLDNEW
« no previous file with comments | « test/invalid_ops_test.dart ('k') | test/late_exception_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698