inplace introduce: do not touch next field type; fix introduce over local variable with initializer; tests

This commit is contained in:
anna
2011-06-29 17:05:22 +04:00
parent 7214e19e0c
commit 04c88b7765
18 changed files with 407 additions and 213 deletions

View File

@@ -15,44 +15,37 @@
*/
package com.intellij.refactoring;
import com.intellij.codeInsight.template.TemplateManager;
import com.intellij.codeInsight.template.impl.TemplateManagerImpl;
import com.intellij.codeInsight.template.impl.TemplateState;
import com.intellij.openapi.editor.Editor;
import com.intellij.openapi.project.Project;
import com.intellij.openapi.util.Pass;
import com.intellij.psi.PsiExpression;
import com.intellij.psi.PsiLocalVariable;
import com.intellij.psi.PsiModifier;
import com.intellij.psi.util.PsiTreeUtil;
import com.intellij.refactoring.introduceField.InplaceIntroduceFieldPopup;
import com.intellij.refactoring.introduceField.IntroduceFieldCentralPanel;
import com.intellij.refactoring.introduce.inplace.AbstractInplaceIntroducer;
import com.intellij.refactoring.introduceField.IntroduceFieldHandler;
import com.intellij.testFramework.LightCodeInsightTestCase;
import org.jetbrains.annotations.NotNull;
/**
* User: anna
* Date: 3/16/11
*/
public class InplaceIntroduceFieldTest extends LightCodeInsightTestCase {
public class InplaceIntroduceFieldTest extends AbstractInplaceIntroduceTest {
private static final String BASE_PATH = "/refactoring/inplaceIntroduceField/";
public void testAnchor() throws Exception {
doTest(new Pass<InplaceIntroduceFieldPopup>() {
doTest(new Pass<AbstractInplaceIntroducer>() {
@Override
public void pass(InplaceIntroduceFieldPopup inplaceIntroduceFieldPopup) {
public void pass(AbstractInplaceIntroducer inplaceIntroduceFieldPopup) {
}
});
}
public void testReplaceAll() throws Exception {
doTest(new Pass<InplaceIntroduceFieldPopup>() {
doTest(new Pass<AbstractInplaceIntroducer>() {
@Override
public void pass(InplaceIntroduceFieldPopup inplaceIntroduceFieldPopup) {
public void pass(AbstractInplaceIntroducer inplaceIntroduceFieldPopup) {
inplaceIntroduceFieldPopup.setReplaceAllOccurrences(true);
}
});
@@ -66,76 +59,17 @@ public class InplaceIntroduceFieldTest extends LightCodeInsightTestCase {
doTestEscape();
}
private void doTestEscape() throws Exception {
String name = getTestName(true);
configureByFile(BASE_PATH + name + ".java");
final boolean enabled = getEditor().getSettings().isVariableInplaceRenameEnabled();
TemplateManagerImpl templateManager = (TemplateManagerImpl)TemplateManager.getInstance(getProject());
try {
templateManager.setTemplateTesting(true);
getEditor().getSettings().setVariableInplaceRenameEnabled(true);
final MyIntroduceFieldHandler introduceFieldHandler = new MyIntroduceFieldHandler();
final PsiExpression expression =
PsiTreeUtil.getParentOfType(getFile().findElementAt(getEditor().getCaretModel().getOffset()), PsiExpression.class);
if (expression != null) {
introduceFieldHandler.invokeImpl(getProject(), expression, getEditor());
} else {
final PsiLocalVariable localVariable =
PsiTreeUtil.getParentOfType(getFile().findElementAt(getEditor().getCaretModel().getOffset()), PsiLocalVariable.class);
assertNotNull(localVariable);
introduceFieldHandler.invokeImpl(getProject(), localVariable, getEditor());
}
TemplateState state = TemplateManagerImpl.getTemplateState(getEditor());
assert state != null;
state.gotoEnd(true);
checkResultByFile(BASE_PATH + name + "_after.java");
}
finally {
myEditor.getSettings().setVariableInplaceRenameEnabled(enabled);
templateManager.setTemplateTesting(false);
InplaceIntroduceFieldPopup.setInitializationPlace(null);
}
@Override
protected String getBasePath() {
return BASE_PATH;
}
private void doTest(final Pass<InplaceIntroduceFieldPopup> pass) throws Exception {
String name = getTestName(true);
configureByFile(BASE_PATH + name + ".java");
final boolean enabled = getEditor().getSettings().isVariableInplaceRenameEnabled();
TemplateManagerImpl templateManager = (TemplateManagerImpl)TemplateManager.getInstance(getProject());
try {
templateManager.setTemplateTesting(true);
getEditor().getSettings().setVariableInplaceRenameEnabled(true);
final MyIntroduceFieldHandler introduceFieldHandler = new MyIntroduceFieldHandler();
final PsiExpression expression =
PsiTreeUtil.getParentOfType(getFile().findElementAt(getEditor().getCaretModel().getOffset()), PsiExpression.class);
if (expression != null) {
introduceFieldHandler.invokeImpl(getProject(), expression, getEditor());
} else {
final PsiLocalVariable localVariable =
PsiTreeUtil.getParentOfType(getFile().findElementAt(getEditor().getCaretModel().getOffset()), PsiLocalVariable.class);
assertNotNull(localVariable);
introduceFieldHandler.invokeImpl(getProject(), localVariable, getEditor());
}
pass.pass(introduceFieldHandler.getInplaceIntroduceFieldPopup());
TemplateState state = TemplateManagerImpl.getTemplateState(getEditor());
assert state != null;
state.gotoEnd(false);
checkResultByFile(BASE_PATH + name + "_after.java");
}
catch (Throwable e) {
e.printStackTrace();
}
finally {
myEditor.getSettings().setVariableInplaceRenameEnabled(enabled);
templateManager.setTemplateTesting(false);
InplaceIntroduceFieldPopup.setInitializationPlace(null);
}
@Override
protected MyIntroduceHandler createIntroduceHandler() {
return new MyIntroduceFieldHandler();
}
private static class MyIntroduceFieldHandler extends IntroduceFieldHandler {
public static class MyIntroduceFieldHandler extends IntroduceFieldHandler implements MyIntroduceHandler {
@Override
public boolean invokeImpl(Project project, @NotNull PsiExpression selectedExpr, Editor editor) {
return super.invokeImpl(project, selectedExpr, editor);