[Injimob 1629] refactor OpenId4vp setter methods into non static methods

kotlin/openId4VP/src/main/java/io/mosip/openID4VP/OpenId4VP.kt

@@ -9,33 +9,30 @@ import io.mosip.openID4VP.dto.Verifier
 import io.mosip.openID4VP.networkManager.NetworkManagerClient.Companion.sendHttpPostRequest
 import okhttp3.ResponseBody.Companion.toResponseBody
 
+private val logTag = Logger.getLogTag(AuthorizationResponse::class.simpleName!!)
 class OpenId4VP(private val traceabilityId: String) {
     lateinit var authorizationRequest: AuthorizationRequest
-    private lateinit var logTag: String
+    private lateinit var presentationDefinitionId: String
+    private var responseUri: String? = null
 
-    companion object {
-        private lateinit var presentationDefinitionId: String
-        private var responseUri: String? = null
-
-        fun setResponseUri(responseUri: String) {
-            this.responseUri = responseUri
-        }
+    fun setResponseUri(responseUri: String) {
+        this.responseUri = responseUri
+    }
 
-        fun setPresentationDefinitionId(id: String) {
-            this.presentationDefinitionId = id
-        }
+    fun setPresentationDefinitionId(id: String) {
+        this.presentationDefinitionId = id
     }
 
     fun authenticateVerifier(
         encodedAuthorizationRequest: String, trustedVerifiers: List<Verifier>
     ): Map<String, String> {
         try {
             Logger.setTraceability(traceabilityId)
-            logTag = Logger.getLogTag(AuthorizationRequest::class.simpleName!!)
-            authorizationRequest =
-                AuthorizationRequest.getAuthorizationRequest(encodedAuthorizationRequest)
+            authorizationRequest = AuthorizationRequest.getAuthorizationRequest(
+                encodedAuthorizationRequest, ::setResponseUri
+            )
             return AuthenticationResponse.getAuthenticationResponse(
-                authorizationRequest, trustedVerifiers
+                authorizationRequest, trustedVerifiers, ::setPresentationDefinitionId
             )
         } catch (exception: Exception) {
             sendErrorToVerifier(exception)

kotlin/openId4VP/src/main/java/io/mosip/openID4VP/authenticationResponse/AuthenticationResponse.kt

@@ -1,6 +1,5 @@
 package io.mosip.openID4VP.authenticationResponse
 
-import io.mosip.openID4VP.OpenId4VP
 import io.mosip.openID4VP.authorizationRequest.AuthorizationRequest
 import io.mosip.openID4VP.authorizationRequest.exception.AuthorizationRequestExceptions
 import io.mosip.openID4VP.authorizationRequest.presentationDefinition.PresentationDefinition
@@ -12,6 +11,7 @@ class AuthenticationResponse {
         fun getAuthenticationResponse(
             authorizationRequest: AuthorizationRequest,
             trustedVerifiers: List<Verifier>,
+            setPresentationDefinitionId: (String) -> Unit
         ): Map<String, String> {
             val response = mutableMapOf<String, String>()
             validateVerifierClientID(
@@ -25,7 +25,7 @@ class AuthenticationResponse {
                     presentationDefinitionJson?.let {
                         val presentationDefinition: PresentationDefinition =
                             validatePresentationDefinition(presentationDefinitionJson)
-                        OpenId4VP.setPresentationDefinitionId(presentationDefinition.id)
+                        setPresentationDefinitionId(presentationDefinition.id)
                         response.put("presentation_definition", presentationDefinitionJson)
                     }
                     val scope = authorizationRequest.scope

kotlin/openId4VP/src/main/java/io/mosip/openID4VP/authorizationRequest/AuthorizationRequest.kt

@@ -1,14 +1,12 @@
 package io.mosip.openID4VP.authorizationRequest
 
-import io.mosip.openID4VP.OpenId4VP
 import io.mosip.openID4VP.authorizationRequest.exception.AuthorizationRequestExceptions
 import io.mosip.openID4VP.common.Decoder
 import io.mosip.openID4VP.common.Logger
 import java.net.URI
 import java.net.URLDecoder
 import java.net.URLEncoder
 import java.nio.charset.StandardCharsets
-import io.mosip.openID4VP.OpenId4VP.Companion
 
 private val logTag = Logger.getLogTag(AuthorizationRequest::class.simpleName!!)
 
@@ -23,17 +21,21 @@ class AuthorizationRequest(
     val state: String
 ) {
     companion object {
-        fun getAuthorizationRequest(encodedAuthorizationRequest: String): AuthorizationRequest {
+        fun getAuthorizationRequest(
+            encodedAuthorizationRequest: String, setResponseUri: (String) -> Unit
+        ): AuthorizationRequest {
             try {
                 val decodedAuthorizationRequest =
                     Decoder.decodeBase64ToString(encodedAuthorizationRequest)
-                return parseAuthorizationRequest(decodedAuthorizationRequest)
+                return parseAuthorizationRequest(decodedAuthorizationRequest, setResponseUri)
             } catch (e: Exception) {
                 throw e
             }
         }
 
-        private fun parseAuthorizationRequest(decodedAuthorizationRequest: String): AuthorizationRequest {
+        private fun parseAuthorizationRequest(
+            decodedAuthorizationRequest: String, setResponseUri: (String) -> Unit
+        ): AuthorizationRequest {
             try {
                 val queryStart = decodedAuthorizationRequest.indexOf('?') + 1
                 val queryString = decodedAuthorizationRequest.substring(queryStart)
@@ -44,7 +46,7 @@ class AuthorizationRequest(
                     ?: throw AuthorizationRequestExceptions.InvalidQueryParams("Query parameters are missing in the Authorization request")
 
                 val params = extractQueryParams(query)
-                validateQueryParams(params)
+                validateQueryParams(params, setResponseUri)
                 return createAuthorizationRequest(params)
             } catch (exception: Exception) {
                 Logger.error(logTag, exception)
@@ -64,7 +66,9 @@ class AuthorizationRequest(
             }
         }
 
-        private fun validateQueryParams(params: Map<String, String>) {
+        private fun validateQueryParams(
+            params: Map<String, String>, setResponseUri: (String) -> Unit
+        ) {
             val requiredRequestParams = mutableListOf(
                 "response_uri",
                 "client_id",
@@ -97,7 +101,7 @@ class AuthorizationRequest(
             requiredRequestParams.forEach { param ->
                 val value = params[param] ?: throw AuthorizationRequestExceptions.MissingInput(param)
                 if (param == "response_uri") {
-                    OpenId4VP.setResponseUri(value)
+                    setResponseUri(value)
                 }
                 require(value.isNotEmpty()) {
                     throw AuthorizationRequestExceptions.InvalidInput(param)

kotlin/openId4VP/src/test/java/io/mosip/openID4VP/authorizationResponse/AuthorizationResponseTest.kt

@@ -12,7 +12,6 @@ import io.mosip.openID4VP.common.UUIDGenerator
 import io.mosip.openID4VP.dto.VPResponseMetadata
 import io.mosip.openID4VP.dto.Verifier
 import io.mosip.openID4VP.networkManager.exception.NetworkManagerClientExceptions
-import okhttp3.HttpUrl
 import okhttp3.mockwebserver.MockResponse
 import okhttp3.mockwebserver.MockWebServer
 import org.junit.After
@@ -72,7 +71,7 @@ class AuthorizationResponseTest {
         )
         mockWebServer = MockWebServer()
         mockWebServer.start(8080)
-        OpenId4VP.setPresentationDefinitionId("6498781c-f291-4969-9cd5-2c273858f38f")
+        openId4VP.setPresentationDefinitionId("6498781c-f291-4969-9cd5-2c273858f38f")
         openId4VP.authorizationRequest = AuthorizationRequest(
             clientId = "https://injiverify.dev2.mosip.net",
             responseType = "vp_token",