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

Side by Side Diff: test/server/batch_test.dart

Issue 864983002: Fix a couple test timeouts. (Closed) Base URL: git@github.com:dart-lang/json_rpc_2@master
Patch Set: Created 5 years, 11 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
« no previous file with comments | « no previous file | test/server/server_test.dart » ('j') | 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) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 json_rpc_2.test.server.batch_test; 5 library json_rpc_2.test.server.batch_test;
6 6
7 import 'dart:convert'; 7 import 'dart:convert';
8 8
9 import 'package:unittest/unittest.dart'; 9 import 'package:unittest/unittest.dart';
10 import 'package:json_rpc_2/error_code.dart' as error_code; 10 import 'package:json_rpc_2/error_code.dart' as error_code;
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 {'jsonrpc': '2.0', 'result': 'foo', 'id': 1}, 63 {'jsonrpc': '2.0', 'result': 'foo', 'id': 1},
64 {'jsonrpc': '2.0', 'result': 'value', 'id': 3} 64 {'jsonrpc': '2.0', 'result': 'value', 'id': 3}
65 ]))); 65 ])));
66 }); 66 });
67 67
68 test('returns nothing if every request is a notification', () { 68 test('returns nothing if every request is a notification', () {
69 expect(controller.handleRequest([ 69 expect(controller.handleRequest([
70 {'jsonrpc': '2.0', 'method': 'foo'}, 70 {'jsonrpc': '2.0', 'method': 'foo'},
71 {'jsonrpc': '2.0', 'method': 'id', 'params': ['value']}, 71 {'jsonrpc': '2.0', 'method': 'id', 'params': ['value']},
72 {'jsonrpc': '2.0', 'method': 'arg', 'params': {'arg': 'value'}} 72 {'jsonrpc': '2.0', 'method': 'arg', 'params': {'arg': 'value'}}
73 ]), completion(isNull)); 73 ]), doesNotComplete);
74 }); 74 });
75 75
76 test('returns an error if the batch is empty', () { 76 test('returns an error if the batch is empty', () {
77 expectErrorResponse(controller, [], error_code.INVALID_REQUEST, 77 expectErrorResponse(controller, [], error_code.INVALID_REQUEST,
78 'A batch must contain at least one request.'); 78 'A batch must contain at least one request.');
79 }); 79 });
80 80
81 test('disallows nested batches', () { 81 test('disallows nested batches', () {
82 expect(controller.handleRequest([ 82 expect(controller.handleRequest([
83 [{'jsonrpc': '2.0', 'method': 'foo', 'id': 1}] 83 [{'jsonrpc': '2.0', 'method': 'foo', 'id': 1}]
84 ]), completion(equals([{ 84 ]), completion(equals([{
85 'jsonrpc': '2.0', 85 'jsonrpc': '2.0',
86 'id': null, 86 'id': null,
87 'error': { 87 'error': {
88 'code': error_code.INVALID_REQUEST, 88 'code': error_code.INVALID_REQUEST,
89 'message': 'Request must be an Array or an Object.', 89 'message': 'Request must be an Array or an Object.',
90 'data': {'request': [{'jsonrpc': '2.0', 'method': 'foo', 'id': 1}]} 90 'data': {'request': [{'jsonrpc': '2.0', 'method': 'foo', 'id': 1}]}
91 } 91 }
92 }]))); 92 }])));
93 }); 93 });
94 } 94 }
OLDNEW
« no previous file with comments | « no previous file | test/server/server_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698