From 4d9bd99f7d3f16375e5909ea26edf880137f18fe Mon Sep 17 00:00:00 2001
From: Diego Ocampo <docampoherrera@atlassian.com>
Date: Wed, 24 May 2023 14:10:57 +1000
Subject: [PATCH] Removing coroutines

---
 build.gradle.kts                                     |  1 -
 .../com/atlassian/onetime/service/SecretProvider.kt  |  6 +++---
 .../com/atlassian/onetime/service/TOTPService.kt     | 12 ++++++------
 .../com/atlassian/onetime/core/HOTPGeneratorTest.kt  |  2 +-
 .../com/atlassian/onetime/core/TOTPGeneratorTest.kt  |  2 +-
 .../onetime/service/DefaultTOTPServiceTest.kt        |  2 +-
 6 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/build.gradle.kts b/build.gradle.kts
index 24f5ab7..aae9e81 100644
--- a/build.gradle.kts
+++ b/build.gradle.kts
@@ -50,7 +50,6 @@ tasks.withType<KotlinCompile> {
       "-progressive",
       "-java-parameters",
       "-opt-in=kotlin.time.ExperimentalTime",
-      "-opt-in=kotlinx.coroutines.ExperimentalCoroutinesApi",
       "-opt-in=kotlin.RequiresOptIn"
     )
 
diff --git a/src/main/kotlin/com/atlassian/onetime/service/SecretProvider.kt b/src/main/kotlin/com/atlassian/onetime/service/SecretProvider.kt
index 1d88953..bf6a7c3 100644
--- a/src/main/kotlin/com/atlassian/onetime/service/SecretProvider.kt
+++ b/src/main/kotlin/com/atlassian/onetime/service/SecretProvider.kt
@@ -4,7 +4,7 @@ import com.atlassian.onetime.model.TOTPSecret
 import java.security.SecureRandom
 
 fun interface SecretProvider {
-  suspend fun generateSecret(): TOTPSecret
+  fun generateSecret(): TOTPSecret
 }
 
 class AsciiRangeSecretProvider : SecretProvider {
@@ -13,14 +13,14 @@ class AsciiRangeSecretProvider : SecretProvider {
     val ASCII_RANGE: CharRange = (' '..'z')
   }
 
-  override suspend fun generateSecret() = TOTPSecret(
+  override fun generateSecret() = TOTPSecret(
     (1..20).map { ASCII_RANGE.random() }.joinToString("").toByteArray()
   )
 }
 
 class RandomSecretProvider : SecretProvider {
 
-  override suspend fun generateSecret() =
+  override fun generateSecret() =
     SecureRandom().let {
       val byteArray = ByteArray(20)
       it.nextBytes(byteArray)
diff --git a/src/main/kotlin/com/atlassian/onetime/service/TOTPService.kt b/src/main/kotlin/com/atlassian/onetime/service/TOTPService.kt
index 123e6b4..32b2aa7 100644
--- a/src/main/kotlin/com/atlassian/onetime/service/TOTPService.kt
+++ b/src/main/kotlin/com/atlassian/onetime/service/TOTPService.kt
@@ -10,15 +10,15 @@ import java.net.URLEncoder
 
 interface TOTPService {
 
-  suspend fun generateTotpSecret(): TOTPSecret
+  fun generateTotpSecret(): TOTPSecret
 
-  suspend fun generateTOTPUrl(
+  fun generateTOTPUrl(
     totpSecret: TOTPSecret,
     emailAddress: EmailAddress,
     issuer: Issuer
   ): URI
 
-  suspend fun verify(
+  fun verify(
     code: TOTP,
     totpSecret: TOTPSecret
   ): TOTPVerificationResult
@@ -49,9 +49,9 @@ class DefaultTOTPService(
     private const val DIGITS_QUERY_PARAM = "digits"
     private const val PERIOD_QUERY_PARAM = "period"
   }
-  override suspend fun generateTotpSecret(): TOTPSecret = totpConfiguration.secretProvider.generateSecret()
+  override fun generateTotpSecret(): TOTPSecret = totpConfiguration.secretProvider.generateSecret()
 
-  override suspend fun generateTOTPUrl(
+  override fun generateTOTPUrl(
     totpSecret: TOTPSecret,
     emailAddress: EmailAddress,
     issuer: Issuer
@@ -67,7 +67,7 @@ class DefaultTOTPService(
     return URI(template)
   }
 
-  override suspend fun verify(
+  override fun verify(
     code: TOTP,
     totpSecret: TOTPSecret
   ): TOTPVerificationResult {
diff --git a/src/test/kotlin/com/atlassian/onetime/core/HOTPGeneratorTest.kt b/src/test/kotlin/com/atlassian/onetime/core/HOTPGeneratorTest.kt
index ae0a964..04298fa 100644
--- a/src/test/kotlin/com/atlassian/onetime/core/HOTPGeneratorTest.kt
+++ b/src/test/kotlin/com/atlassian/onetime/core/HOTPGeneratorTest.kt
@@ -127,7 +127,7 @@ class HOTPGeneratorTest : FunSpec() {
     }
   }
 
-  private suspend fun given(state: TestState = TestState(), test: suspend TestState.(HOTPGenerator) -> Unit) {
+  private fun given(state: TestState = TestState(), test: TestState.(HOTPGenerator) -> Unit) {
     with(state) {
       test(state.hotpGenerator)
     }
diff --git a/src/test/kotlin/com/atlassian/onetime/core/TOTPGeneratorTest.kt b/src/test/kotlin/com/atlassian/onetime/core/TOTPGeneratorTest.kt
index a07d7f2..284b6b1 100644
--- a/src/test/kotlin/com/atlassian/onetime/core/TOTPGeneratorTest.kt
+++ b/src/test/kotlin/com/atlassian/onetime/core/TOTPGeneratorTest.kt
@@ -118,7 +118,7 @@ class TOTPGeneratorTest : FunSpec() {
     }
   }
 
-  private suspend fun given(state: TestState = TestState(), test: suspend TestState.(TOTPGenerator) -> Unit) {
+  private fun given(state: TestState = TestState(), test: TestState.(TOTPGenerator) -> Unit) {
     with(state) {
       test(state.totpGenerator)
     }
diff --git a/src/test/kotlin/com/atlassian/onetime/service/DefaultTOTPServiceTest.kt b/src/test/kotlin/com/atlassian/onetime/service/DefaultTOTPServiceTest.kt
index 7878c82..c0b8f9f 100644
--- a/src/test/kotlin/com/atlassian/onetime/service/DefaultTOTPServiceTest.kt
+++ b/src/test/kotlin/com/atlassian/onetime/service/DefaultTOTPServiceTest.kt
@@ -251,7 +251,7 @@ class DefaultTOTPServiceTest : FunSpec({
   }
 })
 
-private suspend fun given(state: TestState = TestState(), test: suspend TestState.(DefaultTOTPService) -> Unit) {
+private fun given(state: TestState = TestState(), test: TestState.(DefaultTOTPService) -> Unit) {
   with(state) {
     test(state.defaultTOTPService)
   }