mirror of
https://gitflic.ru/project/openide/openide.git
synced 2026-10-02 04:35:22 +07:00
test++
This commit is contained in:
Dmitry Jemerov
@@ -0,0 +1,46 @@
|
||||
/*
|
||||
* User: anna
|
||||
* Date: 11-Jun-2009
|
||||
*/
|
||||
package com.intellij.refactoring;
|
||||
|
||||
import com.intellij.openapi.projectRoots.Sdk;
|
||||
import com.intellij.openapi.projectRoots.impl.JavaSdkImpl;
|
||||
import com.intellij.openapi.roots.LanguageLevelProjectExtension;
|
||||
import com.intellij.pom.java.LanguageLevel;
|
||||
import com.intellij.testFramework.LightCodeInsightTestCase;
|
||||
import com.intellij.JavaTestUtil;
|
||||
|
||||
public class ExtractMethod15Test extends LightCodeInsightTestCase {
|
||||
private LanguageLevel myPreviousLanguageLevel;
|
||||
private static final String BASE_PATH = "/refactoring/extractMethod15/";
|
||||
|
||||
@Override
|
||||
protected String getTestDataPath() {
|
||||
return JavaTestUtil.getJavaTestDataPath();
|
||||
}
|
||||
|
||||
protected void setUp() throws Exception {
|
||||
super.setUp();
|
||||
myPreviousLanguageLevel = LanguageLevelProjectExtension.getInstance(getJavaFacade().getProject()).getLanguageLevel();
|
||||
LanguageLevelProjectExtension.getInstance(getJavaFacade().getProject()).setLanguageLevel(LanguageLevel.JDK_1_5);
|
||||
}
|
||||
|
||||
protected void tearDown() throws Exception {
|
||||
LanguageLevelProjectExtension.getInstance(getJavaFacade().getProject()).setLanguageLevel(myPreviousLanguageLevel);
|
||||
super.tearDown();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithMultOccurences() throws Exception {
|
||||
configureByFile(BASE_PATH + getTestName(false) + ".java");
|
||||
boolean success = ExtractMethodTest.performExtractMethod(true, true, getEditor(), getFile(), getProject());
|
||||
assertTrue(success);
|
||||
checkResultByFile(BASE_PATH + getTestName(false) + "_after.java");
|
||||
}
|
||||
|
||||
|
||||
@Override
|
||||
protected Sdk getProjectJDK() {
|
||||
return JavaSdkImpl.getMockJdk15("java 1.5");
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,493 @@
|
||||
package com.intellij.refactoring;
|
||||
|
||||
import com.intellij.codeInsight.CodeInsightUtil;
|
||||
import com.intellij.openapi.editor.Editor;
|
||||
import com.intellij.openapi.project.Project;
|
||||
import com.intellij.psi.PsiElement;
|
||||
import com.intellij.psi.PsiExpression;
|
||||
import com.intellij.psi.PsiFile;
|
||||
import com.intellij.psi.codeStyle.CodeStyleSettings;
|
||||
import com.intellij.psi.codeStyle.CodeStyleSettingsManager;
|
||||
import com.intellij.refactoring.extractMethod.ExtractMethodProcessor;
|
||||
import com.intellij.refactoring.extractMethod.PrepareFailedException;
|
||||
import com.intellij.refactoring.util.duplicates.Match;
|
||||
import com.intellij.testFramework.LightCodeInsightTestCase;
|
||||
import com.intellij.util.IncorrectOperationException;
|
||||
import com.intellij.JavaTestUtil;
|
||||
import org.jetbrains.annotations.NonNls;
|
||||
|
||||
import java.util.List;
|
||||
|
||||
public class ExtractMethodTest extends LightCodeInsightTestCase {
|
||||
@NonNls private static final String BASE_PATH = "/refactoring/extractMethod/";
|
||||
private boolean myCatchOnNewLine = true;
|
||||
|
||||
@Override
|
||||
protected String getTestDataPath() {
|
||||
return JavaTestUtil.getJavaTestDataPath();
|
||||
}
|
||||
|
||||
|
||||
public void testExitPoints1() throws Exception {
|
||||
doExitPointsTest(true);
|
||||
}
|
||||
|
||||
public void testExitPoints2() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testExitPoints3() throws Exception {
|
||||
doExitPointsTest(true);
|
||||
}
|
||||
|
||||
public void testExitPoints4() throws Exception {
|
||||
doExitPointsTest(false);
|
||||
}
|
||||
|
||||
public void testExitPointsInsideLoop() throws Exception {
|
||||
doExitPointsTest(true);
|
||||
}
|
||||
|
||||
public void testExitPoints5() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testExitPoints6() throws Exception {
|
||||
doExitPointsTest(false);
|
||||
}
|
||||
|
||||
public void testExitPoints7() throws Exception {
|
||||
doExitPointsTest(false);
|
||||
}
|
||||
|
||||
public void testExitPoints8() throws Exception {
|
||||
doExitPointsTest(false);
|
||||
}
|
||||
|
||||
public void testExitPoints9() throws Exception {
|
||||
doExitPointsTest(false);
|
||||
}
|
||||
|
||||
public void testBooleanExpression() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testScr6241() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testScr7091() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testScr10464() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testScr9852() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testUseVarAfterTry() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testOneBranchAssignment() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testExtractFromCodeBlock() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testUnusedInitializedVar() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testTryFinally() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testFinally() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testExtractFromAnonymous() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testSCR12245() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testSCR15815() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testSCR27887() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testSCR28427() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testTryFinallyInsideFor() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testExtractFromTryFinally() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void _testExtractFromTryFinally2() throws Exception { // IDEADEV-11844
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testLesyaBug() throws Exception {
|
||||
myCatchOnNewLine = false;
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testForEach() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testAnonInner() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testFinalParamUsedInsideAnon() throws Exception {
|
||||
CodeStyleSettingsManager.getSettings(getProject()).GENERATE_FINAL_PARAMETERS = false;
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testNonFinalWritableParam() throws Exception {
|
||||
CodeStyleSettingsManager.getSettings(getProject()).GENERATE_FINAL_PARAMETERS = true;
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithContinue() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithContinueNoReturn() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithStaticInitializer() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testExpressionDuplicates() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicates() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicates2() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicates3() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicates4() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicates5() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithOutputValue() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithOutputValue1() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithMultExitPoints() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithReturn() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithReturn2() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testCodeDuplicatesWithComments() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testSCR32924() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testFinalOutputVar() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testIdeaDev2291() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testOxfordBug() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testIDEADEV33368() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testGuardMethodDuplicates() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testGuardMethodDuplicates1() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testInstanceMethodDuplicatesInStaticContext() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
|
||||
public void testLValueNotDuplicate() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
protected void doDuplicatesTest() throws Exception {
|
||||
doTest(true);
|
||||
}
|
||||
|
||||
public void testExtractFromFinally() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testNoShortCircuit() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testIDEADEV11748() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testIDEADEV11848() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testIDEADEV11036() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testLocalClass() throws Exception {
|
||||
doPrepareErrorTest("Cannot extract method because the selected code fragment uses local classes defined outside of the fragment");
|
||||
}
|
||||
|
||||
public void testLocalClassUsage() throws Exception {
|
||||
doPrepareErrorTest("Cannot extract method because the selected code fragment defines local classes used outside of the fragment");
|
||||
}
|
||||
|
||||
public void testStaticImport() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testThisCall() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testChainedConstructor() throws Exception {
|
||||
doChainedConstructorTest(false);
|
||||
}
|
||||
|
||||
public void testChainedConstructorDuplicates() throws Exception {
|
||||
doChainedConstructorTest(true);
|
||||
}
|
||||
|
||||
public void testChainedConstructorInvalidDuplicates() throws Exception {
|
||||
doChainedConstructorTest(true);
|
||||
}
|
||||
|
||||
public void testReturnFromTry() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testForceBraces() throws Exception {
|
||||
final CodeStyleSettings settings = CodeStyleSettingsManager.getSettings(getProject());
|
||||
int old = settings.IF_BRACE_FORCE;
|
||||
settings.IF_BRACE_FORCE = CodeStyleSettings.FORCE_BRACES_ALWAYS;
|
||||
try {
|
||||
doTest();
|
||||
}
|
||||
finally {
|
||||
settings.IF_BRACE_FORCE = old;
|
||||
}
|
||||
}
|
||||
|
||||
public void testConstantConditionsAffectingControlFlow() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
public void testNotInitializedInsideFinally() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testGenericsParameters() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testParamsUsedInLocalClass() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
private void doChainedConstructorTest(final boolean replaceAllDuplicates) throws Exception {
|
||||
configureByFile(BASE_PATH + getTestName(false) + ".java");
|
||||
boolean success = performExtractMethod(true, replaceAllDuplicates, getEditor(), getFile(), getProject(), true);
|
||||
assertTrue(success);
|
||||
checkResultByFile(BASE_PATH + getTestName(false) + "_after.java");
|
||||
}
|
||||
|
||||
public void testReassignedVarAfterCall() throws Exception {
|
||||
final CodeStyleSettings settings = CodeStyleSettingsManager.getSettings(getProject());
|
||||
boolean oldGenerateFinalLocals = settings.GENERATE_FINAL_LOCALS;
|
||||
try {
|
||||
settings.GENERATE_FINAL_LOCALS = true;
|
||||
doTest();
|
||||
}
|
||||
finally {
|
||||
settings.GENERATE_FINAL_LOCALS = oldGenerateFinalLocals;
|
||||
}
|
||||
}
|
||||
|
||||
public void testNullableCheck() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testNullableCheck1() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testNullableCheckVoid() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testSimpleArrayAccess() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testArrayAccess() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testArrayAccess1() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testArrayAccessWithDuplicates() throws Exception {
|
||||
doDuplicatesTest();
|
||||
}
|
||||
|
||||
public void testVerboseArrayAccess() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testReturnStatementFolding() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testWriteArrayAccess() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
public void testShortCircuit() throws Exception {
|
||||
doTest();
|
||||
}
|
||||
|
||||
private void doPrepareErrorTest(final String expectedMessage) throws Exception {
|
||||
String expectedError = null;
|
||||
try {
|
||||
doExitPointsTest(false);
|
||||
}
|
||||
catch(PrepareFailedException ex) {
|
||||
expectedError = ex.getMessage();
|
||||
}
|
||||
assertEquals(expectedMessage, expectedError);
|
||||
}
|
||||
|
||||
private void doExitPointsTest(boolean shouldSucceed) throws Exception {
|
||||
String fileName = getTestName(false) + ".java";
|
||||
configureByFile(BASE_PATH + fileName);
|
||||
boolean success = performAction(false, false);
|
||||
assertEquals(shouldSucceed, success);
|
||||
}
|
||||
|
||||
void doTest() throws Exception {
|
||||
final CodeStyleSettings settings = CodeStyleSettingsManager.getSettings(getProject());
|
||||
settings.ELSE_ON_NEW_LINE = true;
|
||||
settings.CATCH_ON_NEW_LINE = myCatchOnNewLine;
|
||||
doTest(true);
|
||||
}
|
||||
|
||||
private void doTest(boolean duplicates) throws Exception {
|
||||
configureByFile(BASE_PATH + getTestName(false) + ".java");
|
||||
boolean success = performAction(true, duplicates);
|
||||
assertTrue(success);
|
||||
checkResultByFile(BASE_PATH + getTestName(false) + "_after.java");
|
||||
}
|
||||
|
||||
private static boolean performAction(boolean doRefactor, boolean replaceAllDuplicates) throws Exception {
|
||||
return performExtractMethod(doRefactor, replaceAllDuplicates, getEditor(), getFile(), getProject());
|
||||
}
|
||||
|
||||
public static boolean performExtractMethod(boolean doRefactor, boolean replaceAllDuplicates, Editor editor, PsiFile file, Project project)
|
||||
throws PrepareFailedException, IncorrectOperationException {
|
||||
return performExtractMethod(doRefactor, replaceAllDuplicates, editor, file, project, false);
|
||||
}
|
||||
|
||||
public static boolean performExtractMethod(boolean doRefactor, boolean replaceAllDuplicates, Editor editor, PsiFile file, Project project,
|
||||
final boolean extractChainedConstructor)
|
||||
throws PrepareFailedException, IncorrectOperationException {
|
||||
int startOffset = editor.getSelectionModel().getSelectionStart();
|
||||
int endOffset = editor.getSelectionModel().getSelectionEnd();
|
||||
|
||||
PsiElement[] elements;
|
||||
PsiExpression expr = CodeInsightUtil.findExpressionInRange(file, startOffset, endOffset);
|
||||
if (expr != null) {
|
||||
elements = new PsiElement[]{expr};
|
||||
}
|
||||
else {
|
||||
elements = CodeInsightUtil.findStatementsInRange(file, startOffset, endOffset);
|
||||
}
|
||||
assertTrue(elements.length > 0);
|
||||
|
||||
final ExtractMethodProcessor processor =
|
||||
new ExtractMethodProcessor(project, editor, elements, null, "Extract Method", "newMethod", null);
|
||||
processor.setShowErrorDialogs(false);
|
||||
processor.setChainedConstructor(extractChainedConstructor);
|
||||
|
||||
if (!processor.prepare()) {
|
||||
return false;
|
||||
}
|
||||
|
||||
if (doRefactor) {
|
||||
processor.testRun();
|
||||
}
|
||||
|
||||
if (replaceAllDuplicates) {
|
||||
final List<Match> duplicates = processor.getDuplicates();
|
||||
for (final Match match : duplicates) {
|
||||
if (!match.getMatchStart().isValid() || !match.getMatchEnd().isValid()) continue;
|
||||
processor.processMatch(match);
|
||||
}
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user