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

Unified Diff: tests/standalone/io/process_sync_test.dart

Issue 21816002: Add Process.runSync for running processe synchronously. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fixed another Windows issue Created 7 years, 4 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
« runtime/bin/process_win.cc ('K') | « tests/standalone/io/process_sync_script.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/standalone/io/process_sync_test.dart
diff --git a/tests/standalone/io/process_sync_test.dart b/tests/standalone/io/process_sync_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..04bc59f11cf1a32f2c59cd134f6eae39f8c8621e
--- /dev/null
+++ b/tests/standalone/io/process_sync_test.dart
@@ -0,0 +1,63 @@
+// Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+import "package:expect/expect.dart";
+import 'package:path/path.dart';
+import "dart:io";
+
+test(int blockCount,
+ int stdoutBlockSize,
+ int stderrBlockSize,
+ int exitCode,
+ [int nonWindowsExitCode]) {
+ // Get the Dart script file that generates output.
+ var scriptFile = new File(join(dirname(Platform.script),
+ "process_sync_script.dart"));
+ var args = [scriptFile.path,
+ blockCount.toString(),
+ stdoutBlockSize.toString(),
+ stderrBlockSize.toString(),
+ exitCode.toString()];
+ ProcessResult syncResult = Process.runSync(Platform.executable, args);
+ Expect.equals(blockCount * stdoutBlockSize, syncResult.stdout.length);
+ Expect.equals(blockCount * stderrBlockSize, syncResult.stderr.length);
+ if (Platform.isWindows) {
+ Expect.equals(exitCode, syncResult.exitCode);
+ } else {
+ if (nonWindowsExitCode == null) {
+ Expect.equals(exitCode, syncResult.exitCode);
+ } else {
+ Expect.equals(nonWindowsExitCode, syncResult.exitCode);
+ }
+ }
+ Process.run(Platform.executable, args).then((asyncResult) {
+ Expect.equals(syncResult.stdout, asyncResult.stdout);
+ Expect.equals(syncResult.stderr, asyncResult.stderr);
+ Expect.equals(syncResult.exitCode, asyncResult.exitCode);
+ });
+}
+
+main() {
+ test(10, 10, 10, 0);
+ test(10, 100, 10, 0);
+ test(10, 10, 100, 0);
+ test(100, 1, 10, 0);
+ test(100, 10, 1, 0);
+ test(100, 1, 1, 0);
+ test(1, 100000, 100000, 0);
+
+ // The buffer size used in process.h.
+ var kBufferSize = 16 * 1024;
+ test(1, kBufferSize, kBufferSize, 0);
+ test(1, kBufferSize - 1, kBufferSize + 1, 0);
+ test(kBufferSize - 1, 1, 1, 0);
+ test(kBufferSize, 1, 1, 0);
+ test(kBufferSize + 1, 1, 1, 0);
+
+ test(10, 10, 10, 1);
+ test(10, 10, 10, 255);
+ test(10, 10, 10, -1, 255);
+ test(10, 10, 10, -255, 1);
+}
+
« runtime/bin/process_win.cc ('K') | « tests/standalone/io/process_sync_script.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698