Improve analysis_server/protocol.dart test coverage.

pkg/analysis_server/test/protocol_test.dart

 // 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 test.protocol;
 
 import 'dart:convert';
 
 import 'package:analysis_server/src/protocol.dart';
 import 'package:unittest/unittest.dart';
 
 main() {
   group('Notification', () {
     test('getParameter_defined', NotificationTest.getParameter_defined);
     test('getParameter_undefined', NotificationTest.getParameter_undefined);
     test('fromJson', NotificationTest.fromJson);
     test('fromJson_withParams', NotificationTest.fromJson_withParams);
   });
   group('Request', () {
     test('getParameter_defined', RequestTest.getParameter_defined);
     test('getParameter_undefined', RequestTest.getParameter_undefined);
     test('getRequiredParameter_defined', RequestTest.getRequiredParameter_defined);
     test('getRequiredParameter_undefined', RequestTest.getRequiredParameter_undefined);
     test('fromJson', RequestTest.fromJson);
     test('fromJson_invalidId', RequestTest.fromJson_invalidId);
     test('fromJson_invalidMethod', RequestTest.fromJson_invalidMethod);
     test('fromJson_invalidParams', RequestTest.fromJson_invalidParams);
     test('fromJson_withParams', RequestTest.fromJson_withParams);
+    test('toBool', RequestTest.toBool);
+    test('toInt', RequestTest.toInt);
     test('toJson', RequestTest.toJson);
     test('toJson_withParams', RequestTest.toJson_withParams);
   });
+  group('RequestError', () {
+    test('create', RequestErrorTest.create);
+    test('create_methodNotFound', RequestErrorTest.create_methodNotFound);
+    test('create_invalidParameters', RequestErrorTest.create_invalidParameters);
+    test('create_invalidRequest', RequestErrorTest.create_invalidRequest);
+    test('create_internalError', RequestErrorTest.create_internalError);
+    test('create_parseError', RequestErrorTest.create_parseError);
+    test('fromJson', RequestErrorTest.fromJson);
+    test('toJson', RequestErrorTest.toJson);
+  });
   group('Response', () {
     test('create_contextDoesNotExist', ResponseTest.create_contextDoesNotExist);
     test('create_invalidRequestFormat', ResponseTest.create_invalidRequestFormat);
     test('create_missingRequiredParameter', ResponseTest.create_missingRequiredParameter);
+    test('create_unknownAnalysisOption', ResponseTest.create_unknownAnalysisOption);
     test('create_unknownRequest', ResponseTest.create_unknownRequest);
     test('setResult', ResponseTest.setResult);
     test('fromJson', ResponseTest.fromJson);
     test('fromJson_withError', ResponseTest.fromJson_withError);
     test('fromJson_withResult', ResponseTest.fromJson_withResult);
   });
 }
 
 class NotificationTest {
   static void getParameter_defined() {
@@ skipping 55 matching lines @@
     String name = 'name';
     String value = 'value';
     Request request = new Request('0', '');
     request.setParameter(name, value);
     expect(request.getRequiredParameter(name), equals(value));
   }
 
   static void getRequiredParameter_undefined() {
     String name = 'name';
     Request request = new Request('0', '');
-    expect(() => request.getRequiredParameter(name), throwsA(new isInstanceOf<RequestFailure>()));
+    expect(() => request.getRequiredParameter(name), _throwsRequestFailure);
   }
 
   static void fromJson() {
     Request original = new Request('one', 'aMethod');
     String json = JSON.encode(original.toJson());
     Request request = new Request.fromString(json);
     expect(request.id, equals('one'));
     expect(request.method, equals('aMethod'));
   }
 
@@ skipping 18 matching lines @@
   static void fromJson_withParams() {
     Request original = new Request('one', 'aMethod');
     original.setParameter('foo', 'bar');
     String json = JSON.encode(original.toJson());
     Request request = new Request.fromString(json);
     expect(request.id, equals('one'));
     expect(request.method, equals('aMethod'));
     expect(request.getParameter('foo'), equals('bar'));
   }
 
+  static void toBool() {
+    Request request = new Request('0', '');
+    expect(request.toBool(true), isTrue);
+    expect(request.toBool(false), isFalse);
+    expect(request.toBool('true'), isTrue);
+    expect(request.toBool('false'), isFalse);

[Brian Wilkerson, 2014/04/07 18:51:48]

Might be good to add a test for a string that is neither 'true' nor 'false'
(like 'abc').

[scheglov, 2014/04/07 19:01:41]

Done.

+    expect(() => request.toBool(42), _throwsRequestFailure);
+  }
+
+  static void toInt() {
+    Request request = new Request('0', '');
+    expect(request.toInt(1), equals(1));
+    expect(request.toInt('2'), equals(2));
+    expect(() => request.toInt('xxx'), _throwsRequestFailure);
+    expect(() => request.toInt(request), _throwsRequestFailure);
+  }
+
   static void toJson() {
     Request request = new Request('one', 'aMethod');
     expect(request.toJson(), equals({
       Request.ID : 'one',
       Request.METHOD : 'aMethod'
     }));
   }
 
   static void toJson_withParams() {
     Request request = new Request('one', 'aMethod');
     request.setParameter('foo', 'bar');
     expect(request.toJson(), equals({
       Request.ID : 'one',
       Request.METHOD : 'aMethod',
       Request.PARAMS : {'foo' : 'bar'}
     }));
   }
 }
 
+class RequestErrorTest {
+  static void create() {
+    RequestError error = new RequestError(42, 'msg');
+    expect(error.code, 42);
+    expect(error.message, "msg");
+    expect(error.toJson(), equals({
+      RequestError.CODE: 42,
+      RequestError.MESSAGE: "msg"
+    }));
+  }
+  static void create_parseError() {

[Brian Wilkerson, 2014/04/07 18:51:48]

nit: We should probably be consistent with spacing. I like the blank lines
between members, but we can discuss if you feel strongly.

[scheglov, 2014/04/07 19:01:41]

Agree, we need to put an empty line between members.
Fixed.

+    RequestError error = new RequestError.parseError();
+    expect(error.code, RequestError.CODE_PARSE_ERROR);
+    expect(error.message, "Parse error");
+  }
+  static void create_methodNotFound() {
+    RequestError error = new RequestError.methodNotFound();
+    expect(error.code, RequestError.CODE_METHOD_NOT_FOUND);
+    expect(error.message, "Method not found");
+  }
+  static void create_invalidParameters() {
+    RequestError error = new RequestError.invalidParameters();
+    expect(error.code, RequestError.CODE_INVALID_PARAMS);
+    expect(error.message, "Invalid parameters");
+  }
+  static void create_invalidRequest() {
+    RequestError error = new RequestError.invalidRequest();
+    expect(error.code, RequestError.CODE_INVALID_REQUEST);
+    expect(error.message, "Invalid request");
+  }
+  static void create_internalError() {
+    RequestError error = new RequestError.internalError();
+    expect(error.code, RequestError.CODE_INTERNAL_ERROR);
+    expect(error.message, "Internal error");
+  }
+  static void fromJson() {
+    var json = {
+        RequestError.CODE: RequestError.CODE_PARSE_ERROR,
+        RequestError.MESSAGE: 'foo',
+        RequestError.DATA: {'ints': [1, 2, 3]}
+    };
+    RequestError error = new RequestError.fromJson(json);
+    expect(error.code, RequestError.CODE_PARSE_ERROR);
+    expect(error.message, "foo");
+    expect(error.data['ints'], [1, 2, 3]);
+    expect(error.getData('ints'), [1, 2, 3]);
+  }
+  static void toJson() {
+    RequestError error = new RequestError(0, 'msg');
+    error.setData('answer', 42);
+    error.setData('question', 'unknown');
+    expect(error.toJson(), {
+        RequestError.CODE: 0,
+        RequestError.MESSAGE: 'msg',
+        RequestError.DATA: {'answer': 42, 'question': 'unknown'}
+    });
+  }
+}
+
 class ResponseTest {
   static void create_contextDoesNotExist() {
     Response response = new Response.contextDoesNotExist(new Request('0', ''));
     expect(response.id, equals('0'));
     expect(response.error, isNotNull);
     expect(response.toJson(), equals({
       Response.ID: '0',
       Response.ERROR: {'code': -1, 'message': 'Context does not exist'}
     }));
   }
@@ skipping 11 matching lines @@
   static void create_missingRequiredParameter() {
     Response response = new Response.missingRequiredParameter(new Request('0', ''), 'x');
     expect(response.id, equals('0'));
     expect(response.error, isNotNull);
     expect(response.toJson(), equals({
       Response.ID: '0',
       Response.ERROR: {'code': -5, 'message': 'Missing required parameter: x'}
     }));
   }
 
+  static void create_unknownAnalysisOption() {
+    Response response = new Response.unknownAnalysisOption(new Request('0', ''), 'x');
+    expect(response.id, equals('0'));
+    expect(response.error, isNotNull);
+    expect(response.toJson(), equals({
+      Response.ID: '0',
+      Response.ERROR: {'code': -6, 'message': 'Unknown analysis option: "x"'}
+    }));
+  }
+
   static void create_unknownRequest() {
     Response response = new Response.unknownRequest(new Request('0', ''));
     expect(response.id, equals('0'));
     expect(response.error, isNotNull);
     expect(response.toJson(), equals({
       Response.ID: '0',
-      Response.ERROR: {'code': -7, 'message': 'Unknown request'}
+          Response.ERROR: {'code': -7, 'message': 'Unknown request'}
     }));
   }
 
   static void setResult() {
     String resultName = 'name';
     String resultValue = 'value';
     Response response = new Response('0');
     response.setResult(resultName, resultValue);
+    expect(response.getResult(resultName), same(resultValue));
     expect(response.toJson(), equals({
       Response.ID: '0',
       Response.ERROR: null,
       Response.RESULT: {
         resultName: resultValue
       }
     }));
   }
 
   static void fromJson() {
@@ skipping 15 matching lines @@
   static void fromJson_withResult() {
     Response original = new Response('myId');
     original.setResult('foo', 'bar');
     Response response = new Response.fromJson(original.toJson());
     expect(response.id, equals('myId'));
     Map<String, Object> result = response.result;
     expect(result.length, equals(1));
     expect(result['foo'], equals('bar'));
   }
 }
+
+Matcher _throwsRequestFailure = throwsA(new isInstanceOf<RequestFailure>());