Create TrailingCommaIntention

#KT-34744

idea/testData/quickfix/addDataModifier/abstract.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Foo.component2'
 // ACTION: Create member function 'Foo.component1'
 // ACTION: Create member function 'Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put arguments on separate lines
 // ERROR: Cannot create an instance of an abstract class
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component1()' function

idea/testData/quickfix/addDataModifier/inner.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Test.Foo.component2'
 // ACTION: Create member function 'Test.Foo.component1'
 // ACTION: Create member function 'Test.Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put arguments on separate lines
 // ERROR: Destructuring declaration initializer of type Test.Foo must have a 'component1()' function
 // ERROR: Destructuring declaration initializer of type Test.Foo must have a 'component2()' function

idea/testData/quickfix/addDataModifier/invisibleParameter.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Foo.component2'
 // ACTION: Create member function 'Foo.component1'
 // ACTION: Create member function 'Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put arguments on separate lines
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component1()' function
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component2()' function

idea/testData/quickfix/addDataModifier/invisibleParameter2.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Foo.component2'
 // ACTION: Create member function 'Foo.component1'
 // ACTION: Create member function 'Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put arguments on separate lines
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component1()' function
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component2()' function

idea/testData/quickfix/addDataModifier/noParameter.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Foo.component2'
 // ACTION: Create member function 'Foo.component1'
 // ACTION: Create member function 'Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component1()' function
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component2()' function
 class Foo()

idea/testData/quickfix/addDataModifier/notVarVal.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Foo.component2'
 // ACTION: Create member function 'Foo.component1'
 // ACTION: Create member function 'Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put arguments on separate lines
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component1()' function
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component2()' function

idea/testData/quickfix/addDataModifier/open.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Foo.component2'
 // ACTION: Create member function 'Foo.component1'
 // ACTION: Create member function 'Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put arguments on separate lines
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component1()' function
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component2()' function

idea/testData/quickfix/addDataModifier/sealed.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Foo.component2'
 // ACTION: Create member function 'Foo.component1'
 // ACTION: Create member function 'Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put arguments on separate lines
 // ERROR: Cannot access '<init>': it is private in 'Foo'
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component1()' function

idea/testData/quickfix/addDataModifier/vararg.kt

@@ -3,6 +3,7 @@
 // ACTION: Create extension function 'Foo.component2'
 // ACTION: Create member function 'Foo.component1'
 // ACTION: Create member function 'Foo.component2'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put arguments on separate lines
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component1()' function
 // ERROR: Destructuring declaration initializer of type Foo must have a 'component2()' function

idea/testData/quickfix/addInline/local.kt

@@ -1,4 +1,5 @@
 // "Add 'inline' to function 'foo'" "false"
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Modifier 'crossinline' is allowed only for function parameters of an inline function
 
 fun bar() {

idea/testData/quickfix/addVarianceModifier/emptyTest.kt

@@ -1,3 +1,4 @@
 // "Add 'out' variance" "false"
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Implement interface
 interface Empty<<caret>T>

idea/testData/quickfix/autoImports/multiDeclarationExtensionComponentNoOperator.test

@@ -3,6 +3,7 @@
 // ERROR: Destructuring declaration initializer of type Some must have a 'component1()' function
 // ACTION: Create extension function 'Some.component1'
 // ACTION: Create member function 'Some.component1'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Introduce import alias
 
 

idea/testData/quickfix/changeSignature/removeUnusedParameterOfPropertySetter.kt

@@ -1,4 +1,5 @@
 // "Remove parameter 'value'" "false"
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify type explicitly
 class Abacaba {
     var foo: String

idea/testData/quickfix/createFromUsage/createClass/typeReference/noAnnotationForTypeBound.kt

@@ -3,5 +3,6 @@
 // ACTION: Create interface 'NotExistent'
 // ACTION: Create type parameter 'NotExistent' in class 'TPB'
 // ACTION: Create test
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Unresolved reference: NotExistent
 class TPB<X : <caret>NotExistent>

idea/testData/quickfix/createFromUsage/createClass/typeReference/noEnumForTypeBound.kt

@@ -3,5 +3,6 @@
 // ACTION: Create interface 'NotExistent'
 // ACTION: Create type parameter 'NotExistent' in class 'TPB'
 // ACTION: Create test
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Unresolved reference: NotExistent
 class TPB<X : <caret>NotExistent>

idea/testData/quickfix/createFromUsage/createClass/typeReference/noObjectForTypeBound.kt

@@ -3,5 +3,6 @@
 // ACTION: Create interface 'NotExistent'
 // ACTION: Create type parameter 'NotExistent' in class 'TPB'
 // ACTION: Create test
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Unresolved reference: NotExistent
 class TPB<X : <caret>NotExistent>

idea/testData/quickfix/createFromUsage/createTypeParameter/inContainingDeclaration/qualifiedType.kt

@@ -3,6 +3,7 @@
 // ACTION: Create class 'X'
 // ACTION: Create enum 'X'
 // ACTION: Create interface 'X'
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Unresolved reference: X
 
 class A

idea/testData/quickfix/createFromUsage/createTypeParameter/inContainingDeclaration/typeQualifier.kt

@@ -3,6 +3,7 @@
 // ACTION: Create enum 'X'
 // ACTION: Create interface 'X'
 // ACTION: Create object 'X'
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Unresolved reference: X
 
 class A

idea/testData/quickfix/createFromUsage/createTypeParameter/inContainingDeclaration/withTypeArguments.kt

@@ -1,6 +1,7 @@
 // "Create type parameter 'X'" "false"
 // ACTION: Create class 'X'
 // ACTION: Create interface 'X'
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Unresolved reference: X
 
 class A

idea/testData/quickfix/createFromUsage/createTypeParameter/inReferencedDeclaration/missingArguments.kt

@@ -1,4 +1,5 @@
 // "Create type parameter in class 'X'" "false"
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Introduce import alias
 // ERROR: 2 type arguments expected for class X<T, U>
 class X<T, U>

idea/testData/quickfix/createFromUsage/createTypeParameter/inReferencedDeclaration/notOnTypeArgumentList.kt

@@ -1,4 +1,5 @@
 // "Create type parameter in class 'X'" "false"
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Introduce import alias
 // ERROR: No type arguments expected for class X
 

idea/testData/quickfix/createFromUsage/createVariable/localVariable/inAnnotation.kt

@@ -1,6 +1,7 @@
 // "Create local variable 'defaultPreferencesName'" "false"
 // ACTION: Create parameter 'defaultPreferencesName'
 // ACTION: Create property 'defaultPreferencesName'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Rename reference
 // ERROR: Unresolved reference: defaultPreferencesName
 // ERROR: Default value of annotation parameter must be a compile-time constant

idea/testData/quickfix/createFromUsage/createVariable/parameter/dataClassPropertyByDestructuringEntryWithSkippedIndex.kt

@@ -1,5 +1,6 @@
 // "Create property 'address2' as constructor parameter" "false"
 // ACTION: Create property 'address' as constructor parameter
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Introduce import alias
 // ACTION: Make 'Person' data class
 // ERROR: Destructuring declaration initializer of type Person must have a 'component3()' function

idea/testData/quickfix/decreaseVisibility/exposedParameterType.kt

@@ -1,5 +1,6 @@
 // "Make 'foo' private" "false"
 // ACTION: Convert parameter to receiver
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Make 'Nested' internal
 // ACTION: Make 'Nested' public
 // ACTION: Remove parameter 'arg'

idea/testData/quickfix/decreaseVisibility/exposedTypeInAnnotation.kt

@@ -1,5 +1,6 @@
 // "Make '<init>' internal" "false"
 // DISABLE-ERRORS
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Introduce import alias
 // ACTION: Make 'My' public
 

idea/testData/quickfix/experimental/override.kt

@@ -5,6 +5,7 @@
 // ACTION: Add '@OptIn(MyExperimentalAPI::class)' annotation to 'foo'
 // ACTION: Add '@OptIn(MyExperimentalAPI::class)' annotation to containing class 'Derived'
 // ACTION: Add '-Xopt-in=MyExperimentalAPI' to module light_idea_test_case compiler arguments
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: This declaration overrides experimental member of supertype 'Base' and must be annotated with '@MyExperimentalAPI'
 
 @RequiresOptIn

idea/testData/quickfix/functionWithLambdaExpressionBody/removeBraces/functionHasComment.kt

@@ -2,6 +2,7 @@
 // TOOL: org.jetbrains.kotlin.idea.inspections.FunctionWithLambdaExpressionBodyInspection
 // ACTION: Convert to block body
 // ACTION: Convert to run { ... }
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify return type explicitly

idea/testData/quickfix/functionWithLambdaExpressionBody/removeBraces/functionHasMultiStatements.kt

@@ -3,6 +3,7 @@
 // ACTION: Convert to block body
 // ACTION: Convert to run { ... }
 // ACTION: Specify explicit lambda signature
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify return type explicitly
 fun test(a: Int, b: Int) = <caret>{

idea/testData/quickfix/functionWithLambdaExpressionBody/removeBraces/functionHasNoStatement.kt

@@ -3,6 +3,7 @@
 // ACTION: Convert to block body
 // ACTION: Convert to multi-line lambda
 // ACTION: Convert to run { ... }
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify return type explicitly

idea/testData/quickfix/functionWithLambdaExpressionBody/removeBraces/funtionIsUsed.kt

@@ -2,6 +2,7 @@
 // TOOL: org.jetbrains.kotlin.idea.inspections.FunctionWithLambdaExpressionBodyInspection
 // ACTION: Convert to block body
 // ACTION: Convert to multi-line lambda
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify return type explicitly

idea/testData/quickfix/functionWithLambdaExpressionBody/removeBraces/getterHasComment.kt

@@ -3,6 +3,7 @@
 // ACTION: Convert property getter to initializer
 // ACTION: Convert to block body
 // ACTION: Convert to run { ... }
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify type explicitly

idea/testData/quickfix/functionWithLambdaExpressionBody/removeBraces/getterHasMultiStatements.kt

@@ -3,6 +3,7 @@
 // ACTION: Convert property getter to initializer
 // ACTION: Convert to block body
 // ACTION: Convert to run { ... }
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify type explicitly

idea/testData/quickfix/functionWithLambdaExpressionBody/removeBraces/getterHasNoStatement.kt

@@ -4,6 +4,7 @@
 // ACTION: Convert to block body
 // ACTION: Convert to multi-line lambda
 // ACTION: Convert to run { ... }
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify type explicitly

idea/testData/quickfix/functionWithLambdaExpressionBody/removeBraces/getterIsUsed.kt

@@ -3,6 +3,7 @@
 // ACTION: Convert property getter to initializer
 // ACTION: Convert to block body
 // ACTION: Convert to multi-line lambda
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify type explicitly

idea/testData/quickfix/functionWithLambdaExpressionBody/wrapRun/funtionIsUsed.kt

@@ -2,6 +2,7 @@
 // TOOL: org.jetbrains.kotlin.idea.inspections.FunctionWithLambdaExpressionBodyInspection
 // ACTION: Convert to block body
 // ACTION: Convert to multi-line lambda
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify return type explicitly

idea/testData/quickfix/functionWithLambdaExpressionBody/wrapRun/getterIsUsed.kt

@@ -3,6 +3,7 @@
 // ACTION: Convert property getter to initializer
 // ACTION: Convert to block body
 // ACTION: Convert to multi-line lambda
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify explicit lambda signature
 // ACTION: Specify type explicitly

idea/testData/quickfix/makePrivateAndOverrideMember/parameterHasInternal.before.Main.kt

@@ -2,6 +2,7 @@
 // DISABLE-ERRORS
 // ACTION: Convert to secondary constructor
 // ACTION: Create test
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Make private
 // ACTION: Make protected
 // ACTION: Make public

idea/testData/quickfix/memberVisibilityCanBePrivate/getter.before.Main.kt

@@ -1,6 +1,7 @@
 // "Add 'private' modifier" "false"
 // ACTION: Convert to secondary constructor
 // ACTION: Create test
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Move to class body
 
 class My(val <caret>parameter: Int) {

idea/testData/quickfix/modifiers/addOpenToClassDeclaration/finalJavaUpperBound.before.Main.kt

@@ -1,5 +1,6 @@
 // "class org.jetbrains.kotlin.idea.quickfix.AddModifierFix" "false"
 // ACTION: Create test
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Inline type parameter
 // ACTION: Introduce import alias
 // ACTION: Remove final upper bound

idea/testData/quickfix/nullables/unsafeInfixCall/unsafeComparisonInWhen.kt

@@ -1,6 +1,7 @@
 // "Replace with safe (?.) call" "false"
 // ACTION: Add non-null asserted (!!) call
 // ACTION: Flip '<='
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Replace overloaded operator with function call
 // ERROR: Operator call corresponds to a dot-qualified call 'w?.x.compareTo(42)' which is not allowed on a nullable receiver 'w?.x'.
 

idea/testData/quickfix/platformTypesInspection/nestedNoAssertRuntime.kt

@@ -2,6 +2,7 @@
 // ACTION: Convert function to property
 // ACTION: Convert to block body
 // ACTION: Create test
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Specify return type explicitly
 
 fun foo<caret>() = arrayOf(java.lang.String.valueOf(1))

idea/testData/quickfix/removeSingleLambdaParameter/multiple.kt

@@ -5,6 +5,7 @@
 // ACTION: Specify type explicitly
 // ACTION: Convert to anonymous function
 // ACTION: Convert to multi-line lambda
+// ACTION: Enable a trailing comma by default in the formatter
 // RUNTIME_WITH_FULL_JDK
 
 fun main() {

idea/testData/quickfix/removeSingleLambdaParameter/propertyWithImplicitType.kt

@@ -4,6 +4,7 @@
 // ACTION: Convert to also
 // ACTION: Convert to apply
 // ACTION: Convert to multi-line lambda
+// ACTION: Enable a trailing comma by default in the formatter
 fun test() {
     val f = { <caret>i: Int -> foo() }
     bar(f)

idea/testData/quickfix/renameToUnderscore/functionExpressionParameterNoRemoveParameter.kt

@@ -3,6 +3,7 @@
 // ACTION: Convert parameter to receiver
 // ACTION: Rename to _
 // ACTION: Specify return type explicitly
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put parameters on separate lines
 
 fun foo(block: (String, Int) -> Unit) {

idea/testData/quickfix/renameToUnderscore/lambdaParameterNoRemoveParameter.kt

@@ -1,4 +1,5 @@
 // "Remove parameter 'x'" "false"
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Move lambda argument into parentheses
 // ACTION: Remove explicit lambda parameter types (may break code)
 // ACTION: Rename to _

idea/testData/quickfix/renameToUnderscore/noActionForCommonFunction.kt

@@ -1,5 +1,6 @@
 // "Rename to _" "false"
 // ACTION: Convert parameter to receiver
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Remove parameter 'x'
 
 fun foo(x<caret>: Int) {

idea/testData/quickfix/renameUnresolvedReference/typeRef.kt

@@ -3,6 +3,7 @@
 // ACTION: Create class 'X'
 // ACTION: Create enum 'X'
 // ACTION: Create interface 'X'
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Unresolved reference: X
 // ERROR: Unresolved reference: X
 class A {

idea/testData/quickfix/replaceWithSafeCallForScopeFunction/letWithWrongImplicitThis.kt

@@ -7,6 +7,7 @@
 // ACTION: Specify explicit lambda signature
 // ACTION: Add return@let
 // ACTION: Convert to single-line lambda
+// ACTION: Enable a trailing comma by default in the formatter
 // ERROR: Only safe (?.) or non-null asserted (!!.) calls are allowed on a nullable receiver of type String?
 
 fun String?.foo(a: String?) {

idea/testData/quickfix/specifyOverrideExplicitly/notPossible.kt

@@ -2,6 +2,7 @@
 // ACTION: Create test
 // ACTION: Make primary constructor internal
 // ACTION: Make primary constructor private
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Extract 'C' from current file
 // ACTION: Rename file to C.kt
 

idea/testData/quickfix/typeMismatch/typeMismatchOnReturnedExpression/dontChangeFunctionReturnTypeToErrorType.kt

@@ -4,6 +4,7 @@
 // ACTION: Create class 'NoSuchType'
 // ACTION: Create enum 'NoSuchType'
 // ACTION: Create interface 'NoSuchType'
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Remove explicit lambda parameter types (may break code)
 // ACTION: Create type parameter 'NoSuchType' in function 'foo'
 // ERROR: Type mismatch: inferred type is ([ERROR : NoSuchType]) -> Int but Int was expected

idea/testData/quickfix/typeOfAnnotationMember/star.kt

@@ -1,6 +1,7 @@
 // "Replace array of boxed with array of primitive" "false"
 // ERROR: Invalid type of annotation member
 // ACTION: Introduce import alias
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Put parameters on one line
 annotation class SuperAnnotation(
         val foo: <caret>Array<*>,

idea/testData/quickfix/typeOfAnnotationMember/string.kt

@@ -1,6 +1,7 @@
 // "Replace array of boxed with array of primitive" "false"
 // ACTION: Put parameters on one line
 // ACTION: Introduce import alias
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Convert to vararg parameter (may break code)
 annotation class SuperAnnotation(
         val str: <caret>Array<String>

idea/testData/quickfix/unusedSuppressAnnotation/notForDeprecated.kt

@@ -1,4 +1,5 @@
 // "Suppress unused warning if annotated by 'kotlin.Deprecated'" "false"
 // ACTION: Create test
+// ACTION: Enable a trailing comma by default in the formatter
 @Deprecated("")
 fun foo<caret>(){}

idea/testData/quickfix/variables/removeValVarFromParameter/constructorParameter.kt

@@ -4,6 +4,7 @@
 // ACTION: Make protected
 // ACTION: Create test
 // ACTION: Convert to secondary constructor
+// ACTION: Enable a trailing comma by default in the formatter
 // ACTION: Move to class body
 class C(<caret>val x: String) {
 }