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

Side by Side Diff: pkg/analysis_server/test/analysis/notification_implemented_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.implemented; 5 library test.analysis.notification.implemented;
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:analysis_server/src/services/index/index.dart'; 12 import 'package:analysis_server/src/services/index/index.dart';
13 import 'package:test/test.dart'; 13 import 'package:test/test.dart';
14 import 'package:test_reflective_loader/test_reflective_loader.dart'; 14 import 'package:test_reflective_loader/test_reflective_loader.dart';
15 15
16 import '../analysis_abstract.dart'; 16 import '../analysis_abstract.dart';
17 17
18 main() { 18 main() {
19 defineReflectiveSuite(() { 19 defineReflectiveSuite(() {
20 defineReflectiveTests(AnalysisNotificationImplementedTest); 20 defineReflectiveTests(AnalysisNotificationImplementedTest);
21 }); 21 });
22 } 22 }
23 23
24 @reflectiveTest 24 @reflectiveTest
25 class AnalysisNotificationImplementedTest extends AbstractAnalysisTest { 25 class AnalysisNotificationImplementedTest extends AbstractAnalysisTest {
26 List<ImplementedClass> implementedClasses; 26 List<ImplementedClass> implementedClasses;
27 List<ImplementedMember> implementedMembers; 27 List<ImplementedMember> implementedMembers;
28 28
29 @override
30 bool get enableNewAnalysisDriver => false;
31
29 /** 32 /**
30 * Validates that there is an [ImplementedClass] at the offset of [search]. 33 * Validates that there is an [ImplementedClass] at the offset of [search].
31 * 34 *
32 * If [length] is not specified explicitly, then length of an identifier 35 * If [length] is not specified explicitly, then length of an identifier
33 * from [search] is used. 36 * from [search] is used.
34 */ 37 */
35 void assertHasImplementedClass(String search, [int length = -1]) { 38 void assertHasImplementedClass(String search, [int length = -1]) {
36 int offset = findOffset(search); 39 int offset = findOffset(search);
37 if (length == -1) { 40 if (length == -1) {
38 length = findIdentifierLength(search); 41 length = findIdentifierLength(search);
(...skipping 391 matching lines...) Expand 10 before | Expand all | Expand 10 after
430 if (times == 0 || implementedClasses != null) { 433 if (times == 0 || implementedClasses != null) {
431 return new Future.value(); 434 return new Future.value();
432 } 435 }
433 return new Future.delayed( 436 return new Future.delayed(
434 new Duration(milliseconds: 1), () => waitForNotification(times - 1)); 437 new Duration(milliseconds: 1), () => waitForNotification(times - 1));
435 } 438 }
436 439
437 return waitForNotification(30000); 440 return waitForNotification(30000);
438 } 441 }
439 } 442 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698