Chore: 테스트 코드 구체적인 예외 타입 명시

Author: wjdrjs00

core/security/src/main/java/com/threegap/bitnagil/security/Crypto.kt

@@ -17,9 +17,16 @@ class Crypto(
 
     fun decrypt(bytes: ByteArray): ByteArray {
         val cipher = Cipher.getInstance(transformation)
+        require(bytes.size >= cipher.blockSize) {
+            INVALID_INPUT_TOO_SHORT_MSG
+        }
         val iv = bytes.copyOfRange(0, cipher.blockSize)
         val data = bytes.copyOfRange(cipher.blockSize, bytes.size)
         cipher.init(Cipher.DECRYPT_MODE, keyProvider.getKey(), IvParameterSpec(iv))
         return cipher.doFinal(data)
     }
+
+    companion object {
+        private const val INVALID_INPUT_TOO_SHORT_MSG = "[ERROR] 복호화할 수 없는 입력입니다."
+    }
 }

core/security/src/test/java/com/threegap/bitnagil/security/CryptoTest.kt

@@ -4,6 +4,7 @@ import org.junit.Assert.assertEquals
 import org.junit.Assert.assertNotEquals
 import org.junit.Assert.assertThrows
 import org.junit.Test
+import javax.crypto.BadPaddingException
 import javax.crypto.KeyGenerator
 import javax.crypto.SecretKey
 
@@ -51,16 +52,24 @@ class CryptoTest {
     }
 
     @Test
-    fun `잘못된 데이터로 복호화를 시도하면 예외가 발생해야 한다`() {
+    fun `입력값이 너무 짧은 경우 IllegalArgumentException이 발생해야 한다`() {
         // given
         val invalid = ByteArray(4) { 0x00 }
 
         // when & then
-        assertThrows(Exception::class.java) {
+        assertThrows(IllegalArgumentException::class.java) {
             crypto.decrypt(invalid)
         }
     }
 
+    @Test
+    fun `빈 바이트 배열 암호화 시 예외가 발생하지 않아야 한다`() {
+        val input = ByteArray(0)
+        val encrypted = crypto.encrypt(input)
+        val decrypted = crypto.decrypt(encrypted)
+        assertEquals(String(input), String(decrypted))
+    }
+
     @Test
     fun `IV 일부가 조작된 경우 복호화하면 원래 데이터와 달라야 한다`() {
         // given
@@ -83,7 +92,7 @@ class CryptoTest {
         encrypted[encrypted.lastIndex] = encrypted.last().inc()
 
         // when & then
-        assertThrows(Exception::class.java) {
+        assertThrows(BadPaddingException::class.java) {
             crypto.decrypt(encrypted)
         }
     }
@@ -106,7 +115,7 @@ class CryptoTest {
         val otherCrypto = Crypto(otherKeyProvider, "AES/CBC/PKCS5Padding")
 
         // when & then
-        assertThrows(Exception::class.java) {
+        assertThrows(BadPaddingException::class.java) {
             otherCrypto.decrypt(encrypted)
         }
     }