fix(email-service): fix to update entity when re-request email token (not save new entity)

Author: 0JIEUN0

src/main/kotlin/com/duckbox/service/EmailService.kt

@@ -48,16 +48,24 @@ class EmailService (
         }
         val token = code.toString()
 
-        // create EmailAuth entity
         val expirationTime = createTime + EXPIRATION_TIME
-        emailAuthRepository.save(
-            EmailAuth(
-            email = targetEmail,
-            token = token,
-            expirationTime = expirationTime,
-            expired = false
-        )
-        )
+        runCatching {
+            emailAuthRepository.findByEmail(targetEmail)
+        }.onSuccess {
+            // re-request
+            it.token = token
+            it.expirationTime = expirationTime
+            it.expired = false
+            emailAuthRepository.save(it)
+        }.onFailure {
+            // create EmailAuth entity
+            emailAuthRepository.save(EmailAuth(
+                email = targetEmail,
+                token = token,
+                expirationTime = expirationTime,
+                expired = false
+            ))
+        }
 
         return token
     }

src/test/kotlin/com/duckbox/service/EmailServiceTest.kt

@@ -42,7 +42,24 @@ class EmailServiceTest {
         emailAuth.apply {
             assertThat(email).isEqualTo(testEmail)
             assertThat(expired).isEqualTo(false)
-            assertThat(token).isEqualTo(token)
+            assertThat(this.token).isEqualTo(token)
+        }
+    }
+
+    @Test
+    fun is_createEmailToken_works_well_on_re_request() {
+        // act
+        val token: String = emailService.createEmailToken(testEmail)
+        val token2: String = emailService.createEmailToken(testEmail)
+
+        // assert
+        assertThat(token.length).isEqualTo(6)
+        assertThat(token).isNotEqualTo(token2)
+        val emailAuth = emailAuthRepository.findByEmail(testEmail)
+        emailAuth.apply {
+            assertThat(email).isEqualTo(testEmail)
+            assertThat(expired).isEqualTo(false)
+            assertThat(this.token).isEqualTo(token2)
         }
     }