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

Side by Side Diff: pkg/analysis_server/test/edit/edit_domain_test.dart

Issue 477323005: Use generated classes to create requests in analysis server unit tests. (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.edit; 5 library test.domain.edit;
6 6
7 import 'package:analysis_server/src/analysis_server.dart'; 7 import 'package:analysis_server/src/analysis_server.dart';
8 import 'package:analysis_server/src/constants.dart';
9 import 'package:analysis_server/src/edit/edit_domain.dart'; 8 import 'package:analysis_server/src/edit/edit_domain.dart';
10 import 'package:analysis_server/src/protocol.dart'; 9 import 'package:analysis_server/src/protocol2.dart';
11 import 'package:analysis_testing/mock_sdk.dart'; 10 import 'package:analysis_testing/mock_sdk.dart';
12 import 'package:analyzer/file_system/memory_file_system.dart'; 11 import 'package:analyzer/file_system/memory_file_system.dart';
13 import 'package:unittest/unittest.dart'; 12 import 'package:unittest/unittest.dart';
14 13
15 import '../mocks.dart'; 14 import '../mocks.dart';
16 15
17 main() { 16 main() {
18 groupSep = ' | '; 17 groupSep = ' | ';
19 18
20 MockServerChannel serverChannel; 19 MockServerChannel serverChannel;
21 MemoryResourceProvider resourceProvider; 20 MemoryResourceProvider resourceProvider;
22 AnalysisServer server; 21 AnalysisServer server;
23 EditDomainHandler handler; 22 EditDomainHandler handler;
24 23
25 setUp(() { 24 setUp(() {
26 serverChannel = new MockServerChannel(); 25 serverChannel = new MockServerChannel();
27 resourceProvider = new MemoryResourceProvider(); 26 resourceProvider = new MemoryResourceProvider();
28 server = new AnalysisServer( 27 server = new AnalysisServer(
29 serverChannel, resourceProvider, new MockPackageMapProvider(), null, 28 serverChannel, resourceProvider, new MockPackageMapProvider(), null,
30 new MockSdk()); 29 new MockSdk());
31 handler = new EditDomainHandler(server); 30 handler = new EditDomainHandler(server);
32 }); 31 });
33 32
34 group('EditDomainHandler', () { 33 group('EditDomainHandler', () {
35 test('getAssists', () { 34 test('getAssists', () {
36 var request = new Request('0', EDIT_GET_ASSISTS); 35 // TODO(paulberry): Why are we passing null here?
37 request.setParameter(FILE, null); 36 var request = new EditGetAssistsParams(null, null, null).toRequest('0');
38 request.setParameter(OFFSET, null);
39 request.setParameter(LENGTH, null);
40 var response = handler.handleRequest(request); 37 var response = handler.handleRequest(request);
41 // TODO(brianwilkerson) implement 38 // TODO(brianwilkerson) implement
42 //expect(response, isNull); 39 //expect(response, isNull);
43 }); 40 });
44 41
45 test('getFixes', () { 42 test('getFixes', () {
46 var request = new Request('0', EDIT_GET_FIXES); 43 // TODO(paulberry): Test not yet written, and the code below doesn't
47 request.setParameter(ERRORS, []); 44 // follow the API.
48 var response = handler.handleRequest(request); 45 // var request = new Request('0', EDIT_GET_FIXES);
49 // TODO(brianwilkerson) implement 46 // request.setParameter(ERRORS, []);
50 //expect(response, isNull); 47 // var response = handler.handleRequest(request);
48 // // TODO(brianwilkerson) implement
49 // //expect(response, isNull);
51 }); 50 });
52 }); 51 });
53 } 52 }
OLDNEW
« no previous file with comments | « pkg/analysis_server/test/edit/assists_test.dart ('k') | pkg/analysis_server/test/edit/fixes_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698