Skip to content

Commit 0c7634a

Browse files
committed
lint
1 parent c85f632 commit 0c7634a

File tree

2 files changed

+16
-18
lines changed
  • aws-auth-cognito/src
    • main/java/com/amplifyframework/statemachine/codegen/data
    • test/java/com/amplifyframework/auth/cognito/data

2 files changed

+16
-18
lines changed

aws-auth-cognito/src/main/java/com/amplifyframework/statemachine/codegen/data/Tokens.kt

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,8 @@ import com.amplifyframework.auth.exceptions.UnknownException
2020
import com.amplifyframework.statemachine.util.mask
2121
import java.time.Instant
2222
import kotlin.text.Charsets.UTF_8
23-
import kotlinx.serialization.Serializable
24-
import org.json.JSONObject
2523
import kotlinx.serialization.KSerializer
24+
import kotlinx.serialization.Serializable
2625
import kotlinx.serialization.SerializationException
2726
import kotlinx.serialization.builtins.serializer
2827
import kotlinx.serialization.encoding.Decoder
@@ -31,6 +30,7 @@ import kotlinx.serialization.json.JsonDecoder
3130
import kotlinx.serialization.json.JsonObject
3231
import kotlinx.serialization.json.JsonPrimitive
3332
import kotlinx.serialization.json.jsonPrimitive
33+
import org.json.JSONObject
3434

3535
internal abstract class Jwt {
3636
abstract val tokenValue: String
@@ -155,17 +155,15 @@ internal data class CognitoUserPoolTokens(
155155
/**
156156
* Helper function to extract token value from either flat or nested format
157157
*/
158-
private fun extractTokenValue(decoder: Decoder, tokenType: String): String {
159-
return if (decoder is JsonDecoder) {
160-
when (val element = decoder.decodeJsonElement()) {
161-
is JsonPrimitive -> element.content // Flat format: "token": "value"
162-
is JsonObject -> element["tokenValue"]?.jsonPrimitive?.content
163-
?: throw SerializationException("Missing tokenValue in nested $tokenType")
164-
else -> throw SerializationException("Expected string or object for $tokenType")
165-
}
166-
} else {
167-
decoder.decodeString() // Fallback for non-JSON decoders
158+
private fun extractTokenValue(decoder: Decoder, tokenType: String): String = if (decoder is JsonDecoder) {
159+
when (val element = decoder.decodeJsonElement()) {
160+
is JsonPrimitive -> element.content // Flat format: "token": "value"
161+
is JsonObject -> element["tokenValue"]?.jsonPrimitive?.content
162+
?: throw SerializationException("Missing tokenValue in nested $tokenType")
163+
else -> throw SerializationException("Expected string or object for $tokenType")
168164
}
165+
} else {
166+
decoder.decodeString() // Fallback for non-JSON decoders
169167
}
170168

171169
/**
@@ -193,4 +191,4 @@ internal object RefreshTokenAsStringSerializer : KSerializer<RefreshToken> {
193191
override val descriptor = String.serializer().descriptor
194192
override fun serialize(encoder: Encoder, value: RefreshToken) = encoder.encodeString(value.tokenValue)
195193
override fun deserialize(decoder: Decoder) = RefreshToken(extractTokenValue(decoder, "RefreshToken"))
196-
}
194+
}

aws-auth-cognito/src/test/java/com/amplifyframework/auth/cognito/data/TokensTest.kt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ import org.junit.Test
2828
class TokensTest {
2929
private val tokenString =
3030
"eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwidXNlcm5hbWUiOiJqZG" +
31-
"9lIiwiaWF0IjoxNzU2OTk4Mjc4LCJleHAiOjE3NTY5OTg1NzgsIm9yaWdpbl9qdGkiOiJhYWFhYWFhYS1iYmJiLWNjY2MtZGRkZC1lZ" +
32-
"WVlZWVlZWVlZWUifQ.3Mvd5WVi1z1GpQ37hEoev6DzYNv9lWNL-fGfQTxUYx4"
31+
"9lIiwiaWF0IjoxNzU2OTk4Mjc4LCJleHAiOjE3NTY5OTg1NzgsIm9yaWdpbl9qdGkiOiJhYWFhYWFhYS1iYmJiLWNjY2MtZGRkZC1lZ" +
32+
"WVlZWVlZWVlZWUifQ.3Mvd5WVi1z1GpQ37hEoev6DzYNv9lWNL-fGfQTxUYx4"
3333

3434
@Test
3535
fun `identity token returns expiry`() {
@@ -88,8 +88,8 @@ class TokensTest {
8888
expiration = null
8989
)
9090
cognitoTokens.toString() shouldBe
91-
"CognitoUserPoolTokens(idToken=eyJh***, accessToken=eyJh***, " +
92-
"refreshToken=eyJh***, expiration=null)"
91+
"CognitoUserPoolTokens(idToken=eyJh***, accessToken=eyJh***, " +
92+
"refreshToken=eyJh***, expiration=null)"
9393
}
9494

9595
@Test
@@ -202,4 +202,4 @@ class TokensTest {
202202
val serializedNested = Json.encodeToString(tokensFromNested)
203203
serializedFlat shouldBe serializedNested
204204
}
205-
}
205+
}

0 commit comments

Comments
 (0)