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

Unified Diff: pkg/json_rpc_2/test/server/invalid_request_test.dart

Issue 205533005: Create a package that implements a JSON-RPC 2.0 server. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: code review Created 6 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/json_rpc_2/test/server/batch_test.dart ('k') | pkg/json_rpc_2/test/server/parameters_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/json_rpc_2/test/server/invalid_request_test.dart
diff --git a/pkg/json_rpc_2/test/server/invalid_request_test.dart b/pkg/json_rpc_2/test/server/invalid_request_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..feeefea9c4c544b4c765119bdf1c58113a2a2548
--- /dev/null
+++ b/pkg/json_rpc_2/test/server/invalid_request_test.dart
@@ -0,0 +1,86 @@
+// Copyright (c) 2014, 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.
+
+library json_rpc_2.test.server.invalid_request_test;
+
+import 'dart:convert';
+
+import 'package:unittest/unittest.dart';
+import 'package:json_rpc_2/error_code.dart' as error_code;
+import 'package:json_rpc_2/json_rpc_2.dart' as json_rpc;
+
+import 'utils.dart';
+
+void main() {
+ var server;
+ setUp(() => server = new json_rpc.Server());
+
+ test("a non-Array/Object request is invalid", () {
+ expectErrorResponse(server, 'foo', error_code.INVALID_REQUEST,
+ 'Request must be an Array or an Object.');
+ });
+
+ test("requests must have a jsonrpc key", () {
+ expectErrorResponse(server, {
+ 'method': 'foo',
+ 'id': 1234
+ }, error_code.INVALID_REQUEST, 'Request must contain a "jsonrpc" key.');
+ });
+
+ test("the jsonrpc version must be 2.0", () {
+ expectErrorResponse(server, {
+ 'jsonrpc': '1.0',
+ 'method': 'foo',
+ 'id': 1234
+ }, error_code.INVALID_REQUEST,
+ 'Invalid JSON-RPC version "1.0", expected "2.0".');
+ });
+
+ test("requests must have a method key", () {
+ expectErrorResponse(server, {
+ 'jsonrpc': '2.0',
+ 'id': 1234
+ }, error_code.INVALID_REQUEST, 'Request must contain a "method" key.');
+ });
+
+ test("request method must be a string", () {
+ expectErrorResponse(server, {
+ 'jsonrpc': '2.0',
+ 'method': 1234,
+ 'id': 1234
+ }, error_code.INVALID_REQUEST,
+ 'Request method must be a string, but was 1234.');
+ });
+
+ test("request params must be an Array or Object", () {
+ expectErrorResponse(server, {
+ 'jsonrpc': '2.0',
+ 'method': 'foo',
+ 'params': 1234,
+ 'id': 1234
+ }, error_code.INVALID_REQUEST,
+ 'Request params must be an Array or an Object, but was 1234.');
+ });
+
+ test("request id may not be an Array or Object", () {
+ expect(server.handleRequest({
+ 'jsonrpc': '2.0',
+ 'method': 'foo',
+ 'id': {'bad': 'id'}
+ }), completion(equals({
+ 'jsonrpc': '2.0',
+ 'id': null,
+ 'error': {
+ 'code': error_code.INVALID_REQUEST,
+ 'message': 'Request id must be a string, number, or null, but was '
+ '{"bad":"id"}.',
+ 'data': {'request': {
+ 'jsonrpc': '2.0',
+ 'method': 'foo',
+ 'id': {'bad': 'id'}
+ }}
+ }
+ })));
+ });
+}
« no previous file with comments | « pkg/json_rpc_2/test/server/batch_test.dart ('k') | pkg/json_rpc_2/test/server/parameters_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698