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

Side by Side Diff: pkg/analysis_server/test/integration/analysis/update_content_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 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 library test.integration.analysis.update.content;
6
7 import 'package:analysis_server/plugin/protocol/protocol.dart'; 5 import 'package:analysis_server/plugin/protocol/protocol.dart';
8 import 'package:test/test.dart'; 6 import 'package:test/test.dart';
9 import 'package:test_reflective_loader/test_reflective_loader.dart'; 7 import 'package:test_reflective_loader/test_reflective_loader.dart';
10 8
11 import '../integration_tests.dart'; 9 import '../integration_tests.dart';
12 10
13 main() { 11 main() {
14 defineReflectiveSuite(() { 12 defineReflectiveSuite(() {
15 defineReflectiveTests(Test); 13 defineReflectiveTests(UpdateContentTest);
14 defineReflectiveTests(UpdateContentTest_Driver);
16 }); 15 });
17 } 16 }
18 17
19 @reflectiveTest 18 class AbstractUpdateContentTest extends AbstractAnalysisServerIntegrationTest {
20 class Test extends AbstractAnalysisServerIntegrationTest {
21 test_updateContent() { 19 test_updateContent() {
22 String pathname = sourcePath('test.dart'); 20 String pathname = sourcePath('test.dart');
23 String goodText = r''' 21 String goodText = r'''
24 main() { 22 main() {
25 print("Hello, world!"); 23 print("Hello, world!");
26 }'''; 24 }''';
27 String badText = goodText.replaceAll(';', ''); 25 String badText = goodText.replaceAll(';', '');
28 writeFile(pathname, badText); 26 writeFile(pathname, badText);
29 standardAnalysisSetup(); 27 standardAnalysisSetup();
30 return analysisFinished 28 return analysisFinished
(...skipping 30 matching lines...) Expand all
61 {pathname: new RemoveContentOverlay()}); 59 {pathname: new RemoveContentOverlay()});
62 }) 60 })
63 .then((result) => analysisFinished) 61 .then((result) => analysisFinished)
64 .then((_) { 62 .then((_) {
65 // Now there should be errors again, because the contents on disk are no 63 // Now there should be errors again, because the contents on disk are no
66 // longer overridden. 64 // longer overridden.
67 expect(currentAnalysisErrors[pathname], isNotEmpty); 65 expect(currentAnalysisErrors[pathname], isNotEmpty);
68 }); 66 });
69 } 67 }
70 } 68 }
69
70 @reflectiveTest
71 class UpdateContentTest extends AbstractUpdateContentTest {}
72
73 @reflectiveTest
74 class UpdateContentTest_Driver extends AbstractUpdateContentTest {
75 @override
76 bool get enableNewAnalysisDriver => true;
77
78 @failingTest
79 test_updateContent() {
80 // Expected: non-empty
81 // Actual: []
82 return super.test_updateContent();
83 }
84 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698