increase test coverage

xinsong-cui · xinsong-cui · commit d86b08ddbc2e · 2025-03-11T17:23:46.000-04:00
diff --git a/services/sqs/e2eTest/src/SqsMd5ChecksumValidationTest.kt b/services/sqs/e2eTest/src/SqsMd5ChecksumValidationTest.kt
@@ -46,17 +46,14 @@ class SqsMd5ChecksumValidationTest {
             when (val response = context.response.getOrNull()) {
                 is SendMessageResponse -> {
                     val modifiedResponse = response.copy {
-                        md5OfMessageBody = wrongMd5ofMessageBody
                         md5OfMessageAttributes = wrongMd5ofMessageAttribute
-                        md5OfMessageSystemAttributes = wrongMd5ofMessageSystemAttribute
                     }
                     return Result.success(modifiedResponse)
                 }
                 is ReceiveMessageResponse -> {
                     val modifiedMessages = response.messages?.map { message ->
                         message.copy {
                             md5OfBody = wrongMd5ofMessageBody
-                            md5OfMessageAttributes = wrongMd5ofMessageAttribute
                         }
                     }
 
@@ -68,8 +65,6 @@ class SqsMd5ChecksumValidationTest {
                 is SendMessageBatchResponse -> {
                     val modifiedEntries = response.successful.map { entry ->
                         entry.copy {
-                            md5OfMessageBody = wrongMd5ofMessageBody
-                            md5OfMessageAttributes = wrongMd5ofMessageAttribute
                             md5OfMessageSystemAttributes = wrongMd5ofMessageSystemAttribute
                         }
                     }
@@ -147,6 +142,10 @@ class SqsMd5ChecksumValidationTest {
                         dataType = "String"
                         stringValue = TEST_MESSAGE_ATTRIBUTES_VALUE
                     },
+                    TEST_MESSAGE_ATTRIBUTES_NAME to MessageAttributeValue {
+                        dataType = "Binary"
+                        binaryValue = TEST_MESSAGE_ATTRIBUTES_VALUE.toByteArray()
+                    },
                 )
                 messageSystemAttributes = mapOf(
                     MessageSystemAttributeNameForSends.AwsTraceHeader to MessageSystemAttributeValue {

Original file line number	Diff line number	Diff line change
`@@ -46,17 +46,14 @@ class SqsMd5ChecksumValidationTest {`
`46`	`46`	`when (val response = context.response.getOrNull()) {`
`47`	`47`	`is SendMessageResponse -> {`
`48`	`48`	`val modifiedResponse = response.copy {`
`49`		`- md5OfMessageBody = wrongMd5ofMessageBody`
`50`	`49`	`md5OfMessageAttributes = wrongMd5ofMessageAttribute`
`51`		`- md5OfMessageSystemAttributes = wrongMd5ofMessageSystemAttribute`
`52`	`50`	`}`
`53`	`51`	`return Result.success(modifiedResponse)`
`54`	`52`	`}`
`55`	`53`	`is ReceiveMessageResponse -> {`
`56`	`54`	`val modifiedMessages = response.messages?.map { message ->`
`57`	`55`	`message.copy {`
`58`	`56`	`md5OfBody = wrongMd5ofMessageBody`
`59`		`- md5OfMessageAttributes = wrongMd5ofMessageAttribute`
`60`	`57`	`}`
`61`	`58`	`}`
`62`	`59`
`@@ -68,8 +65,6 @@ class SqsMd5ChecksumValidationTest {`
`68`	`65`	`is SendMessageBatchResponse -> {`
`69`	`66`	`val modifiedEntries = response.successful.map { entry ->`
`70`	`67`	`entry.copy {`
`71`		`- md5OfMessageBody = wrongMd5ofMessageBody`
`72`		`- md5OfMessageAttributes = wrongMd5ofMessageAttribute`
`73`	`68`	`md5OfMessageSystemAttributes = wrongMd5ofMessageSystemAttribute`
`74`	`69`	`}`
`75`	`70`	`}`
`@@ -147,6 +142,10 @@ class SqsMd5ChecksumValidationTest {`
`147`	`142`	`dataType = "String"`
`148`	`143`	`stringValue = TEST_MESSAGE_ATTRIBUTES_VALUE`
`149`	`144`	`},`
	`145`	`+ TEST_MESSAGE_ATTRIBUTES_NAME to MessageAttributeValue {`
	`146`	`+ dataType = "Binary"`
	`147`	`+ binaryValue = TEST_MESSAGE_ATTRIBUTES_VALUE.toByteArray()`
	`148`	`+ },`
`150`	`149`	`)`
`151`	`150`	`messageSystemAttributes = mapOf(`
`152`	`151`	`MessageSystemAttributeNameForSends.AwsTraceHeader to MessageSystemAttributeValue {`