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

Unified Diff: pkg/analysis_server/test/integration/analysis/get_errors_after_analysis_test.dart

Issue 2564863003: Get all integration tests running with the new driver (Closed)
Patch Set: Created 4 years 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 side-by-side diff with in-line comments
Download patch
Index: pkg/analysis_server/test/integration/analysis/get_errors_after_analysis_test.dart
diff --git a/pkg/analysis_server/test/integration/analysis/get_errors_after_analysis_test.dart b/pkg/analysis_server/test/integration/analysis/get_errors_after_analysis_test.dart
index 37caf81e16e54cf737b241f263cd1b0d1d23172a..e7d57c575b14410316bb678fecb71a4d60323aa4 100644
--- a/pkg/analysis_server/test/integration/analysis/get_errors_after_analysis_test.dart
+++ b/pkg/analysis_server/test/integration/analysis/get_errors_after_analysis_test.dart
@@ -2,19 +2,49 @@
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
-library test.integration.analysis.get.errors.after.analysis;
-
import 'package:test_reflective_loader/test_reflective_loader.dart';
import 'get_errors.dart';
main() {
defineReflectiveSuite(() {
- defineReflectiveTests(Test);
+ defineReflectiveTests(GetErrorsAfterTest);
+ defineReflectiveTests(GetErrorsAfterTest_Driver);
});
}
+class AbstractGetErrorsAfterTest extends AnalysisDomainGetErrorsTest {
+ AbstractGetErrorsAfterTest() : super(true);
+}
+
@reflectiveTest
-class Test extends AnalysisDomainGetErrorsTest {
- Test() : super(true);
+class GetErrorsAfterTest extends AbstractGetErrorsAfterTest {}
+
+@reflectiveTest
+class GetErrorsAfterTest_Driver extends AbstractGetErrorsAfterTest {
+ @override
+ bool get enableNewAnalysisDriver => true;
+
+ @failingTest
+ test_getErrors() {
+ // UnimplementedError: Server responded with an error:
+ // {"error":{"code":"SERVER_ERROR","message":"Bad state: Should not be used with the new analysis driver","stackTrace":"
+ // #0 ContextManagerImpl.folderMap (package:analysis_server/src/context_manager.dart:550:7)
+ // #1 ContextManagerImpl.analysisContexts (package:analysis_server/src/context_manager.dart:546:53)
+ // #2 AnalysisServer.analysisContexts (package:analysis_server/src/analysis_server.dart:453:22)
+ // #3 AnalysisServer.getContextSourcePair (package:analysis_server/src/analysis_server.dart:722:37)
+ // #4 AnalysisServer.getAnalysisContext (package:analysis_server/src/analysis_server.dart:590:12)
+ // #5 AnalysisServer.onFileAnalysisComplete (package:analysis_server/src/analysis_server.dart:948:31)
+ // #6 AnalysisDomainHandler.getErrors (package:analysis_server/src/domain_analysis.dart:54:16)
+ // #7 AnalysisDomainHandler.handleRequest (package:analysis_server/src/domain_analysis.dart:201:16)
+ // #8 AnalysisServer.handleRequest.<anonymous closure>.<anonymous closure> (package:analysis_server/src/analysis_server.dart:873:45)
+ // #9 _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:189:15)
+ // #10 AnalysisServer.handleRequest.<anonymous closure> (package:analysis_server/src/analysis_server.dart:869:50)
+ // #11 _rootRun (dart:async/zone.dart:1150)
+ // #12 _CustomZone.run (dart:async/zone.dart:1026)
+ // #13 _CustomZone.runGuarded (dart:async/zone.dart:924)
+ // #14 runZoned (dart:async/zone.dart:1501)
+ // #15 AnalysisServer.handleRequest (package:analysis_server/src/analysis_server.dart:868:5)
+ return super.test_getErrors();
+ }
}

Powered by Google App Engine
This is Rietveld 408576698