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

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

Issue 1787803003: Improve used name relations and fields indexing. (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 1531a405148cd34ffec165346b17f9d4b818ebc9..d2f5205b44b6f44fde79a8f4d9bfd7585d0d82c3 100644
--- a/pkg/analysis_server/test/services/search/search_engine2_test.dart
+++ b/pkg/analysis_server/test/services/search/search_engine2_test.dart
@@ -128,22 +128,22 @@ class A {
}
}
main(A a, p) {
- a.test(); // a-inv-r-q
- a.test = 1; // a-ref-r-q
- a.test += 2; // a-ref-r-q
- print(a.test); // a-ref-r-q
- p.test(); // p-inv-ur-q
- p.test = 1; // p-ref-ur-q
- p.test += 2; // p-ref-ur-q
- print(p.test); // p-ref-ur-q
+ print(a.test); // r
+ a.test = 1; // r
+ a.test += 2; // r
+ a.test(); // r
+ print(p.test); // ur
+ p.test = 1; // ur
+ p.test += 2; // ur
+ p.test(); // ur
}
''');
Element main = findElement('main');
var expected = [
- _expectIdQ(main, MatchKind.REFERENCE, 'test(); // p-inv-ur-q'),
- _expectIdQ(main, MatchKind.REFERENCE, 'test = 1; // p-ref-ur-q'),
- _expectIdQ(main, MatchKind.REFERENCE, 'test += 2; // p-ref-ur-q'),
- _expectIdQ(main, MatchKind.REFERENCE, 'test); // p-ref-ur-q'),
+ _expectIdQU(main, MatchKind.READ, 'test); // ur'),
+ _expectIdQU(main, MatchKind.WRITE, 'test = 1; // ur'),
+ _expectIdQU(main, MatchKind.READ_WRITE, 'test += 2; // ur'),
+ _expectIdQU(main, MatchKind.INVOCATION, 'test(); // ur'),
];
List<SearchMatch> matches =
await searchEngine.searchMemberReferences('test');
@@ -245,7 +245,7 @@ class A {
Element main = findElement('main');
Element fieldParameter = findElement('field', ElementKind.PARAMETER);
var expected = [
- _expectIdQ(fieldParameter, MatchKind.REFERENCE, 'field}'),
+ _expectIdQ(fieldParameter, MatchKind.WRITE, 'field}'),
_expectIdQ(main, MatchKind.REFERENCE, 'field: 1'),
_expectId(main, MatchKind.READ, 'field); // ref-nq'),
_expectIdQ(main, MatchKind.READ, 'field); // ref-q'),
@@ -740,10 +740,16 @@ class NoMatchABCDE {}
}
ExpectedMatch _expectIdQ(Element element, MatchKind kind, String search,
Brian Wilkerson 2016/03/11 18:14:03 nit: it might be good to document what 'Q' and 'QU
- {int length}) {
+ {int length, bool isResolved: true}) {
return _expectId(element, kind, search, isQualified: true, length: length);
}
+ ExpectedMatch _expectIdQU(Element element, MatchKind kind, String search,
+ {int length}) {
+ return _expectId(element, kind, search,
+ isQualified: true, isResolved: false, length: length);
+ }
+
void _indexTestUnit(String code) {
resolveTestUnit(code);
index.indexUnit(testUnit);
« no previous file with comments | « pkg/analysis_server/lib/src/services/search/search_engine_internal2.dart ('k') | pkg/analyzer/lib/src/summary/format.fbs » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698