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

Unified Diff: editor/tools/plugins/com.google.dart.server_test/src/com/google/dart/server/internal/remote/RemoteAnalysisServerImplTest.java

Issue 551963002: Add feedback for INLINE_METHOD refactoring. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 3 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: editor/tools/plugins/com.google.dart.server_test/src/com/google/dart/server/internal/remote/RemoteAnalysisServerImplTest.java
diff --git a/editor/tools/plugins/com.google.dart.server_test/src/com/google/dart/server/internal/remote/RemoteAnalysisServerImplTest.java b/editor/tools/plugins/com.google.dart.server_test/src/com/google/dart/server/internal/remote/RemoteAnalysisServerImplTest.java
index 748935b173c52d70263d7eb7efbfb2a8b27d235e..de85fd56f2e505020342e157f66a391f7f07d198 100644
--- a/editor/tools/plugins/com.google.dart.server_test/src/com/google/dart/server/internal/remote/RemoteAnalysisServerImplTest.java
+++ b/editor/tools/plugins/com.google.dart.server_test/src/com/google/dart/server/internal/remote/RemoteAnalysisServerImplTest.java
@@ -50,6 +50,7 @@ import com.google.dart.server.generated.types.HighlightRegion;
import com.google.dart.server.generated.types.HighlightRegionType;
import com.google.dart.server.generated.types.HoverInformation;
import com.google.dart.server.generated.types.InlineLocalVariableFeedback;
+import com.google.dart.server.generated.types.InlineMethodFeedback;
import com.google.dart.server.generated.types.InlineMethodOptions;
import com.google.dart.server.generated.types.Location;
import com.google.dart.server.generated.types.NavigationRegion;
@@ -1800,7 +1801,7 @@ public class RemoteAnalysisServerImplTest extends AbstractRemoteServerTest {
" }",
" ],",
" 'feedback': {",
- " 'name': myVar,",
+ " 'name': 'myVar',",
" 'occurrences': 3",
" },",
" 'change': " + getSourceChangeJson() + ",",
@@ -1815,6 +1816,52 @@ public class RemoteAnalysisServerImplTest extends AbstractRemoteServerTest {
assertEquals(3, feedback.getOccurrences());
}
+ public void test_edit_getRefactoring_response_feedback_inlineMethod() throws Exception {
+ final RefactoringFeedback[] feedbackArray = {null};
+ RefactoringOptions options = null;
+ server.edit_getRefactoring(
+ RefactoringKind.INLINE_METHOD,
+ "file1.dart",
+ 1,
+ 2,
+ false,
+ options,
+ new GetRefactoringConsumer() {
+ @Override
+ public void computedRefactorings(List<RefactoringProblem> problems,
+ RefactoringFeedback feedback, SourceChange change, List<String> potentialEdits) {
+ feedbackArray[0] = feedback;
+ }
+ });
+
+ putResponse(//
+ "{",
+ " 'id': '0',",
+ " 'result': {",
+ " 'problems': [",
+ " {",
+ " 'severity': 'INFO',",
+ " 'message': 'message1'",
+ " }",
+ " ],",
+ " 'feedback': {",
+ " 'className': 'myClassName',",
+ " 'methodName': 'myMethodName',",
+ " 'isDeclaration': true",
+ " },",
+ " 'change': " + getSourceChangeJson() + ",",
+ " 'potentialEdits': ['one']",
+ " }",
+ "}");
+ server.test_waitForWorkerComplete();
+
+ // assertions on 'feedback'
+ InlineMethodFeedback feedback = (InlineMethodFeedback) feedbackArray[0];
+ assertEquals("myClassName", feedback.getClassName());
+ assertEquals("myMethodName", feedback.getMethodName());
+ assertEquals(true, feedback.isDeclaration());
+ }
+
public void test_edit_getRefactoring_response_feedback_rename() throws Exception {
final Object[] feedbackArray = {null};
RefactoringOptions options = null;

Powered by Google App Engine
This is Rietveld 408576698