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

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

Issue 2884593003: Reverse the sense of the flag to make it easier to find test that need to be converted (Closed)
Patch Set: Created 3 years, 7 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 library test.analysis.notification.highlights2; 5 library test.analysis.notification.highlights2;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 8
9 import 'package:analysis_server/protocol/protocol.dart'; 9 import 'package:analysis_server/protocol/protocol.dart';
10 import 'package:analysis_server/protocol/protocol_generated.dart'; 10 import 'package:analysis_server/protocol/protocol_generated.dart';
11 import 'package:analysis_server/src/constants.dart'; 11 import 'package:analysis_server/src/constants.dart';
12 import 'package:test/test.dart'; 12 import 'package:test/test.dart';
13 import 'package:test_reflective_loader/test_reflective_loader.dart'; 13 import 'package:test_reflective_loader/test_reflective_loader.dart';
14 14
15 import '../analysis_abstract.dart'; 15 import '../analysis_abstract.dart';
16 16
17 main() { 17 main() {
18 defineReflectiveSuite(() { 18 defineReflectiveSuite(() {
19 defineReflectiveTests(AnalysisNotificationHighlightsTest); 19 defineReflectiveTests(AnalysisNotificationHighlightsTest);
20 defineReflectiveTests(HighlightTypeTest); 20 defineReflectiveTests(HighlightTypeTest);
21 }); 21 });
22 } 22 }
23 23
24 @reflectiveTest 24 @reflectiveTest
25 class AnalysisNotificationHighlightsTest extends AbstractAnalysisTest { 25 class AnalysisNotificationHighlightsTest extends AbstractAnalysisTest {
26 List<HighlightRegion> regions; 26 List<HighlightRegion> regions;
27 27
28 @override
29 bool get enableNewAnalysisDriver => false;
30
28 void assertHasRawRegion(HighlightRegionType type, int offset, int length) { 31 void assertHasRawRegion(HighlightRegionType type, int offset, int length) {
29 for (HighlightRegion region in regions) { 32 for (HighlightRegion region in regions) {
30 if (region.offset == offset && 33 if (region.offset == offset &&
31 region.length == length && 34 region.length == length &&
32 region.type == type) { 35 region.type == type) {
33 return; 36 return;
34 } 37 }
35 } 38 }
36 fail('Expected to find (offset=$offset; length=$length; type=$type) in\n' 39 fail('Expected to find (offset=$offset; length=$length; type=$type) in\n'
37 '${regions.join('\n')}'); 40 '${regions.join('\n')}');
(...skipping 1064 matching lines...) Expand 10 before | Expand all | Expand 10 after
1102 void test_toString() { 1105 void test_toString() {
1103 expect(HighlightRegionType.CLASS.toString(), 'HighlightRegionType.CLASS'); 1106 expect(HighlightRegionType.CLASS.toString(), 'HighlightRegionType.CLASS');
1104 } 1107 }
1105 1108
1106 void test_valueOf_unknown() { 1109 void test_valueOf_unknown() {
1107 expect(() { 1110 expect(() {
1108 new HighlightRegionType('no-such-type'); 1111 new HighlightRegionType('no-such-type');
1109 }, throws); 1112 }, throws);
1110 } 1113 }
1111 } 1114 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698