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

Unified Diff: pkg/analysis_services/test/correction/assist_test.dart

Issue 447613002: Make Change implementation consistent with the server spec. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 4 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_services/test/correction/assist_test.dart
diff --git a/pkg/analysis_services/test/correction/assist_test.dart b/pkg/analysis_services/test/correction/assist_test.dart
index 73765d0d18a875aa6553da9be24033e38850316a..c4de6865e021b463cbdc00436a99da057b1be43a 100644
--- a/pkg/analysis_services/test/correction/assist_test.dart
+++ b/pkg/analysis_services/test/correction/assist_test.dart
@@ -34,7 +34,7 @@ class AssistProcessorTest extends AbstractSingleUnitTest {
Assist assist;
Change change;
String resultCode;
- LinkedPositionGroup linkedPositionGroup;
+ LinkedEditGroup linkedPositionGroup;
/**
* Asserts that there is an [Assist] of the given [kind] at [offset] which
@@ -60,11 +60,12 @@ class AssistProcessorTest extends AbstractSingleUnitTest {
assertHasAssist(kind, expected);
}
- void assertHasPositionGroup(String id, List<Position> expectedPositions) {
- List<LinkedPositionGroup> linkedPositionGroups =
- change.linkedPositionGroups;
- for (LinkedPositionGroup group in linkedPositionGroups) {
+ void assertHasPositionGroup(String id, int expectedLength,
+ List<Position> expectedPositions) {
+ List<LinkedEditGroup> linkedPositionGroups = change.linkedEditGroups;
+ for (LinkedEditGroup group in linkedPositionGroups) {
if (group.id == id) {
+ expect(group.length, expectedLength);
expect(group.positions, unorderedEquals(expectedPositions));
linkedPositionGroup = group;
return;
@@ -96,8 +97,7 @@ class AssistProcessorTest extends AbstractSingleUnitTest {
Position expectedPosition(String search) {
int offset = resultCode.indexOf(search);
- int length = getLeadingIdentifierLength(search);
- return new Position(testFile, offset, length);
+ return new Position(testFile, offset);
}
List<Position> expectedPositions(List<String> patterns) {
@@ -108,6 +108,13 @@ class AssistProcessorTest extends AbstractSingleUnitTest {
return positions;
}
+ List<LinkedEditSuggestion> expectedSuggestions(LinkedEditSuggestionKind kind,
+ List<String> values) {
+ return values.map((value) {
+ return new LinkedEditSuggestion(kind, value);
+ }).toList();
+ }
+
void setUp() {
super.setUp();
index = createLocalMemoryIndex();
@@ -305,10 +312,13 @@ main() {
}
List<int> readBytes() => <int>[];
''');
- assertHasPositionGroup('NAME', expectedPositions(['readBytes = ']));
+ assertHasPositionGroup('NAME', 9, expectedPositions(['readBytes = ']));
expect(
- linkedPositionGroup.proposals,
- unorderedEquals(['list', 'bytes2', 'readBytes']));
+ linkedPositionGroup.suggestions,
+ unorderedEquals(
+ expectedSuggestions(
+ LinkedEditSuggestionKind.VARIABLE,
+ ['list', 'bytes2', 'readBytes'])));
}
void test_assignToLocalVariable_alreadyAssignment() {
@@ -2027,8 +2037,8 @@ main() {
void _assertHasLinkedPositions(String groupId, List<String> expectedStrings) {
List<Position> expectedPositions = _findResultPositions(expectedStrings);
- List<LinkedPositionGroup> groups = change.linkedPositionGroups;
- for (LinkedPositionGroup group in groups) {
+ List<LinkedEditGroup> groups = change.linkedEditGroups;
+ for (LinkedEditGroup group in groups) {
if (group.id == groupId) {
List<Position> actualPositions = group.positions;
expect(actualPositions, unorderedEquals(expectedPositions));
@@ -2039,10 +2049,10 @@ main() {
}
void _assertHasLinkedProposals(String groupId, List<String> expected) {
- List<LinkedPositionGroup> groups = change.linkedPositionGroups;
- for (LinkedPositionGroup group in groups) {
+ List<LinkedEditGroup> groups = change.linkedEditGroups;
+ for (LinkedEditGroup group in groups) {
if (group.id == groupId) {
- expect(group.proposals, expected);
+ expect(group.suggestions, expected);
return;
}
}
@@ -2053,8 +2063,7 @@ main() {
List<Position> positions = <Position>[];
for (String search in searchStrings) {
int offset = resultCode.indexOf(search);
- int length = getLeadingIdentifierLength(search);
- positions.add(new Position(testFile, offset, length));
+ positions.add(new Position(testFile, offset));
}
return positions;
}
« no previous file with comments | « pkg/analysis_services/lib/src/correction/source_buffer.dart ('k') | pkg/analysis_services/test/correction/change_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698