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

Unified Diff: pkg/analysis_server/test/services/search/search_engine2_test.dart

Issue 1778363003: Improve search for constructors and import elements. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: pkg/analysis_server/test/services/search/search_engine2_test.dart
diff --git a/pkg/analysis_server/test/services/search/search_engine2_test.dart b/pkg/analysis_server/test/services/search/search_engine2_test.dart
index 4df802da79d47475aae30c81e91344c9601866c7..e3ecab7c8baad75597ac3b6da89248cad4dd3872 100644
--- a/pkg/analysis_server/test/services/search/search_engine2_test.dart
+++ b/pkg/analysis_server/test/services/search/search_engine2_test.dart
@@ -257,6 +257,36 @@ class A {
await _verifyReferences(element, expected);
}
+ test_searchReferences_FieldElement_synthetic() async {
+ _indexTestUnit('''
+class A {
+ get field => null;
+ set field(x) {}
+ main() {
+ // getter
+ print(field); // ref-nq
+ print(this.field); // ref-q
+ field(); // inv-nq
+ this.field(); // inv-q
+ // setter
+ field = 2; // ref-nq;
+ this.field = 3; // ref-q;
+ }
+}
+''');
+ FieldElement element = findElement('field', ElementKind.FIELD);
+ Element main = findElement('main');
+ var expected = [
+ _expectId(main, MatchKind.READ, 'field); // ref-nq'),
+ _expectIdQ(main, MatchKind.READ, 'field); // ref-q'),
+ _expectId(main, MatchKind.INVOCATION, 'field(); // inv-nq'),
+ _expectIdQ(main, MatchKind.INVOCATION, 'field(); // inv-q'),
+ _expectId(main, MatchKind.WRITE, 'field = 2; // ref-nq'),
+ _expectIdQ(main, MatchKind.WRITE, 'field = 3; // ref-q'),
+ ];
+ await _verifyReferences(element, expected);
+ }
+
test_searchReferences_FunctionElement() async {
_indexTestUnit('''
test() {}
@@ -311,7 +341,7 @@ main() {
test_searchReferences_ImportElement_noPrefix() async {
_indexTestUnit('''
-import 'dart:math';
+import 'dart:math' show max, PI, Random;
main() {
print(PI);
print(new Random());
@@ -334,7 +364,7 @@ Random bar() => null;
test_searchReferences_ImportElement_withPrefix() async {
_indexTestUnit('''
-import 'dart:math' as math;
+import 'dart:math' as math show max, PI, Random;
main() {
print(math.PI);
print(new math.Random());
@@ -712,11 +742,6 @@ class NoMatchABCDE {}
return _expectId(element, kind, search, isQualified: true, length: length);
}
-// ExpectedMatch _expectIdU(Element element, MatchKind kind, String search) {
-// return _expectId(element, kind, search,
-// isQualified: true, isResolved: false);
-// }
-
void _indexTestUnit(String code) {
resolveTestUnit(code);
index.indexUnit(testUnit);
@@ -726,6 +751,7 @@ class NoMatchABCDE {}
Element element, List<ExpectedMatch> expectedMatches) async {
List<SearchMatch> matches = await searchEngine.searchReferences(element);
_assertMatches(matches, expectedMatches);
+ expect(matches, hasLength(expectedMatches.length));
}
static void _assertMatches(

Powered by Google App Engine
This is Rietveld 408576698