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

Side by Side Diff: pkg/intl/test/message_extraction/message_extraction_test.dart

Issue 15670004: Remove (unused) ProcessOptions in test. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 6 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 | no next file » | 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) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, 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 message_extraction_test; 5 library message_extraction_test;
6 6
7 import 'package:unittest/unittest.dart'; 7 import 'package:unittest/unittest.dart';
8 import 'dart:io'; 8 import 'dart:io';
9 import 'dart:async'; 9 import 'dart:async';
10 import 'package:pathos/path.dart' as path; 10 import 'package:pathos/path.dart' as path;
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 print("exitCode=${previousResult.exitCode}"); 96 print("exitCode=${previousResult.exitCode}");
97 } 97 }
98 var filesInTheRightDirectory = filenames.map((x) => dir(x)).toList(); 98 var filesInTheRightDirectory = filenames.map((x) => dir(x)).toList();
99 // Inject the script argument --output-dir in between the script and its 99 // Inject the script argument --output-dir in between the script and its
100 // arguments. 100 // arguments.
101 var args = [] 101 var args = []
102 ..addAll(vmArgs) 102 ..addAll(vmArgs)
103 ..add(filesInTheRightDirectory.first) 103 ..add(filesInTheRightDirectory.first)
104 ..addAll(["--output-dir=${dir()}"]) 104 ..addAll(["--output-dir=${dir()}"])
105 ..addAll(filesInTheRightDirectory.skip(1)); 105 ..addAll(filesInTheRightDirectory.skip(1));
106 var options = new ProcessOptions()
107 ..stdoutEncoding=Encoding.UTF_8
108 ..stderrEncoding=Encoding.UTF_8;
109 var result = Process.run(dart, args); 106 var result = Process.run(dart, args);
110 return result; 107 return result;
111 } 108 }
112 109
113 Future<ProcessResult> extractMessages(ProcessResult previousResult) => run( 110 Future<ProcessResult> extractMessages(ProcessResult previousResult) => run(
114 previousResult, 111 previousResult,
115 ['extract_to_json.dart', '--suppress-warnings', 'sample_with_messages.dart', 112 ['extract_to_json.dart', '--suppress-warnings', 'sample_with_messages.dart',
116 'part_of_sample_with_messages.dart']); 113 'part_of_sample_with_messages.dart']);
117 114
118 Future<ProcessResult> generateTranslationFiles(ProcessResult previousResult) => 115 Future<ProcessResult> generateTranslationFiles(ProcessResult previousResult) =>
(...skipping 99 matching lines...) Expand 10 before | Expand all | Expand 10 after
218 verify("Dies ergibt sich aus einer Methode"); 215 verify("Dies ergibt sich aus einer Methode");
219 verify("Diese Methode ist nicht eine Lambda"); 216 verify("Diese Methode ist nicht eine Lambda");
220 verify("Dies ergibt sich aus einer statischen Methode"); 217 verify("Dies ergibt sich aus einer statischen Methode");
221 verify("This is missing some translations"); 218 verify("This is missing some translations");
222 verify("Antike griechische Galgenmännchen Zeichen: 𐅆𐅇"); 219 verify("Antike griechische Galgenmännchen Zeichen: 𐅆𐅇");
223 // verify("Die Sache ist, well"); 220 // verify("Die Sache ist, well");
224 // expect("Einer der knifflige Dinge ist der Plural"); 221 // expect("Einer der knifflige Dinge ist der Plural");
225 // expect("Zu den kniffligen Dinge Pluralformen"); 222 // expect("Zu den kniffligen Dinge Pluralformen");
226 verify("Escapes: "); 223 verify("Escapes: ");
227 verify("\r\f\b\t\v."); 224 verify("\r\f\b\t\v.");
228 } 225 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698