Chromium Code Reviews| Index: sdk/lib/_internal/pub/test/lish/upload_form_provides_invalid_json_test.dart |
| diff --git a/sdk/lib/_internal/pub/test/lish/upload_form_provides_invalid_json_test.dart b/sdk/lib/_internal/pub/test/lish/upload_form_provides_invalid_json_test.dart |
| index 21ca53a1732803843b210a6cb186e6017036a1ce..2173885cfd4ce88a46f22850a7a873417d89e73e 100644 |
| --- a/sdk/lib/_internal/pub/test/lish/upload_form_provides_invalid_json_test.dart |
| +++ b/sdk/lib/_internal/pub/test/lish/upload_form_provides_invalid_json_test.dart |
| @@ -24,8 +24,9 @@ main() { |
| request.response.close(); |
| }); |
| - expect(pub.nextErrLine(), completion(equals('Invalid server response:'))); |
| - expect(pub.nextErrLine(), completion(equals('{not json'))); |
| + pub.stderr.expect(emitsString( |
| + 'Invalid server response:\n' |
| + '{not json')); |
|
Bob Nystrom
2014/02/14 17:55:02
In most other places, you seem to do a separate pu
nweiz
2014/02/18 22:01:10
The places with two expect calls have `JSON.encode
Bob Nystrom
2014/02/18 22:12:59
I don't have a strong preference I just thought th
|
| pub.shouldExit(1); |
| }); |
| } |