Add test for unquoted if-none-match header

Author: afranken

integration-tests/src/test/kotlin/com/adobe/testing/s3mock/its/GetPutDeleteObjectIT.kt

@@ -226,24 +226,6 @@ internal class GetPutDeleteObjectIT : S3TestBase() {
     // .hasMessageContaining(NO_SUCH_KEY)
   }
 
-  @Test
-  @S3VerifiedSuccess(year = 2025)
-  fun `GET object succeeds with unquoted if-match header`(testInfo: TestInfo) {
-    val (bucketName, putObjectResponse) = givenBucketAndObject(testInfo, UPLOAD_FILE_NAME)
-    val matchingEtag = putObjectResponse.eTag()
-    // TODO: should work with both quoted and unquoted etags
-    val unquotedEtag = matchingEtag.substring(1, matchingEtag.length - 1)
-    s3Client
-      .getObject {
-        it.bucket(bucketName)
-        it.key(UPLOAD_FILE_NAME)
-        it.ifMatch(matchingEtag)
-      }.use {
-        assertThat(it.response().eTag()).isEqualTo(matchingEtag)
-        assertThat(it.response().contentLength()).isEqualTo(UPLOAD_FILE_LENGTH)
-      }
-  }
-
   @Test
   @S3VerifiedSuccess(year = 2025)
   fun copyObjectToNonExistingDestination_noSuchBucket(testInfo: TestInfo) {
@@ -1354,6 +1336,24 @@ internal class GetPutDeleteObjectIT : S3TestBase() {
       }
   }
 
+  @Test
+  @S3VerifiedSuccess(year = 2025)
+  fun `GET object succeeds with unquoted if-match=true`(testInfo: TestInfo) {
+    val (bucketName, putObjectResponse) = givenBucketAndObject(testInfo, UPLOAD_FILE_NAME)
+    val matchingEtag = putObjectResponse.eTag()
+    // TODO: should work with both quoted and unquoted etags
+    val unquotedEtag = matchingEtag.substring(1, matchingEtag.length - 1)
+    s3Client
+      .getObject {
+        it.bucket(bucketName)
+        it.key(UPLOAD_FILE_NAME)
+        it.ifMatch(matchingEtag)
+      }.use {
+        assertThat(it.response().eTag()).isEqualTo(matchingEtag)
+        assertThat(it.response().contentLength()).isEqualTo(UPLOAD_FILE_LENGTH)
+      }
+  }
+
   @Test
   @S3VerifiedSuccess(year = 2025)
   fun `GET object succeeds with if-match=true and if-unmodified-since=false`(testInfo: TestInfo) {
@@ -1425,6 +1425,24 @@ internal class GetPutDeleteObjectIT : S3TestBase() {
       .hasMessageContaining("Service: S3, Status Code: 304")
   }
 
+  @Test
+  @S3VerifiedSuccess(year = 2025)
+  fun `GET object fails with unquoted if-none-match=false`(testInfo: TestInfo) {
+    val (bucketName, putObjectResponse) = givenBucketAndObject(testInfo, UPLOAD_FILE_NAME)
+    val matchingEtag = putObjectResponse.eTag()
+    // TODO: should work with both quoted and unquoted etags
+    val unquotedEtag = matchingEtag.substring(1, matchingEtag.length - 1)
+
+    assertThatThrownBy {
+      s3Client.getObject {
+        it.bucket(bucketName)
+        it.key(UPLOAD_FILE_NAME)
+        it.ifNoneMatch(matchingEtag)
+      }
+    }.isInstanceOf(S3Exception::class.java)
+      .hasMessageContaining("Service: S3, Status Code: 304")
+  }
+
   @Test
   @S3VerifiedSuccess(year = 2025)
   fun `GET object succeeds with if-modified-since=true`(testInfo: TestInfo) {