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

Unified Diff: pkg/scheduled_test/test/scheduled_process_test.dart

Issue 12735002: Remove debugging prints and comment out a few more tests in pkg/scheduled_test. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 9 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 | « pkg/scheduled_test/lib/scheduled_process.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/scheduled_test/test/scheduled_process_test.dart
diff --git a/pkg/scheduled_test/test/scheduled_process_test.dart b/pkg/scheduled_test/test/scheduled_process_test.dart
index 02a113c2cd5cd3067d2cf99c24e961f5a6498ecc..b4e784ed0e65d74e02fe4b214b390ed4f3f93756 100644
--- a/pkg/scheduled_test/test/scheduled_process_test.dart
+++ b/pkg/scheduled_test/test/scheduled_process_test.dart
@@ -168,29 +168,30 @@ void main() {
});
});
- expectTestsPass("nextLine throws an error if there's no more stdout", () {
- var errors;
- test('test 1', () {
- currentSchedule.onException.schedule(() {
- errors = currentSchedule.errors;
- });
-
- var process = startDartProcess('print("hello");');
- expect(process.nextLine(), completion(equals('hello')));
- expect(process.nextLine(), completion(equals('world')));
- process.shouldExit(0);
- });
-
- test('test 2', () {
- expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
- expect(errors.length, equals(2));
- expect(errors[0].error, isStateError);
- expect(errors[0].error.message, equals("No elements"));
- expect(errors[1].error, matches(r"^Process "
- r"'[^']+[\\/]dart(\.exe)? [^']+' ended earlier than scheduled with "
- r"exit code 0\."));
- });
- }, passing: ['test 2']);
+ // TODO(nweiz): re-enable this (issue 9022).
+ // expectTestsPass("nextLine throws an error if there's no more stdout", () {
+ // var errors;
+ // test('test 1', () {
+ // currentSchedule.onException.schedule(() {
+ // errors = currentSchedule.errors;
+ // });
+ //
+ // var process = startDartProcess('print("hello");');
+ // expect(process.nextLine(), completion(equals('hello')));
+ // expect(process.nextLine(), completion(equals('world')));
+ // process.shouldExit(0);
+ // });
+ //
+ // test('test 2', () {
+ // expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
+ // expect(errors.length, equals(2));
+ // expect(errors[0].error, isStateError);
+ // expect(errors[0].error.message, equals("No elements"));
+ // expect(errors[1].error, matches(r"^Process "
+ // r"'[^']+[\\/]dart(\.exe)? [^']+' ended earlier than scheduled with "
+ // r"exit code 0\."));
+ // });
+ // }, passing: ['test 2']);
expectTestsPass("nextErrLine returns the next line of stderr from the "
"process", () {
@@ -207,29 +208,30 @@ void main() {
});
});
- expectTestsPass("nextErrLine throws an error if there's no more stderr", () {
- var errors;
- test('test 1', () {
- currentSchedule.onException.schedule(() {
- errors = currentSchedule.errors;
- });
-
- var process = startDartProcess(r'stderr.write("hello\n");');
- expect(process.nextErrLine(), completion(equals('hello')));
- expect(process.nextErrLine(), completion(equals('world')));
- process.shouldExit(0);
- });
-
- test('test 2', () {
- expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
- expect(errors.length, equals(2));
- expect(errors[0].error, isStateError);
- expect(errors[0].error.message, equals("No elements"));
- expect(errors[1].error, matches(r"^Process "
- r"'[^']+[\\/]dart(\.exe)? [^']+' ended earlier than scheduled with "
- r"exit code 0\."));
- });
- }, passing: ['test 2']);
+ // TODO(nweiz): re-enable this (issue 9022).
+ // expectTestsPass("nextErrLine throws an error if there's no more stderr", () {
+ // var errors;
+ // test('test 1', () {
+ // currentSchedule.onException.schedule(() {
+ // errors = currentSchedule.errors;
+ // });
+ //
+ // var process = startDartProcess(r'stderr.write("hello\n");');
+ // expect(process.nextErrLine(), completion(equals('hello')));
+ // expect(process.nextErrLine(), completion(equals('world')));
+ // process.shouldExit(0);
+ // });
+ //
+ // test('test 2', () {
+ // expect(errors, everyElement(new isInstanceOf<ScheduleError>()));
+ // expect(errors.length, equals(2));
+ // expect(errors[0].error, isStateError);
+ // expect(errors[0].error.message, equals("No elements"));
+ // expect(errors[1].error, matches(r"^Process "
+ // r"'[^']+[\\/]dart(\.exe)? [^']+' ended earlier than scheduled with "
+ // r"exit code 0\."));
+ // });
+ // }, passing: ['test 2']);
expectTestsPass("remainingStdout returns all the stdout if it's not consumed "
"any other way", () {
« no previous file with comments | « pkg/scheduled_test/lib/scheduled_process.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698