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

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

Issue 629043002: Remove from AnalysisServer the 'test_' methods which now has non-test implementations. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 2 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
« no previous file with comments | « pkg/analysis_server/lib/src/analysis_server.dart ('k') | pkg/analysis_server/test/mocks.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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.analysis; 5 library test.domain.analysis;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 8
9 import 'package:analysis_server/src/analysis_server.dart'; 9 import 'package:analysis_server/src/analysis_server.dart';
10 import 'package:analysis_server/src/constants.dart'; 10 import 'package:analysis_server/src/constants.dart';
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 group('setAnalysisRoots', () { 49 group('setAnalysisRoots', () {
50 Response testSetAnalysisRoots(List<String> included, 50 Response testSetAnalysisRoots(List<String> included,
51 List<String> excluded) { 51 List<String> excluded) {
52 Request request = new AnalysisSetAnalysisRootsParams(included, 52 Request request = new AnalysisSetAnalysisRootsParams(included,
53 excluded).toRequest('0'); 53 excluded).toRequest('0');
54 return handler.handleRequest(request); 54 return handler.handleRequest(request);
55 } 55 }
56 56
57 group('excluded', () { 57 group('excluded', () {
58 test('excluded folder', () { 58 test('excluded folder', () {
59 String project = '/project';
60 String fileA = '/project/aaa/a.dart'; 59 String fileA = '/project/aaa/a.dart';
61 String fileB = '/project/bbb/b.dart'; 60 String fileB = '/project/bbb/b.dart';
62 resourceProvider.newFolder(project);
63 resourceProvider.newFile(fileA, '// a'); 61 resourceProvider.newFile(fileA, '// a');
64 resourceProvider.newFile(fileB, '// b'); 62 resourceProvider.newFile(fileB, '// b');
65 var response = testSetAnalysisRoots([project], ['/project/bbb']); 63 var response = testSetAnalysisRoots(['/project'], ['/project/bbb']);
66 var serverRef = server; 64 var serverRef = server;
67 expect(response, isResponseSuccess('0')); 65 expect(response, isResponseSuccess('0'));
68 // unit "a" is resolved eventually 66 // unit "a" is resolved eventually
69 // unit "b" is not resolved 67 // unit "b" is not resolved
70 return waitForServerOperationsPerformed(server).then((_) { 68 return waitForServerOperationsPerformed(server).then((_) {
71 expect(serverRef.test_getResolvedCompilationUnit(fileA), isNotNull); 69 expect(serverRef.getResolvedCompilationUnits(fileA), hasLength(1));
72 expect(serverRef.test_getResolvedCompilationUnit(fileB), isNull); 70 expect(serverRef.getResolvedCompilationUnits(fileB), isEmpty);
73 }); 71 });
74 }); 72 });
75 }); 73 });
76 74
77 group('included', () { 75 group('included', () {
78 test('new folder', () { 76 test('new folder', () {
79 resourceProvider.newFolder('/project'); 77 String file = '/project/bin/test.dart';
80 resourceProvider.newFile('/project/pubspec.yaml', 'name: project'); 78 resourceProvider.newFile('/project/pubspec.yaml', 'name: project');
81 resourceProvider.newFile('/project/bin/test.dart', 'main() {}'); 79 resourceProvider.newFile(file, 'main() {}');
82 var response = testSetAnalysisRoots(['/project'], []); 80 var response = testSetAnalysisRoots(['/project'], []);
83 var serverRef = server; 81 var serverRef = server;
84 expect(response, isResponseSuccess('0')); 82 expect(response, isResponseSuccess('0'));
85 // verify that unit is resolved eventually 83 // verify that unit is resolved eventually
86 return waitForServerOperationsPerformed(server).then((_) { 84 return waitForServerOperationsPerformed(server).then((_) {
87 var unit = 85 var units = serverRef.getResolvedCompilationUnits(file);
88 serverRef.test_getResolvedCompilationUnit('/project/bin/test.dar t'); 86 expect(units, hasLength(1));
89 expect(unit, isNotNull);
90 }); 87 });
91 }); 88 });
92 }); 89 });
93 }); 90 });
94 91
95 group('setPriorityFiles', () { 92 group('setPriorityFiles', () {
96 test('invalid', () { 93 test('invalid', () {
97 // TODO(paulberry): under the "eventual consistency" model this request 94 // TODO(paulberry): under the "eventual consistency" model this request
98 // should not be invalid. 95 // should not be invalid.
99 var request = new AnalysisSetPriorityFilesParams( 96 var request = new AnalysisSetPriorityFilesParams(
(...skipping 537 matching lines...) Expand 10 before | Expand all | Expand 10 after
637 return waitForServerOperationsPerformed(server); 634 return waitForServerOperationsPerformed(server);
638 } 635 }
639 636
640 static String _getCodeString(code) { 637 static String _getCodeString(code) {
641 if (code is List<String>) { 638 if (code is List<String>) {
642 code = code.join('\n'); 639 code = code.join('\n');
643 } 640 }
644 return code as String; 641 return code as String;
645 } 642 }
646 } 643 }
OLDNEW
« no previous file with comments | « pkg/analysis_server/lib/src/analysis_server.dart ('k') | pkg/analysis_server/test/mocks.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698