redundant write actions in tests

This commit is contained in:
Anna Kozlova
2016-03-11 08:58:05 +01:00
parent 875a8dff6b
commit de766d6474
3 changed files with 8 additions and 8 deletions
@@ -124,7 +124,12 @@ public class ExtractMethodObjectHandler implements RefactoringActionHandler, Con
PsiDocumentManager.getInstance(project).commitAllDocuments();
if (processor.isCreateInnerClass()) {
processor.moveUsedMethodsToInner();
ApplicationManager.getApplication().runWriteAction(new Runnable() {
@Override
public void run() {
processor.moveUsedMethodsToInner();
}
});
PsiDocumentManager.getInstance(project).commitAllDocuments();
if (editor != null) {
DuplicatesImpl.processDuplicates(extractProcessor, project, editor);
@@ -57,10 +57,7 @@ public class ExtractMethodObjectTest extends LightRefactoringTestCase {
extractProcessor.prepare();
extractProcessor.testPrepare();
ApplicationManager.getApplication().runWriteAction(() -> {
ExtractMethodObjectHandler.run(getProject(), getEditor(), processor, extractProcessor);
});
ExtractMethodObjectHandler.run(getProject(), getEditor(), processor, extractProcessor);
checkResultByFile("/refactoring/extractMethodObject/" + testName + ".java" + ".after");
}
@@ -64,9 +64,7 @@ public class ExtractMethodObjectWithMultipleExitPointsTest extends LightRefactor
processor.setCreateInnerClass(createInnerClass);
ApplicationManager.getApplication().runWriteAction(() -> {
ExtractMethodObjectHandler.run(getProject(), getEditor(), processor, extractProcessor);
});
ExtractMethodObjectHandler.run(getProject(), getEditor(), processor, extractProcessor);
checkResultByFile("/refactoring/extractMethodObject/multipleExitPoints/" + testName + ".java" + ".after");