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

Side by Side Diff: pkg/analysis_server/test/domain_server_test.dart

Issue 482573004: Change analysis server protocol to omit empty lists when optional. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 4 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 | Annotate | Revision Log
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 test.domain.server; 5 library test.domain.server;
6 6
7 import 'package:analyzer/file_system/physical_file_system.dart'; 7 import 'package:analyzer/file_system/physical_file_system.dart';
8 import 'package:analysis_server/src/analysis_server.dart'; 8 import 'package:analysis_server/src/analysis_server.dart';
9 import 'package:analysis_server/src/constants.dart'; 9 import 'package:analysis_server/src/constants.dart';
10 import 'package:analysis_server/src/domain_server.dart'; 10 import 'package:analysis_server/src/domain_server.dart';
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 SUBSCRIPTIONS: ['noSuchService'] 46 SUBSCRIPTIONS: ['noSuchService']
47 }); 47 });
48 var response = handler.handleRequest(request); 48 var response = handler.handleRequest(request);
49 expect(response, isResponseFailure('0')); 49 expect(response, isResponseFailure('0'));
50 }); 50 });
51 51
52 test('success', () { 52 test('success', () {
53 expect(server.serverServices, isEmpty); 53 expect(server.serverServices, isEmpty);
54 // send request 54 // send request
55 Request request = new ServerSetSubscriptionsParams( 55 Request request = new ServerSetSubscriptionsParams(
56 [ServerService.STATUS]).toRequest('0'); 56 subscriptions: [ServerService.STATUS]).toRequest('0');
57 var response = handler.handleRequest(request); 57 var response = handler.handleRequest(request);
58 expect(response, isResponseSuccess('0')); 58 expect(response, isResponseSuccess('0'));
59 // set of services has been changed 59 // set of services has been changed
60 expect(server.serverServices, contains(ServerService.STATUS)); 60 expect(server.serverServices, contains(ServerService.STATUS));
61 }); 61 });
62 }); 62 });
63 63
64 test('shutdown', () { 64 test('shutdown', () {
65 expect(server.running, isTrue); 65 expect(server.running, isTrue);
66 // send request 66 // send request
67 var request = new Request('0', SERVER_SHUTDOWN); 67 var request = new Request('0', SERVER_SHUTDOWN);
68 var response = handler.handleRequest(request); 68 var response = handler.handleRequest(request);
69 expect(response, isResponseSuccess('0')); 69 expect(response, isResponseSuccess('0'));
70 // server is down 70 // server is down
71 expect(server.running, isFalse); 71 expect(server.running, isFalse);
72 }); 72 });
73 }); 73 });
74 } 74 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698