Extract Function: Infer parameter type candidates

#KT-4940 Fixed

core/descriptors/src/org/jetbrains/jet/lang/types/TypeUtils.java

@@ -376,9 +376,13 @@ public class TypeUtils {
     }
 
     private static void collectImmediateSupertypes(@NotNull JetType type, @NotNull Collection<JetType> result) {
+        boolean isNullable = type.isNullable();
         TypeSubstitutor substitutor = TypeSubstitutor.create(type);
         for (JetType supertype : type.getConstructor().getSupertypes()) {
-            result.add(substitutor.substitute(supertype, Variance.INVARIANT));
+            JetType substitutedType = substitutor.substitute(supertype, Variance.INVARIANT);
+            if (substitutedType != null) {
+                result.add(makeNullableIfNeeded(substitutedType, isNullable));
+            }
         }
     }
 

idea/src/org/jetbrains/jet/plugin/refactoring/extractFunction/ExtractKotlinFunctionHandler.kt

@@ -59,7 +59,8 @@ public class ExtractKotlinFunctionHandler(public val allContainersEnabled: Boole
             editor: Editor,
             file: JetFile,
             elements: List<PsiElement>,
-            targetSibling: PsiElement
+            targetSibling: PsiElement,
+            preprocessor: ((ExtractionDescriptor) -> Unit)? = null
     ) {
         val project = file.getProject()
 
@@ -83,7 +84,7 @@ public class ExtractKotlinFunctionHandler(public val allContainersEnabled: Boole
 
                         dialog.getCurrentDescriptor()
                     }
-
+            preprocessor?.invoke(descriptor)
             project.executeWriteCommand(EXTRACT_FUNCTION) { descriptor.generateFunction() }
         }
 

idea/src/org/jetbrains/jet/plugin/refactoring/extractFunction/ExtractionDescriptor.kt

@@ -31,19 +31,21 @@ import org.jetbrains.jet.lang.psi.psiUtil.replaced
 import org.jetbrains.jet.lang.psi.JetQualifiedExpression
 import org.jetbrains.jet.lang.psi.JetTypeParameter
 import org.jetbrains.jet.lang.psi.JetTypeConstraint
-import kotlin.properties.Delegates
 import org.jetbrains.jet.plugin.refactoring.extractFunction.AnalysisResult.Status
 import org.jetbrains.jet.plugin.refactoring.JetRefactoringBundle
 import org.jetbrains.jet.plugin.refactoring.extractFunction.AnalysisResult.ErrorMessage
 
-data class Parameter(
-        val argumentText: String,
-        val name: String,
-        var mirrorVarName: String?,
-        val parameterType: JetType,
-        val receiverCandidate: Boolean
-) {
+trait Parameter {
+    val argumentText: String
+    val name: String
+    val mirrorVarName: String?
+    val parameterType: JetType
+    val parameterTypeCandidates: List<JetType>
+    val receiverCandidate: Boolean
+
     val nameForRef: String get() = mirrorVarName ?: name
+
+    fun copy(name: String): Parameter
 }
 
 data class TypeParameter(

idea/src/org/jetbrains/jet/plugin/refactoring/extractFunction/extractFunctionUtils.kt

@@ -38,7 +38,6 @@ import org.jetbrains.jet.lang.psi.JetPsiFactory.FunctionBuilder
 import org.jetbrains.jet.plugin.refactoring.JetNameValidatorImpl
 import org.jetbrains.jet.plugin.codeInsight.DescriptorToDeclarationUtil
 import org.jetbrains.jet.plugin.imports.canBeReferencedViaImport
-import org.jetbrains.jet.lang.types.checker.JetTypeChecker
 import org.jetbrains.jet.lang.resolve.DescriptorUtils
 import com.intellij.psi.PsiNamedElement
 import org.jetbrains.jet.lang.descriptors.impl.LocalVariableDescriptor
@@ -56,14 +55,7 @@ import org.jetbrains.jet.lang.diagnostics.Errors
 import org.jetbrains.jet.lang.psi.psiUtil.replaced
 import org.jetbrains.jet.plugin.refactoring.extractFunction.AnalysisResult.Status
 import org.jetbrains.jet.plugin.refactoring.extractFunction.AnalysisResult.ErrorMessage
-import org.jetbrains.jet.lang.cfg.pseudocode.instructions.jumps.*
 import org.jetbrains.jet.lang.cfg.pseudocode.instructions.special.LocalFunctionDeclarationInstruction
-import org.jetbrains.jet.lang.cfg.pseudocode.instructions.eval.WriteValueInstruction
-import org.jetbrains.jet.lang.cfg.pseudocode.instructions.Instruction
-import org.jetbrains.jet.lang.cfg.pseudocode.instructions.eval.CallInstruction
-import org.jetbrains.jet.lang.cfg.pseudocode.instructions.JetElementInstruction
-import org.jetbrains.jet.lang.cfg.pseudocode.instructions.eval.OperationInstruction
-import org.jetbrains.jet.lang.cfg.pseudocode.instructions.eval.ReadValueInstruction
 import org.jetbrains.jet.lang.cfg.pseudocode.instructions.*
 import org.jetbrains.jet.lang.cfg.pseudocode.instructions.eval.*
 import org.jetbrains.jet.lang.cfg.pseudocode.instructions.jumps.*
@@ -72,6 +64,7 @@ import org.jetbrains.jet.lang.cfg.pseudocodeTraverser.traverse
 import org.jetbrains.jet.lang.cfg.pseudocodeTraverser.TraversalOrder
 import org.jetbrains.jet.lang.resolve.bindingContextUtil.getTargetFunctionDescriptor
 import com.intellij.psi.PsiWhiteSpace
+import org.jetbrains.jet.lang.resolve.OverridingUtil
 
 private val DEFAULT_FUNCTION_NAME = "myFun"
 private val DEFAULT_RETURN_TYPE = KotlinBuiltIns.getInstance().getUnitType()
@@ -325,13 +318,57 @@ private fun JetType.processTypeIfExtractable(
     }
 }
 
-private fun ExtractionData.inferParametersInfo(commonParent: PsiElement,
-                                               localInstructions: List<Instruction>,
-                                               bindingContext: BindingContext,
-                                               replacementMap: MutableMap<Int, Replacement>,
-                                               parameters: MutableSet<Parameter>,
-                                               typeParameters: MutableSet<TypeParameter>,
-                                               nonDenotableTypes: MutableSet<JetType>): ErrorMessage? {
+private class MutableParameter(
+        override val argumentText: String,
+        override val name: String,
+        override val mirrorVarName: String?,
+        override val receiverCandidate: Boolean
+): Parameter {
+    // All modifications happen in the same thread
+    private var writable: Boolean = true
+    private val defaultTypes = HashSet<JetType>()
+    private val typePredicates = HashSet<TypePredicate>()
+
+    fun addDefaultType(jetType: JetType) {
+        assert(writable, "Can't add type to non-writable parameter $name")
+        defaultTypes.add(jetType)
+    }
+
+    fun addTypePredicate(predicate: TypePredicate) {
+        assert(writable, "Can't add type predicate to non-writable parameter $name")
+        typePredicates.add(predicate)
+    }
+
+    override val parameterTypeCandidates: List<JetType> by Delegates.lazy {
+        writable = false
+        listOf(parameterType) + TypeUtils.getAllSupertypes(parameterType).filter(and(typePredicates))
+    }
+
+    override val parameterType: JetType by Delegates.lazy {
+        writable = false
+        CommonSupertypes.commonSupertype(defaultTypes)
+    }
+
+    override fun copy(name: String): Parameter = DelegatingParameter(this, name)
+}
+
+private class DelegatingParameter(
+        val original: Parameter,
+        override val name: String
+): Parameter by original {
+    override fun copy(name: String): Parameter = DelegatingParameter(original, name)
+}
+
+private fun ExtractionData.inferParametersInfo(
+        commonParent: PsiElement,
+        pseudocode: Pseudocode,
+        localInstructions: List<Instruction>,
+        bindingContext: BindingContext,
+        replacementMap: MutableMap<Int, Replacement>,
+        parameters: MutableSet<Parameter>,
+        typeParameters: MutableSet<TypeParameter>,
+        nonDenotableTypes: MutableSet<JetType>
+): ErrorMessage? {
     val varNameValidator = JetNameValidatorImpl(
             commonParent.getParentByType(javaClass<JetExpression>()),
             originalElements.first,
@@ -339,7 +376,9 @@ private fun ExtractionData.inferParametersInfo(commonParent: PsiElement,
     )
     val modifiedVarDescriptors = localInstructions.getModifiedVarDescriptors(bindingContext)
 
-    val extractedDescriptorToParameter = HashMap<DeclarationDescriptor, Parameter>()
+    val extractedDescriptorToParameter = HashMap<DeclarationDescriptor, MutableParameter>()
+
+    val valueUsageMap = pseudocode.collectValueUsages()
 
     for (refInfo in getBrokenReferencesInfo(createTemporaryCodeBlock())) {
         val (originalRef, originalDeclaration, originalDescriptor, resolvedCall) = refInfo.resolveResult
@@ -397,57 +436,40 @@ private fun ExtractionData.inferParametersInfo(commonParent: PsiElement,
             if (extractParameter) {
                 val parameterType = when {
                     receiver.exists() -> receiver.getType()
-                    else -> bindingContext[BindingContext.EXPRESSION_TYPE, originalRef] ?: DEFAULT_PARAMETER_TYPE
+                    else -> bindingContext[BindingContext.AUTOCAST, originalRef]
+                            ?: bindingContext[BindingContext.EXPRESSION_TYPE, originalRef]
+                            ?: DEFAULT_PARAMETER_TYPE
                 }
 
                 if (!parameterType.processTypeIfExtractable(bindingContext, typeParameters, nonDenotableTypes)) continue
 
-                val existingParameter = extractedDescriptorToParameter[descriptorToExtract]
-                val parameter: Parameter =
-                        if (existingParameter != null) {
-                            if (!JetTypeChecker.INSTANCE.equalTypes(existingParameter.parameterType, parameterType)) {
-                                val newParameter = existingParameter.copy(
-                                        parameterType = CommonSupertypes.commonSupertype(listOf(existingParameter.parameterType, parameterType))
-                                )
+                val parameterTypePredicate =
+                        pseudocode.getElementValue(originalRef)?.let { getExpectedTypePredicate(it, valueUsageMap, bindingContext) } ?: AllTypes
 
-                                extractedDescriptorToParameter[descriptorToExtract] = newParameter
-
-                                for ((offset, replacement) in replacementMap) {
-                                    if (replacement is ParameterReplacement && replacement.parameter == existingParameter) {
-                                        replacementMap[offset] = replacement.copy(newParameter)
-                                    }
-                                }
-
-                                newParameter
+                val parameter = extractedDescriptorToParameter.getOrPut(descriptorToExtract) {
+                    val parameterName =
+                            if (extractThis) {
+                                JetNameSuggester.suggestNames(parameterType, varNameValidator, null).first()
                             }
-                            else existingParameter
-                        }
-                        else {
-                            val parameterName =
-                                    if (extractThis) {
-                                        JetNameSuggester.suggestNames(parameterType, varNameValidator, null).first()
-                                    }
-                                    else originalDeclaration.getName()!!
+                            else originalDeclaration.getName()!!
 
-                            val mirrorVarName = if (descriptorToExtract in modifiedVarDescriptors)
-                                varNameValidator.validateName(parameterName)!!
-                            else null
+                    val mirrorVarName =
+                            if (descriptorToExtract in modifiedVarDescriptors) varNameValidator.validateName(parameterName)!! else null
 
-                            val argumentText =
-                                    if (hasThisReceiver && extractThis)
-                                        "this@${parameterType.getConstructor().getDeclarationDescriptor()!!.getName().asString()}"
-                                    else
-                                        (thisExpr ?: ref).getText() ?: throw AssertionError("'this' reference shouldn't be empty: code fragment = ${getCodeFragmentText()}")
+                    val argumentText =
+                            if (hasThisReceiver && extractThis)
+                                "this@${parameterType.getConstructor().getDeclarationDescriptor()!!.getName().asString()}"
+                            else
+                                (thisExpr ?: ref).getText() ?: throw AssertionError("'this' reference shouldn't be empty: code fragment = ${getCodeFragmentText()}")
 
-                            val parameter = Parameter(argumentText, parameterName, mirrorVarName, parameterType, extractThis)
+                    MutableParameter(argumentText, parameterName, mirrorVarName, extractThis)
+                }
 
-                            extractedDescriptorToParameter[descriptorToExtract] = parameter
-
-                            parameter
-                        }
+                parameter.addDefaultType(parameterType)
+                parameter.addTypePredicate(parameterTypePredicate)
 
                 replacementMap[refInfo.offsetInBody] =
-                if (hasThisReceiver && extractThis) AddPrefixReplacement(parameter) else RenameReplacement(parameter)
+                        if (hasThisReceiver && extractThis) AddPrefixReplacement(parameter) else RenameReplacement(parameter)
             }
         }
     }
@@ -543,7 +565,9 @@ fun ExtractionData.performAnalysis(): AnalysisResult {
     val parameters = HashSet<Parameter>()
     val typeParameters = HashSet<TypeParameter>()
     val nonDenotableTypes = HashSet<JetType>()
-    val parameterError = inferParametersInfo(commonParent, localInstructions, bindingContext, replacementMap, parameters, typeParameters, nonDenotableTypes)
+    val parameterError = inferParametersInfo(
+            commonParent, pseudocode, localInstructions, bindingContext, replacementMap, parameters, typeParameters, nonDenotableTypes
+    )
     if (parameterError != null) {
         return AnalysisResult(null, Status.CRITICAL_ERROR, listOf(parameterError))
     }
@@ -624,7 +648,7 @@ fun ExtractionDescriptor.validate(): ExtractionDescriptorWithConflicts {
         if (diagnostics.any { it.getFactory() == Errors.UNRESOLVED_REFERENCE }
         || (currentDescriptor != null
         && !ErrorUtils.isError(currentDescriptor)
-        && !compareDescriptors(currentDescriptor, resolveResult.descriptor))) {
+        && !comparePossiblyOverridingDescriptors(currentDescriptor, resolveResult.descriptor))) {
             conflicts.putValue(
                     currentRefExpr,
                     JetRefactoringBundle.message(
@@ -649,6 +673,15 @@ fun ExtractionDescriptor.validate(): ExtractionDescriptorWithConflicts {
     return ExtractionDescriptorWithConflicts(this, conflicts)
 }
 
+private fun comparePossiblyOverridingDescriptors(currentDescriptor: DeclarationDescriptor?, originalDescriptor: DeclarationDescriptor?): Boolean {
+    if (compareDescriptors(currentDescriptor, originalDescriptor)) return true
+    if (originalDescriptor is CallableDescriptor) {
+        if (!OverridingUtil.traverseOverridenDescriptors(originalDescriptor) { !compareDescriptors(currentDescriptor, it) }) return true
+    }
+
+    return false
+}
+
 fun ExtractionDescriptor.getFunctionText(
         withBody: Boolean = true,
         descriptorRenderer: DescriptorRenderer = DescriptorRenderer.FQ_NAMES_IN_TYPES

idea/src/org/jetbrains/jet/plugin/refactoring/extractFunction/ui/KotlinParameterTablePanel.java

@@ -70,17 +70,11 @@ public class KotlinParameterTablePanel extends JPanel {
         }
 
         public String getTypeAsString() {
-            return DescriptorRenderer.SHORT_NAMES_IN_TYPES.renderType(getOriginalParameter().getParameterType());
+            return DescriptorRenderer.SHORT_NAMES_IN_TYPES.renderType(getOriginalParameter().getParameterTypeCandidates().get(0));
         }
 
         public Parameter toParameter() {
-            return new Parameter(
-                    originalParameter.getArgumentText(),
-                    name,
-                    originalParameter.getMirrorVarName(),
-                    originalParameter.getParameterType(),
-                    originalParameter.getReceiverCandidate()
-            );
+            return originalParameter.copy(name);
         }
     }
 

idea/testData/refactoring/extractFunction/basic/fragmentWithComment.kt

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     <selection>// test

idea/testData/refactoring/extractFunction/basic/fragmentWithComment.kt.after

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     if (b(a)) return a

idea/testData/refactoring/extractFunction/basic/fragmentWithMultilineComment.kt

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     <selection>/*

idea/testData/refactoring/extractFunction/basic/fragmentWithMultilineComment.kt.after

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     if (b(a)) return a

idea/testData/refactoring/extractFunction/basic/refInReturn.kt

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     <selection>println(a)

idea/testData/refactoring/extractFunction/basic/refInReturn.kt.after

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     if (b(a)) return a

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalBreakWithIf.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalBreakWithIf.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalBreakWithIfElse.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalBreakWithIfElse.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalBreakWithWhen.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalBreakWithWhen.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalReturnWithIf.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalReturnWithIf.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalReturnWithIfElse.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalReturnWithIfElse.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalReturnWithWhen.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/conditionalReturnWithWhen.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/unconditionalBreakWithIf.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/unconditionalBreakWithIf.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/unconditionalBreakWithIfElse.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/unconditionalBreakWithIfElse.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/unconditionalBreakWithWhen.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/conditionalJumps/unconditionalBreakWithWhen.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/default/defaultCF.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int, Comparable<Int>
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 // SIBLING:
 fun foo(a: Int) {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/default/defaultCF.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int, Comparable<Int>
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 // SIBLING:
 fun foo(a: Int) {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/default/defaultCFWithJumps.kt

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int) {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/default/defaultCFWithJumps.kt.after

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int) {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/default/ignoredReturnValueWithIf.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 fun bar(a: Int): Int {
     println(a)
     return a + 10

idea/testData/refactoring/extractFunction/controlFlow/default/ignoredReturnValueWithIf.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 fun bar(a: Int): Int {
     println(a)
     return a + 10

idea/testData/refactoring/extractFunction/controlFlow/default/ignoredReturnValueWithIfNoBlocks.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 fun bar(a: Int): Int {
     println(a)
     return a + 10

idea/testData/refactoring/extractFunction/controlFlow/default/ignoredReturnValueWithIfNoBlocks.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 fun bar(a: Int): Int {
     println(a)
     return a + 10

idea/testData/refactoring/extractFunction/controlFlow/default/ignoredReturnValueWithWhen.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 fun bar(a: Int): Int {
     println(a)
     return a + 10

idea/testData/refactoring/extractFunction/controlFlow/default/ignoredReturnValueWithWhen.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 fun bar(a: Int): Int {
     println(a)
     return a + 10

idea/testData/refactoring/extractFunction/controlFlow/default/ignoredReturnValueWithWhenNoBlocks.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 fun bar(a: Int): Int {
     println(a)
     return a + 10

idea/testData/refactoring/extractFunction/controlFlow/default/ignoredReturnValueWithWhenNoBlocks.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int, Number, Comparable<Int>, Any
 fun bar(a: Int): Int {
     println(a)
     return a + 10

idea/testData/refactoring/extractFunction/controlFlow/definiteReturns/definiteReturnWithIf.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/definiteReturns/definiteReturnWithIf.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/definiteReturns/definiteReturnWithWhen.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/definiteReturns/definiteReturnWithWhen.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/definiteReturns/singleDefiniteReturn.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/definiteReturns/singleDefiniteReturn.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExprInIfCondition.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExprInIfCondition.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExprInIfElse.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExprInIfElse.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExprInWhenBranch.kt

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExprInWhenBranch.kt.after

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExprInWhenSubject.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExprInWhenSubject.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExpressionBodyFunction.kt

@@ -1,2 +1,4 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int, b: Int): Int = <selection>a + b</selection>

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalExpressionBodyFunction.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int, b: Int): Int = i(a, b)
 

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalIfExpr.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalIfExpr.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalWhenExpr.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/evalWhenExpr.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/simpleEvalExpr.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/evaluateExpression/simpleEvalExpr.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValue.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValue.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValueWithIf.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int, Comparable<Int>
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValueWithIf.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int, Comparable<Int>
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValueWithIfElse.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValueWithIfElse.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValueWithWhen.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int, Comparable<Int>
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValueWithWhen.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int, Comparable<Int>
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValueWithWhenElse.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/outputValues/singleOutputValueWithWhenElse.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/breakWithThrow.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/breakWithThrow.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/continueWithThrow.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/continueWithThrow.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/evalExpressionWithThrow.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/evalExpressionWithThrow.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/nonValuedReturnWithThrow.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int) {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/nonValuedReturnWithThrow.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int) {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/outputValueWithThrow.kt

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/outputValueWithThrow.kt.after

@@ -1,3 +1,4 @@
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     var b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/returnWithThrow.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/controlFlow/throws/returnWithThrow.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 // SIBLING:
 fun foo(a: Int): Int {
     val b: Int = 1

idea/testData/refactoring/extractFunction/defaultContainer/classFunction.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 class A {
     class B {
         fun foo(a: Int, b: Int): Int {

idea/testData/refactoring/extractFunction/defaultContainer/classFunction.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 class A {
     class B {
         fun foo(a: Int, b: Int): Int {

idea/testData/refactoring/extractFunction/defaultContainer/topLevelFunction.kt

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 fun foo(a: Int, b: Int): Int {
     return <selection>a + b - 1</selection>
 }

idea/testData/refactoring/extractFunction/defaultContainer/topLevelFunction.kt.after

@@ -1,3 +1,5 @@
+// PARAM_TYPES: kotlin.Int
+// PARAM_TYPES: kotlin.Int
 fun foo(a: Int, b: Int): Int {
     return i(a, b)
 }

idea/testData/refactoring/extractFunction/parameters/candidateTypes/nonNullableTypes.kt

@@ -0,0 +1,14 @@
+// PARAM_TYPES: kotlin.String, Comparable<String>, CharSequence, kotlin.Any
+// PARAM_TYPES: X<kotlin.Any>
+class X<T> {
+    fun add(t: T) {
+
+    }
+}
+
+// SIBLING:
+fun foo(s: String?, x: X<Any>) {
+    when {
+        s != null -> <selection>x.add(s)</selection>
+    }
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/nonNullableTypes.kt.after

@@ -0,0 +1,18 @@
+// PARAM_TYPES: kotlin.String, Comparable<String>, CharSequence, kotlin.Any
+// PARAM_TYPES: X<kotlin.Any>
+class X<T> {
+    fun add(t: T) {
+
+    }
+}
+
+// SIBLING:
+fun foo(s: String?, x: X<Any>) {
+    when {
+        s != null -> unit(s, x)
+    }
+}
+
+fun unit(s: String, x: X<Any>) {
+    x.add(s)
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/nullableTypes.kt

@@ -0,0 +1,14 @@
+// PARAM_TYPES: kotlin.String?, kotlin.Comparable<String>?, kotlin.CharSequence?, kotlin.Any?
+// PARAM_TYPES: X<kotlin.Any?>
+class X<T> {
+    fun add(t: T) {
+
+    }
+}
+
+// SIBLING:
+fun foo(s: String?, x: X<Any?>) {
+    when {
+        s != null -> <selection>x.add(s)</selection>
+    }
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/nullableTypes.kt.after

@@ -0,0 +1,18 @@
+// PARAM_TYPES: kotlin.String?, kotlin.Comparable<String>?, kotlin.CharSequence?, kotlin.Any?
+// PARAM_TYPES: X<kotlin.Any?>
+class X<T> {
+    fun add(t: T) {
+
+    }
+}
+
+// SIBLING:
+fun foo(s: String?, x: X<Any?>) {
+    when {
+        s != null -> unit(s, x)
+    }
+}
+
+fun unit(s: String?, x: X<Any?>) {
+    x.add(s)
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/typeHierarchy1.kt

@@ -0,0 +1,35 @@
+//PARAM_TYPES: C, AImpl, A
+trait A {
+    fun doA()
+}
+
+open class AImpl: A {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+}
+
+trait B {
+    fun doB()
+}
+
+class C: AImpl(), B {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+
+    override fun doB() {
+        throw UnsupportedOperationException()
+    }
+
+    fun doC() {
+        throw UnsupportedOperationException()
+    }
+}
+
+// SIBLING:
+fun foo(c: C) {
+    <selection>c.doA()</selection>
+    c.doB()
+    c.doC()
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/typeHierarchy1.kt.after

@@ -0,0 +1,39 @@
+//PARAM_TYPES: C, AImpl, A
+trait A {
+    fun doA()
+}
+
+open class AImpl: A {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+}
+
+trait B {
+    fun doB()
+}
+
+class C: AImpl(), B {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+
+    override fun doB() {
+        throw UnsupportedOperationException()
+    }
+
+    fun doC() {
+        throw UnsupportedOperationException()
+    }
+}
+
+// SIBLING:
+fun foo(c: C) {
+    unit(c)
+    c.doB()
+    c.doC()
+}
+
+fun unit(c: C) {
+    c.doA()
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/typeHierarchy2.kt

@@ -0,0 +1,35 @@
+//PARAM_TYPES: C, B
+trait A {
+    fun doA()
+}
+
+open class AImpl: A {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+}
+
+trait B {
+    fun doB()
+}
+
+class C: AImpl(), B {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+
+    override fun doB() {
+        throw UnsupportedOperationException()
+    }
+
+    fun doC() {
+        throw UnsupportedOperationException()
+    }
+}
+
+// SIBLING:
+fun foo(c: C) {
+    c.doA()
+    <selection>c.doB()</selection>
+    c.doC()
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/typeHierarchy2.kt.after

@@ -0,0 +1,39 @@
+//PARAM_TYPES: C, B
+trait A {
+    fun doA()
+}
+
+open class AImpl: A {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+}
+
+trait B {
+    fun doB()
+}
+
+class C: AImpl(), B {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+
+    override fun doB() {
+        throw UnsupportedOperationException()
+    }
+
+    fun doC() {
+        throw UnsupportedOperationException()
+    }
+}
+
+// SIBLING:
+fun foo(c: C) {
+    c.doA()
+    unit(c)
+    c.doC()
+}
+
+fun unit(c: C) {
+    c.doB()
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/typeHierarchy3.kt

@@ -0,0 +1,35 @@
+//PARAM_TYPES: C
+trait A {
+    fun doA()
+}
+
+open class AImpl: A {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+}
+
+trait B {
+    fun doB()
+}
+
+class C: AImpl(), B {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+
+    override fun doB() {
+        throw UnsupportedOperationException()
+    }
+
+    fun doC() {
+        throw UnsupportedOperationException()
+    }
+}
+
+// SIBLING:
+fun foo(c: C) {
+    c.doA()
+    c.doB()
+    <selection>c.doC()</selection>
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/typeHierarchy3.kt.after

@@ -0,0 +1,39 @@
+//PARAM_TYPES: C
+trait A {
+    fun doA()
+}
+
+open class AImpl: A {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+}
+
+trait B {
+    fun doB()
+}
+
+class C: AImpl(), B {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+
+    override fun doB() {
+        throw UnsupportedOperationException()
+    }
+
+    fun doC() {
+        throw UnsupportedOperationException()
+    }
+}
+
+// SIBLING:
+fun foo(c: C) {
+    c.doA()
+    c.doB()
+    unit(c)
+}
+
+fun unit(c: C) {
+    c.doC()
+}

idea/testData/refactoring/extractFunction/parameters/candidateTypes/typeHierarchy4.kt

@@ -0,0 +1,35 @@
+//PARAM_TYPES: C
+trait A {
+    fun doA()
+}
+
+open class AImpl: A {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+}
+
+trait B {
+    fun doB()
+}
+
+class C: AImpl(), B {
+    override fun doA() {
+        throw UnsupportedOperationException()
+    }
+
+    override fun doB() {
+        throw UnsupportedOperationException()
+    }
+
+    fun doC() {
+        throw UnsupportedOperationException()
+    }
+}
+
+// SIBLING:
+fun foo(c: C) {
+    <selection>c.doA()
+    c.doB()</selection>
+    c.doC()
+}