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

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

Issue 335123008: Implement analysis.updateOptions (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Address comments Created 6 years, 6 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/protocol.dart ('k') | no next file » | 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/computer/computer_highlights.dart'; 9 import 'package:analysis_server/src/computer/computer_highlights.dart';
10 import 'package:analysis_server/src/analysis_server.dart'; 10 import 'package:analysis_server/src/analysis_server.dart';
11 import 'package:analysis_server/src/constants.dart'; 11 import 'package:analysis_server/src/constants.dart';
12 import 'package:analysis_server/src/domain_analysis.dart'; 12 import 'package:analysis_server/src/domain_analysis.dart';
13 import 'package:analysis_server/src/protocol.dart'; 13 import 'package:analysis_server/src/protocol.dart';
14 import 'package:analysis_server/src/resource.dart'; 14 import 'package:analysis_server/src/resource.dart';
15 import 'package:analyzer/src/generated/engine.dart';
15 import 'package:unittest/unittest.dart'; 16 import 'package:unittest/unittest.dart';
16 17
17 import 'mocks.dart'; 18 import 'mocks.dart';
18 19
19 20
20 main() { 21 main() {
21 groupSep = ' | '; 22 groupSep = ' | ';
22 23
23 MockServerChannel serverChannel; 24 MockServerChannel serverChannel;
24 MemoryResourceProvider resourceProvider; 25 MemoryResourceProvider resourceProvider;
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 // has been implemented. 107 // has been implemented.
107 // expect(server.getPriorityFiles(), unorderedEquals(fileList)); 108 // expect(server.getPriorityFiles(), unorderedEquals(fileList));
108 } 109 }
109 110
110 setPriorityFiles(['/p1/a.dart', '/p2/b.dart']); 111 setPriorityFiles(['/p1/a.dart', '/p2/b.dart']);
111 setPriorityFiles(['/p2/b.dart', '/p2/c.dart']); 112 setPriorityFiles(['/p2/b.dart', '/p2/c.dart']);
112 setPriorityFiles([]); 113 setPriorityFiles([]);
113 }); 114 });
114 }); 115 });
115 116
116 test('updateOptions', () { 117 group('updateOptions', () {
117 var request = new Request('0', ANALYSIS_UPDATE_OPTIONS); 118 test('invalid', () {
118 request.setParameter( 119 var request = new Request('0', ANALYSIS_UPDATE_OPTIONS);
119 OPTIONS, 120 request.setParameter(OPTIONS, {'not-an-option' : true});
120 { 121 var response = handler.handleRequest(request);
121 'analyzeAngular' : true, 122 expect(response, isResponseFailure('0'));
122 'enableDeferredLoading': true, 123 });
123 'enableEnums': false 124
124 }); 125 test('valid', () {
125 var response = handler.handleRequest(request); 126 AnalysisOptions options = server.contextDirectoryManager.defaultOptions;
126 // TODO(scheglov) implement 127 bool analyzeAngular = !options.analyzeAngular;
127 expect(response, isNull); 128 bool enableDeferredLoading = options.enableDeferredLoading;
129 var request = new Request('0', ANALYSIS_UPDATE_OPTIONS);
130 request.setParameter(
131 OPTIONS,
132 {
133 'analyzeAngular' : analyzeAngular,
134 'enableDeferredLoading': enableDeferredLoading,
135 'enableEnums': false
136 });
137 var response = handler.handleRequest(request);
138 expect(response, isResponseSuccess('0'));
139 expect(options.analyzeAngular, equals(analyzeAngular));
140 expect(options.enableDeferredLoading, equals(enableDeferredLoading));
141 });
128 }); 142 });
129 143
130 test('updateSdks', () { 144 test('updateSdks', () {
131 var request = new Request('0', ANALYSIS_UPDATE_SDKS); 145 var request = new Request('0', ANALYSIS_UPDATE_SDKS);
132 request.setParameter( 146 request.setParameter(
133 ADDED, 147 ADDED,
134 ['/dart/sdk-1.3', '/dart/sdk-1.4']); 148 ['/dart/sdk-1.3', '/dart/sdk-1.4']);
135 request.setParameter( 149 request.setParameter(
136 REMOVED, 150 REMOVED,
137 ['/dart/sdk-1.2']); 151 ['/dart/sdk-1.2']);
(...skipping 1549 matching lines...) Expand 10 before | Expand all | Expand 10 after
1687 // subscribe 1701 // subscribe
1688 helper.addAnalysisSubscriptionHighlights(helper.testFile); 1702 helper.addAnalysisSubscriptionHighlights(helper.testFile);
1689 // wait, has regions 1703 // wait, has regions
1690 return helper.waitForOperationsFinished().then((_) { 1704 return helper.waitForOperationsFinished().then((_) {
1691 var highlights = helper.getHighlights(helper.testFile); 1705 var highlights = helper.getHighlights(helper.testFile);
1692 expect(highlights, isNot(isEmpty)); 1706 expect(highlights, isNot(isEmpty));
1693 }); 1707 });
1694 }); 1708 });
1695 }); 1709 });
1696 } 1710 }
OLDNEW
« no previous file with comments | « pkg/analysis_server/lib/src/protocol.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698