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

Side by Side Diff: lib/src/util/dart.dart

Issue 1056733002: Run test tearDowns and clean up temporary directories when a signal is caught. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: Code review changes Created 5 years, 8 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 | « lib/src/runner/vm/isolate_test.dart ('k') | lib/src/util/io.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 test.util.dart; 5 library test.util.dart;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 import 'dart:io'; 8 import 'dart:io';
9 import 'dart:isolate'; 9 import 'dart:isolate';
10 10
(...skipping 10 matching lines...) Expand all
21 /// 21 ///
22 /// [code] should be the contents of a Dart entrypoint. It may contain imports; 22 /// [code] should be the contents of a Dart entrypoint. It may contain imports;
23 /// they will be resolved in the same context as the host isolate. [message] is 23 /// they will be resolved in the same context as the host isolate. [message] is
24 /// passed to the [main] method of the code being run; the caller is responsible 24 /// passed to the [main] method of the code being run; the caller is responsible
25 /// for using this to establish communication with the isolate. 25 /// for using this to establish communication with the isolate.
26 /// 26 ///
27 /// [packageRoot] controls the package root of the isolate. It may be either a 27 /// [packageRoot] controls the package root of the isolate. It may be either a
28 /// [String] or a [Uri]. 28 /// [String] or a [Uri].
29 Future<Isolate> runInIsolate(String code, message, {packageRoot}) { 29 Future<Isolate> runInIsolate(String code, message, {packageRoot}) {
30 // TODO(nweiz): load code from a local server rather than from a file. 30 // TODO(nweiz): load code from a local server rather than from a file.
31 var dir = Directory.systemTemp.createTempSync().path; 31 var dir = createTempDir();
32 var dartPath = p.join(dir, 'runInIsolate.dart'); 32 var dartPath = p.join(dir, 'runInIsolate.dart');
33 new File(dartPath).writeAsStringSync(code); 33 new File(dartPath).writeAsStringSync(code);
34 var port = new ReceivePort(); 34 var port = new ReceivePort();
35 return Isolate.spawn(_isolateBuffer, { 35 return Isolate.spawn(_isolateBuffer, {
36 'replyTo': port.sendPort, 36 'replyTo': port.sendPort,
37 'uri': p.toUri(dartPath).toString(), 37 'uri': p.toUri(dartPath).toString(),
38 'packageRoot': packageRoot == null ? null : packageRoot.toString(), 38 'packageRoot': packageRoot == null ? null : packageRoot.toString(),
39 'message': message 39 'message': message
40 }).then((isolate) { 40 }).then((isolate) {
41 return port.first.then((response) { 41 return port.first.then((response) {
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
110 } 110 }
111 111
112 var start = contextRunes.offset; 112 var start = contextRunes.offset;
113 for (var spanRune in span.text.runes) { 113 for (var spanRune in span.text.runes) {
114 if (spanRune != contextRunes.current) return null; 114 if (spanRune != contextRunes.current) return null;
115 contextRunes.moveNext(); 115 contextRunes.moveNext();
116 } 116 }
117 117
118 return file.span(start, contextRunes.offset); 118 return file.span(start, contextRunes.offset);
119 } 119 }
OLDNEW
« no previous file with comments | « lib/src/runner/vm/isolate_test.dart ('k') | lib/src/util/io.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698