From 95386d3f75524d66d59427a38fb0d5eced0bd57c Mon Sep 17 00:00:00 2001 From: Matas Date: Fri, 1 Mar 2024 11:52:01 -0500 Subject: [PATCH] misc: correct test assertion and `@Ignore` it (#1239) --- services/s3/e2eTest/src/S3ExpressTest.kt | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/services/s3/e2eTest/src/S3ExpressTest.kt b/services/s3/e2eTest/src/S3ExpressTest.kt index 84538539521..5724d3df3ad 100644 --- a/services/s3/e2eTest/src/S3ExpressTest.kt +++ b/services/s3/e2eTest/src/S3ExpressTest.kt @@ -83,6 +83,7 @@ class S3ExpressTest { } } + @Ignore @Test fun testPresignedPutObject() = runTest { val content = "Presign this!" @@ -98,11 +99,9 @@ class S3ExpressTest { 5.minutes, ) - assertTrue(presigned.url.parameters.decodedParameters.contains("X-Amz-Security-Token")) - // FIXME Presigned requests should use S3 Express Auth Scheme resulting in `X-Amz-S3session-Token` // https://github.com/awslabs/aws-sdk-kotlin/issues/1236 - assertFalse(presigned.url.parameters.decodedParameters.contains(S3_EXPRESS_SESSION_TOKEN_HEADER)) + assertTrue(presigned.url.parameters.decodedParameters.contains(S3_EXPRESS_SESSION_TOKEN_HEADER)) } }