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

Side by Side Diff: pkg/analysis_server/test/analysis/reanalyze_test.dart

Issue 2988743002: Remove the analysus.reanalyze request from the plugin API (Closed)
Patch Set: Restart plugins Created 3 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
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 'dart:async'; 5 import 'dart:async';
6 6
7 import 'package:analysis_server/protocol/protocol.dart'; 7 import 'package:analysis_server/protocol/protocol.dart';
8 import 'package:analysis_server/protocol/protocol_constants.dart'; 8 import 'package:analysis_server/protocol/protocol_constants.dart';
9 import 'package:analysis_server/protocol/protocol_generated.dart'; 9 import 'package:analysis_server/protocol/protocol_generated.dart';
10 import 'package:analyzer_plugin/protocol/protocol_common.dart'; 10 import 'package:analyzer_plugin/protocol/protocol_common.dart';
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 await _resultsAvailable.future; 65 await _resultsAvailable.future;
66 // The file should have been reanalyzed. 66 // The file should have been reanalyzed.
67 expect(filesErrors, contains(testFile)); 67 expect(filesErrors, contains(testFile));
68 // Verify that the syntax error is present (this indicates that the 68 // Verify that the syntax error is present (this indicates that the
69 // content introduced by the call to updateContent is still in effect). 69 // content introduced by the call to updateContent is still in effect).
70 { 70 {
71 List<AnalysisError> errors = filesErrors[testFile]; 71 List<AnalysisError> errors = filesErrors[testFile];
72 expect(errors, hasLength(1)); 72 expect(errors, hasLength(1));
73 } 73 }
74 } 74 }
75
76 test_sentToPlugins() async {
77 createProject();
78 Request request = new Request("0", ANALYSIS_REQUEST_REANALYZE);
79 handleSuccessfulRequest(request);
80 // verify
81 expect(pluginManager.broadcastedRequest, isNotNull);
82 }
83 } 75 }
OLDNEW
« no previous file with comments | « pkg/analysis_server/lib/src/plugin/request_converter.dart ('k') | pkg/analysis_server/test/analysis_abstract.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698