Inlining action: : minor fixes (IDEA-CR-42043)

1. MultiMap.empty() instead of new instance
2. don't process conflicts in test when ignored
This commit is contained in:
Artemiy Sartakov
2019-02-01 15:21:39 +07:00
parent 4b5b6574aa
commit 3971cd929b

View File

@@ -382,7 +382,8 @@ public abstract class BaseRefactoringProcessor implements Runnable {
public static boolean processConflicts(@NotNull Project project, @NotNull MultiMap<PsiElement, String> conflicts) {
if (conflicts.isEmpty()) return true;
if (ApplicationManager.getApplication().isUnitTestMode() && !BaseRefactoringProcessor.ConflictsInTestsException.isTestIgnore()) {
if (ApplicationManager.getApplication().isUnitTestMode()) {
if (BaseRefactoringProcessor.ConflictsInTestsException.isTestIgnore()) return true;
throw new BaseRefactoringProcessor.ConflictsInTestsException(conflicts.values());
}