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

Side by Side Diff: pkg/analysis_server/test/analysis/notification_highlights_test.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.highlights; 5 library test.analysis.notification.highlights;
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 913 matching lines...) Expand 10 before | Expand all | Expand 10 after
951 void test_toString() { 954 void test_toString() {
952 expect(HighlightRegionType.CLASS.toString(), 'HighlightRegionType.CLASS'); 955 expect(HighlightRegionType.CLASS.toString(), 'HighlightRegionType.CLASS');
953 } 956 }
954 957
955 void test_valueOf_unknown() { 958 void test_valueOf_unknown() {
956 expect(() { 959 expect(() {
957 new HighlightRegionType('no-such-type'); 960 new HighlightRegionType('no-such-type');
958 }, throws); 961 }, throws);
959 } 962 }
960 } 963 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698