[FIR] Add MustUse enhancements to known collection/iterable types

For most declarations, enhancements are automatically taken from Kotlin's
supertypes, but functions like Collection.stream() are not declared in Kotlin
sources.

Also add a bunch of tests for Java types enhancements.

#KT-74810
#KT-71195

Author: sandwwraith

analysis/low-level-api-fir/tests-gen/org/jetbrains/kotlin/analysis/low/level/api/fir/diagnostic/compiler/based/LLDiagnosticsFe10TestGenerated.java

@@ -512,11 +512,15 @@ class FirSignatureEnhancement(
     ): FirDeclarationStatus {
         val original = method.status
         if (original.returnValueStatus != ReturnValueStatus.Unspecified) return original
-        val newRvStatus = overriddenMembers.firstNotNullOfOrNull { declaration ->
+        predefinedEnhancementInfo?.returnValueStatus?.takeIf { it != ReturnValueStatus.Unspecified }?.let { newRvStatus ->
+            return original.copy(returnValueStatus = newRvStatus)
+        }
+        overriddenMembers.firstNotNullOfOrNull { declaration ->
             declaration.status.returnValueStatus.takeIf { it != ReturnValueStatus.Unspecified }
+        }?.let { newRvStatus ->
+            return original.copy(returnValueStatus = newRvStatus)
         }
-        if (newRvStatus == null) return original
-        return original.copy(returnValueStatus = newRvStatus)
+        return original
     }
 
     private fun buildEnhancedValueParameter(

analysis/low-level-api-fir/tests-gen/org/jetbrains/kotlin/analysis/low/level/api/fir/diagnostic/compiler/based/LLPartialDiagnosticsFe10TestGenerated.java

@@ -0,0 +1,69 @@
+// RUN_PIPELINE_TILL: BACKEND
+// WITH_STDLIB
+// FULL_JDK
+
+import java.util.Optional
+
+fun list(m: MutableList<String>) {
+    m.add("")
+    <!RETURN_VALUE_NOT_USED!>m.isEmpty()<!>
+    <!RETURN_VALUE_NOT_USED!>m.toString()<!>
+}
+
+fun javaList(m: ArrayList<String>) {
+    m.add("x")
+    <!RETURN_VALUE_NOT_USED!>m.isEmpty()<!>
+}
+
+fun javaSet(s: LinkedHashSet<String>) {
+    s.add("x")
+    <!RETURN_VALUE_NOT_USED!>s.isEmpty()<!>
+    <!RETURN_VALUE_NOT_USED!>s.toString()<!>
+    <!RETURN_VALUE_NOT_USED!>s.first()<!>
+}
+
+fun map(s: MutableMap<String, String>) {
+    <!RETURN_VALUE_NOT_USED!>s.get("")<!>
+    <!RETURN_VALUE_NOT_USED!>s.getOrDefault("", "")<!>
+    s.put("", "")
+    s.putIfAbsent("", "")
+    s.merge("", "") { a, b -> a + b }
+}
+
+fun javaMap(s: LinkedHashMap<String, String>) {
+    <!RETURN_VALUE_NOT_USED!>s.get("")<!>
+    <!RETURN_VALUE_NOT_USED!>s.getOrDefault("", "")<!>
+    s.put("", "")
+    s.putIfAbsent("", "")
+    s.merge("", "") { a, b -> a + b }
+}
+
+fun iterable(i: Iterable<String>) {
+    <!RETURN_VALUE_NOT_USED!>i.iterator()<!>
+    <!RETURN_VALUE_NOT_USED!>i.spliterator()<!>
+}
+
+fun collection(i: Collection<String>) {
+    <!RETURN_VALUE_NOT_USED!>i.iterator()<!>
+    <!RETURN_VALUE_NOT_USED!>i.stream()<!>
+    <!RETURN_VALUE_NOT_USED!>i.parallelStream()<!>
+}
+
+fun optional(s: String?) {
+    <!RETURN_VALUE_NOT_USED!>Optional.of(s!!)<!>
+    <!RETURN_VALUE_NOT_USED!>Optional.empty<String>()<!>
+    <!RETURN_VALUE_NOT_USED!>Optional.ofNullable(s)<!>
+    val o = Optional.ofNullable(s)
+    <!RETURN_VALUE_NOT_USED!>o.get()<!>
+}
+
+fun stringBuilder(sb: StringBuilder, ss: CharSequence) {
+    sb.append("")
+    sb.length // .length is not reported because of KT-80179
+    <!RETURN_VALUE_NOT_USED!>sb.get(0)<!>
+    <!RETURN_VALUE_NOT_USED!>ss.length<!>
+    <!RETURN_VALUE_NOT_USED!>ss.get(0)<!>
+}
+
+/* GENERATED_FIR_TAGS: functionDeclaration, inProjection, lambdaLiteral, nullableType, outProjection, samConversion,
+stringLiteral */

analysis/low-level-api-fir/tests-gen/org/jetbrains/kotlin/analysis/low/level/api/fir/diagnostic/compiler/based/LLReversedDiagnosticsFe10TestGenerated.java

@@ -0,0 +1,69 @@
+// RUN_PIPELINE_TILL: BACKEND
+// WITH_STDLIB
+// FULL_JDK
+
+import java.util.Optional
+
+fun list(m: MutableList<String>) {
+    m.add("")
+    m.isEmpty()
+    m.toString()
+}
+
+fun javaList(m: ArrayList<String>) {
+    m.add("x")
+    m.isEmpty()
+}
+
+fun javaSet(s: LinkedHashSet<String>) {
+    s.add("x")
+    s.isEmpty()
+    s.toString()
+    s.first()
+}
+
+fun map(s: MutableMap<String, String>) {
+    s.get("")
+    s.getOrDefault("", "")
+    s.put("", "")
+    s.putIfAbsent("", "")
+    s.merge("", "") { a, b -> a + b }
+}
+
+fun javaMap(s: LinkedHashMap<String, String>) {
+    s.get("")
+    s.getOrDefault("", "")
+    s.put("", "")
+    s.putIfAbsent("", "")
+    s.merge("", "") { a, b -> a + b }
+}
+
+fun iterable(i: Iterable<String>) {
+    i.iterator()
+    i.spliterator()
+}
+
+fun collection(i: Collection<String>) {
+    i.iterator()
+    i.stream()
+    i.parallelStream()
+}
+
+fun optional(s: String?) {
+    Optional.of(s!!)
+    Optional.empty<String>()
+    Optional.ofNullable(s)
+    val o = Optional.ofNullable(s)
+    o.get()
+}
+
+fun stringBuilder(sb: StringBuilder, ss: CharSequence) {
+    sb.append("")
+    sb.length // .length is not reported because of KT-80179
+    sb.get(0)
+    ss.length
+    ss.get(0)
+}
+
+/* GENERATED_FIR_TAGS: functionDeclaration, inProjection, lambdaLiteral, nullableType, outProjection, samConversion,
+stringLiteral */

compiler/fir/analysis-tests/tests-gen/org/jetbrains/kotlin/test/runners/FirLightTreeOldFrontendDiagnosticsWithLatestLanguageVersionTestGenerated.java

@@ -4,31 +4,8 @@
 @file:MustUseReturnValue
 
 fun stringF(): String = ""
-fun nsf(): String? = "null"
 fun unitF(): Unit = Unit
 
-fun coll(m: MutableList<String>) {
-    m.add("")
-    <!RETURN_VALUE_NOT_USED!>m.isEmpty()<!>
-    <!RETURN_VALUE_NOT_USED!>m.toString()<!>
-}
-
-fun nullable(m: MutableList<String>?) {
-    m?.add("x")
-    <!RETURN_VALUE_NOT_USED!>m?.isEmpty()<!>
-}
-
-fun lhs(s: LinkedHashSet<String>) {
-    s.add("x")
-    <!RETURN_VALUE_NOT_USED!>s.isEmpty()<!>
-    <!RETURN_VALUE_NOT_USED!>s.toString()<!>
-}
-
-fun javaType(m: java.util.ArrayList<String>) {
-    m.add("x")
-    <!RETURN_VALUE_NOT_USED!>m.isEmpty()<!>
-}
-
 fun exlusionPropagation(cond: Boolean, m: MutableList<String>) {
     if (cond) m.add("x") else throw IllegalStateException()
     if (cond) <!RETURN_VALUE_NOT_USED!>stringF()<!> else throw IllegalStateException()