SONARKT-400 Migrate EncryptionAlgorithmCheck to kotlin-analysis-api

Author: leveretka

sonar-kotlin-api/src/main/java/org/sonarsource/kotlin/api/checks/ApiExtensions.kt

@@ -139,11 +139,12 @@ fun KtExpression.predictRuntimeStringValue(bindingContext: BindingContext) =
 fun KtExpression.predictRuntimeStringValue() =
     predictRuntimeValueExpression().stringValue()
 
-fun KtExpression.predictRuntimeStringValueWithSecondaries(bindingContext: BindingContext) =
+fun KtExpression.predictRuntimeStringValueWithSecondaries() = withKaSession {
     mutableListOf<PsiElement>().let {
-        predictRuntimeValueExpression(bindingContext, it)
-            .stringValue(bindingContext, it) to it
+        predictRuntimeValueExpression(it)
+            .stringValue(it) to it
     }
+}
 
 @Deprecated("use kotlin-analysis-api instead", ReplaceWith("this.predictRuntimeIntValue()"))
 fun KtExpression.predictRuntimeIntValue(bindingContext: BindingContext) =

sonar-kotlin-checks/src/main/java/org/sonarsource/kotlin/checks/EncryptionAlgorithmCheck.kt

@@ -16,13 +16,14 @@
  */
 package org.sonarsource.kotlin.checks
 
+import org.jetbrains.kotlin.analysis.api.resolution.KaFunctionCall
 import org.jetbrains.kotlin.psi.KtCallExpression
-import org.jetbrains.kotlin.resolve.calls.model.ResolvedCall
 import org.sonar.check.Rule
 import org.sonarsource.kotlin.api.checks.CallAbstractCheck
 import org.sonarsource.kotlin.api.checks.FunMatcher
-import org.sonarsource.kotlin.api.reporting.SecondaryLocation
+import org.sonarsource.kotlin.api.checks.FunMatcherImpl
 import org.sonarsource.kotlin.api.checks.predictRuntimeStringValueWithSecondaries
+import org.sonarsource.kotlin.api.reporting.SecondaryLocation
 import org.sonarsource.kotlin.api.reporting.KotlinTextRanges.textRange
 import org.sonarsource.kotlin.api.frontend.KotlinFileContext
 
@@ -33,21 +34,20 @@ val CIPHER_GET_INSTANCE_MATCHER = FunMatcher {
     name = "getInstance"
 }
 
-@org.sonarsource.kotlin.api.frontend.K1only
 @Rule(key = "S5542")
 class EncryptionAlgorithmCheck : CallAbstractCheck() {
 
     override val functionsToVisit = listOf(CIPHER_GET_INSTANCE_MATCHER)
 
     override fun visitFunctionCall(
         callExpression: KtCallExpression,
-        resolvedCall: ResolvedCall<*>,
-        kotlinFileContext: KotlinFileContext,
+        resolvedCall: KaFunctionCall<*>,
+        matchedFun: FunMatcherImpl,
+        kotlinFileContext: KotlinFileContext
     ) {
-        val bindingContext = kotlinFileContext.bindingContext
         callExpression.valueArguments.firstOrNull()?.let { argument ->
             argument.getArgumentExpression()!!
-                .predictRuntimeStringValueWithSecondaries(bindingContext).let { (algorithm, secondaries) ->
+                .predictRuntimeStringValueWithSecondaries().let { (algorithm, secondaries) ->
                     algorithm?.getInsecureAlgorithmMessage()?.let { errorMessage ->
                         val locations = secondaries.map { secondaryLocation ->
                             SecondaryLocation(kotlinFileContext.textRange(secondaryLocation), "Transformation definition")