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

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

Issue 2960073002: Generate constants in server as we do in plugin (Closed)
Patch Set: improve names Created 3 years, 5 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
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 import 'package:analysis_server/protocol/protocol.dart'; 5 import 'package:analysis_server/protocol/protocol.dart';
6 import 'package:analysis_server/protocol/protocol_constants.dart';
6 import 'package:analysis_server/protocol/protocol_generated.dart'; 7 import 'package:analysis_server/protocol/protocol_generated.dart';
7 import 'package:analysis_server/src/analysis_server.dart'; 8 import 'package:analysis_server/src/analysis_server.dart';
8 import 'package:analysis_server/src/constants.dart'; 9 import 'package:analysis_server/src/constants.dart';
9 import 'package:analysis_server/src/domain_server.dart'; 10 import 'package:analysis_server/src/domain_server.dart';
10 import 'package:analysis_server/src/plugin/server_plugin.dart'; 11 import 'package:analysis_server/src/plugin/server_plugin.dart';
11 import 'package:analyzer/file_system/memory_file_system.dart'; 12 import 'package:analyzer/file_system/memory_file_system.dart';
12 import 'package:analyzer/instrumentation/instrumentation.dart'; 13 import 'package:analyzer/instrumentation/instrumentation.dart';
13 import 'package:analyzer/src/generated/sdk.dart'; 14 import 'package:analyzer/src/generated/sdk.dart';
14 import 'package:plugin/manager.dart'; 15 import 'package:plugin/manager.dart';
15 import 'package:test/test.dart'; 16 import 'package:test/test.dart';
(...skipping 29 matching lines...) Expand all
45 expect( 46 expect(
46 response.toJson(), 47 response.toJson(),
47 equals({ 48 equals({
48 Response.ID: '0', 49 Response.ID: '0',
49 Response.RESULT: {VERSION: AnalysisServer.VERSION} 50 Response.RESULT: {VERSION: AnalysisServer.VERSION}
50 })); 51 }));
51 }); 52 });
52 53
53 group('setSubscriptions', () { 54 group('setSubscriptions', () {
54 test('invalid service name', () { 55 test('invalid service name', () {
55 Request request = new Request('0', SERVER_SET_SUBSCRIPTIONS, { 56 Request request = new Request('0', SERVER_REQUEST_SET_SUBSCRIPTIONS, {
56 SUBSCRIPTIONS: ['noSuchService'] 57 SUBSCRIPTIONS: ['noSuchService']
57 }); 58 });
58 var response = handler.handleRequest(request); 59 var response = handler.handleRequest(request);
59 expect(response, isResponseFailure('0')); 60 expect(response, isResponseFailure('0'));
60 }); 61 });
61 62
62 test('success', () { 63 test('success', () {
63 expect(server.serverServices, isEmpty); 64 expect(server.serverServices, isEmpty);
64 // send request 65 // send request
65 Request request = 66 Request request =
(...skipping 10 matching lines...) Expand all
76 expect(server.running, isTrue); 77 expect(server.running, isTrue);
77 // send request 78 // send request
78 var request = new ServerShutdownParams().toRequest('0'); 79 var request = new ServerShutdownParams().toRequest('0');
79 var response = handler.handleRequest(request); 80 var response = handler.handleRequest(request);
80 expect(response, isResponseSuccess('0')); 81 expect(response, isResponseSuccess('0'));
81 // server is down 82 // server is down
82 expect(server.running, isFalse); 83 expect(server.running, isFalse);
83 }); 84 });
84 }); 85 });
85 } 86 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698