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

Side by Side Diff: pkg/analysis_server/test/services/correction/assist_test.dart

Issue 2972463003: Use DartEditBuilder to write types and remove dead code (Closed)
Patch Set: Created 3 years, 5 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 import 'dart:async'; 5 import 'dart:async';
6 6
7 import 'package:analysis_server/plugin/edit/assist/assist_core.dart'; 7 import 'package:analysis_server/plugin/edit/assist/assist_core.dart';
8 import 'package:analysis_server/plugin/edit/assist/assist_dart.dart'; 8 import 'package:analysis_server/plugin/edit/assist/assist_dart.dart';
9 import 'package:analysis_server/src/plugin/server_plugin.dart'; 9 import 'package:analysis_server/src/plugin/server_plugin.dart';
10 import 'package:analysis_server/src/services/correction/assist.dart'; 10 import 'package:analysis_server/src/services/correction/assist.dart';
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 class A<T> { 159 class A<T> {
160 main() { 160 main() {
161 for (var item in getValues()) { 161 for (var item in getValues()) {
162 } 162 }
163 } 163 }
164 } 164 }
165 '''); 165 ''');
166 await assertNoAssistAt('var item', DartAssistKind.ADD_TYPE_ANNOTATION); 166 await assertNoAssistAt('var item', DartAssistKind.ADD_TYPE_ANNOTATION);
167 } 167 }
168 168
169 @failingTest
169 test_addTypeAnnotation_BAD_privateType_list() async { 170 test_addTypeAnnotation_BAD_privateType_list() async {
171 // This is now failing because we're suggesting "List" rather than nothing.
172 // Is it really better to produce nothing?
scheglov 2017/07/01 15:31:18 "List" would also be OK.
170 addSource( 173 addSource(
171 '/my_lib.dart', 174 '/my_lib.dart',
172 ''' 175 '''
173 library my_lib; 176 library my_lib;
174 class A {} 177 class A {}
175 class _B extends A {} 178 class _B extends A {}
176 List<_B> getValues() => []; 179 List<_B> getValues() => [];
177 '''); 180 ''');
178 await resolveTestUnit(''' 181 await resolveTestUnit('''
179 import 'my_lib.dart'; 182 import 'my_lib.dart';
(...skipping 4580 matching lines...) Expand 10 before | Expand all | Expand 10 after
4760 4763
4761 @override 4764 @override
4762 final AstProvider astProvider; 4765 final AstProvider astProvider;
4763 4766
4764 @override 4767 @override
4765 final CompilationUnit unit; 4768 final CompilationUnit unit;
4766 4769
4767 _DartAssistContextForValues(this.source, this.selectionOffset, 4770 _DartAssistContextForValues(this.source, this.selectionOffset,
4768 this.selectionLength, this.analysisDriver, this.astProvider, this.unit); 4771 this.selectionLength, this.analysisDriver, this.astProvider, this.unit);
4769 } 4772 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698