Create actual/expect quick fix should render super type correctly

idea/src/org/jetbrains/kotlin/idea/quickfix/expectactual/ExpectActualUtils.kt

@@ -11,6 +11,7 @@ import com.intellij.psi.JavaDirectoryService
 import org.jetbrains.kotlin.descriptors.*
 import org.jetbrains.kotlin.idea.caches.project.implementedDescriptors
 import org.jetbrains.kotlin.idea.caches.resolve.analyzeWithContent
+import org.jetbrains.kotlin.idea.codeInsight.shorten.addToBeShortenedDescendantsToWaitingSet
 import org.jetbrains.kotlin.idea.core.KotlinNameSuggester
 import org.jetbrains.kotlin.idea.core.findOrCreateDirectoryForPackage
 import org.jetbrains.kotlin.idea.core.getFqNameWithImplicitPrefix
@@ -23,11 +24,8 @@ import org.jetbrains.kotlin.idea.core.overrideImplement.makeNotActual
 import org.jetbrains.kotlin.idea.core.toDescriptor
 import org.jetbrains.kotlin.idea.project.platform
 import org.jetbrains.kotlin.idea.refactoring.createKotlinFile
+import org.jetbrains.kotlin.idea.util.*
 import org.jetbrains.kotlin.idea.util.application.runWriteAction
-import org.jetbrains.kotlin.idea.util.hasDeclarationOf
-import org.jetbrains.kotlin.idea.util.hasInlineModifier
-import org.jetbrains.kotlin.idea.util.isEffectivelyActual
-import org.jetbrains.kotlin.idea.util.module
 import org.jetbrains.kotlin.js.descriptorUtils.getJetTypeFqName
 import org.jetbrains.kotlin.lexer.KtTokens
 import org.jetbrains.kotlin.name.FqName
@@ -42,6 +40,7 @@ import org.jetbrains.kotlin.resolve.descriptorUtil.fqNameSafe
 import org.jetbrains.kotlin.resolve.descriptorUtil.module
 import org.jetbrains.kotlin.types.AbbreviatedType
 import org.jetbrains.kotlin.types.KotlinType
+import org.jetbrains.kotlin.utils.addToStdlib.safeAs
 
 fun createFileForDeclaration(module: Module, declaration: KtNamedDeclaration): KtFile? {
     val fileName = declaration.name ?: return null
@@ -123,12 +122,13 @@ internal fun KtPsiFactory.generateClassOrObject(
         val superType = context[BindingContext.TYPE, originalEntry.typeReference]
         val superClassDescriptor = superType?.constructor?.declarationDescriptor as? ClassDescriptor ?: return@forEach
         if (superClassDescriptor.kind == ClassKind.CLASS || superClassDescriptor.kind == ClassKind.ENUM_CLASS) {
+            val entryText = IdeDescriptorRenderers.SOURCE_CODE.renderType(superType) ?: return@forEach
             val newGeneratedEntry = if (generateExpectClass) {
-                createSuperTypeEntry(generatedEntry.typeReference!!.text)
+                createSuperTypeEntry(entryText)
             } else {
-                createSuperTypeCallEntry(generatedEntry.typeReference!!.text + "()")
+                createSuperTypeCallEntry("$entryText()")
             }
-            generatedEntry.replace(newGeneratedEntry)
+            generatedEntry.replace(newGeneratedEntry).safeAs<KtElement>()?.addToBeShortenedDescendantsToWaitingSet()
         }
     }
     if (generatedClass.isAnnotation()) {

idea/testData/multiModuleQuickFix/createActualClassWithSuperTypeFromOtherPackage/common/My.kt

@@ -0,0 +1,9 @@
+// "Create actual class for module testModule_JVM (JVM)" "true"
+// DISABLE-ERRORS
+
+package my
+
+import other.Another
+import other.Other
+
+expect class My<caret> : Other<Another>

idea/testData/multiModuleQuickFix/createActualClassWithSuperTypeFromOtherPackage/common/My.kt.after

@@ -0,0 +1,9 @@
+// "Create actual class for module testModule_JVM (JVM)" "true"
+// DISABLE-ERRORS
+
+package my
+
+import other.Another
+import other.Other
+
+expect class My : Other<Another>

idea/testData/multiModuleQuickFix/createActualClassWithSuperTypeFromOtherPackage/common/Other.kt

@@ -0,0 +1,5 @@
+package other
+
+class Other<T>
+
+class Another

idea/testData/multiModuleQuickFix/createActualClassWithSuperTypeFromOtherPackage/common/Other.kt.after

@@ -0,0 +1,5 @@
+package other
+
+class Other<T>
+
+class Another

idea/testData/multiModuleQuickFix/createActualClassWithSuperTypeFromOtherPackage/jvm/My.kt

@@ -0,0 +1,3 @@
+// My: to be implemented
+// DISABLE-ERRORS
+package my

idea/testData/multiModuleQuickFix/createActualClassWithSuperTypeFromOtherPackage/jvm/My.kt.after

@@ -0,0 +1,8 @@
+// My: to be implemented
+// DISABLE-ERRORS
+package my
+
+import other.Another
+import other.Other
+
+actual class My : Other<Another>()

idea/testData/multiModuleQuickFix/createExpectClassWithSuperTypeFromOtherPackage/common/My.kt

@@ -0,0 +1,4 @@
+// My: to be implemented
+// DISABLE-ERRORS
+
+package my

idea/testData/multiModuleQuickFix/createExpectClassWithSuperTypeFromOtherPackage/common/My.kt.after

@@ -0,0 +1,9 @@
+// My: to be implemented
+// DISABLE-ERRORS
+
+package my
+
+import other.Another
+import other.Other
+
+expect class My : Other<Another>

idea/testData/multiModuleQuickFix/createExpectClassWithSuperTypeFromOtherPackage/common/Other.kt

@@ -0,0 +1,5 @@
+package other
+
+class Other<T>
+
+class Another

idea/testData/multiModuleQuickFix/createExpectClassWithSuperTypeFromOtherPackage/common/Other.kt.after

@@ -0,0 +1,5 @@
+package other
+
+class Other<T>
+
+class Another

idea/testData/multiModuleQuickFix/createExpectClassWithSuperTypeFromOtherPackage/jvm/My.kt

@@ -0,0 +1,8 @@
+// "Create expected class in common module testModule_Common" "true"
+// DISABLE-ERRORS
+package my
+
+import other.Another
+import other.Other
+
+actual class <caret>My : Other<Another>()

idea/testData/multiModuleQuickFix/createExpectClassWithSuperTypeFromOtherPackage/jvm/My.kt.after

@@ -0,0 +1,8 @@
+// "Create expected class in common module testModule_Common" "true"
+// DISABLE-ERRORS
+package my
+
+import other.Another
+import other.Other
+
+actual class My : Other<Another>()

idea/tests/org/jetbrains/kotlin/idea/quickfix/QuickFixMultiModuleTestGenerated.java

@@ -204,6 +204,11 @@ public class QuickFixMultiModuleTestGenerated extends AbstractQuickFixMultiModul
         runTest("idea/testData/multiModuleQuickFix/convertPropertyToFunction/");
     }
 
+    @TestMetadata("createActualClassWithSuperTypeFromOtherPackage")
+    public void testCreateActualClassWithSuperTypeFromOtherPackage() throws Exception {
+        runTest("idea/testData/multiModuleQuickFix/createActualClassWithSuperTypeFromOtherPackage/");
+    }
+
     @TestMetadata("createActualWithRootPackage")
     public void testCreateActualWithRootPackage() throws Exception {
         runTest("idea/testData/multiModuleQuickFix/createActualWithRootPackage/");
@@ -219,6 +224,11 @@ public class QuickFixMultiModuleTestGenerated extends AbstractQuickFixMultiModul
         runTest("idea/testData/multiModuleQuickFix/createClassFromUsageRef/");
     }
 
+    @TestMetadata("createExpectClassWithSuperTypeFromOtherPackage")
+    public void testCreateExpectClassWithSuperTypeFromOtherPackage() throws Exception {
+        runTest("idea/testData/multiModuleQuickFix/createExpectClassWithSuperTypeFromOtherPackage/");
+    }
+
     @TestMetadata("createExpectWithRootPackage")
     public void testCreateExpectWithRootPackage() throws Exception {
         runTest("idea/testData/multiModuleQuickFix/createExpectWithRootPackage/");