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

Unified Diff: tests/compiler/dart2js/dart2js_batch_test.dart

Issue 2345083003: dart2js: run dartfmt on tests (Closed)
Patch Set: revert another multipart test Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/compiler/dart2js/dart2js_batch2_test.dart ('k') | tests/compiler/dart2js/data/exit_code_helper.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/compiler/dart2js/dart2js_batch_test.dart
diff --git a/tests/compiler/dart2js/dart2js_batch_test.dart b/tests/compiler/dart2js/dart2js_batch_test.dart
index 55f8ca77390da5e16203f43f724d12e6b745d6c6..1d41bf0d4010a2d8896515b80f7d61c200ac0a73 100644
--- a/tests/compiler/dart2js/dart2js_batch_test.dart
+++ b/tests/compiler/dart2js/dart2js_batch_test.dart
@@ -16,8 +16,8 @@ var tmpDir;
copyDirectory(Directory sourceDir, Directory destinationDir) {
sourceDir.listSync().forEach((FileSystemEntity element) {
- String newPath = path.join(destinationDir.path,
- path.basename(element.path));
+ String newPath =
+ path.join(destinationDir.path, path.basename(element.path));
if (element is File) {
element.copySync(newPath);
} else if (element is Directory) {
@@ -72,26 +72,22 @@ Future runTests(Process process) {
process.stdin.close();
Future<String> output = process.stdout.transform(UTF8.decoder).join();
Future<String> errorOut = process.stderr.transform(UTF8.decoder).join();
- return Future.wait([output, errorOut])
- .then((result) {
- String stdoutOutput = result[0];
- String stderrOutput = result[1];
+ return Future.wait([output, errorOut]).then((result) {
+ String stdoutOutput = result[0];
+ String stderrOutput = result[1];
- Expect.equals(4, ">>> EOF STDERR".allMatches(stderrOutput).length);
- Expect.equals(4, ">>>".allMatches(stderrOutput).length);
+ Expect.equals(4, ">>> EOF STDERR".allMatches(stderrOutput).length);
+ Expect.equals(4, ">>>".allMatches(stderrOutput).length);
- Expect.equals(2, ">>> TEST OK".allMatches(stdoutOutput).length);
- Expect.equals(2, ">>> TEST FAIL".allMatches(stdoutOutput).length);
- Expect.equals(4, ">>>".allMatches(stdoutOutput).length);
- });
+ Expect.equals(2, ">>> TEST OK".allMatches(stdoutOutput).length);
+ Expect.equals(2, ">>> TEST FAIL".allMatches(stdoutOutput).length);
+ Expect.equals(4, ">>>".allMatches(stdoutOutput).length);
+ });
}
void main() {
var tmpDir;
asyncTest(() {
- return setup()
- .then(launchDart2Js)
- .then(runTests)
- .whenComplete(cleanUp);
+ return setup().then(launchDart2Js).then(runTests).whenComplete(cleanUp);
});
}
« no previous file with comments | « tests/compiler/dart2js/dart2js_batch2_test.dart ('k') | tests/compiler/dart2js/data/exit_code_helper.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698