fix S3MultipartClientGetObjectWiremockTest

Author: L-Applin

services/s3/src/test/java/software/amazon/awssdk/services/s3/internal/multipart/S3MultipartClientGetObjectWiremockTest.java

@@ -147,9 +147,8 @@ public <T> void errorOnThirdPart_shouldCompleteExceptionallyOnlyPartsGreaterThan
         }
     }
 
-    // todo temporary, remove when support for resume is added to multipart file download
-    // @ParameterizedTest
-    // @MethodSource("partSizeAndTransformerParams")
+    @ParameterizedTest
+    @MethodSource("partSizeAndTransformerParams")
     public <T> void partCountValidationFailure_shouldThrowException(
         AsyncResponseTransformerTestSupplier<T> supplier,
         int partSize) {
@@ -176,7 +175,11 @@ public <T> void partCountValidationFailure_shouldThrowException(
 
         // Skip the lazy transformer since the error won't surface unless the content is consumed
         AsyncResponseTransformer<GetObjectResponse, T> transformer = supplier.transformer();
-        if (transformer instanceof InputStreamResponseTransformer || transformer instanceof PublisherAsyncResponseTransformer) {
+        if (transformer instanceof InputStreamResponseTransformer
+            || transformer instanceof PublisherAsyncResponseTransformer
+            // FileAsyncResponseTransformer will be used with ParallelMultipartDownloaderSubscriber for which this test logic
+            // doesn't work
+            || transformer instanceof FileAsyncResponseTransformer) {
             return;
         }
 
@@ -291,15 +294,17 @@ private static Stream<Arguments> partSizeAndTransformerParams() {
     }
 
 
+
+
     /**
      * Testing response transformers that are not retryable when
      * {@link AsyncResponseTransformer#split(SplittingTransformerConfiguration)} is invoked and used with
-     * {@link SplittingTransformer} - {@link PublisherAsyncResponseTransformer}, {@link InputStreamResponseTransformer}, and
-     * {@link FileAsyncResponseTransformer}
+     * {@link SplittingTransformer} - {@link PublisherAsyncResponseTransformer}, {@link InputStreamResponseTransformer}
      * <p>
      *
-     * Retry for multipart download is supported for {@link ByteArrayAsyncResponseTransformer}, tested in
-     * {@link S3MultipartClientGetObjectRetryBehaviorWiremockTest}.
+     * Retry for multipart download is supported for {@link ByteArrayAsyncResponseTransformer}
+     * and {@link FileAsyncResponseTransformer}, tested in {@link S3MultipartClientGetObjectRetryBehaviorWiremockTest} and
+     * {@link S3MultipartFileDownloadWiremockTest} respectively.
      */
     private static Stream<AsyncResponseTransformerTestSupplier<?>> nonRetryableResponseTransformers() {
         return Stream.of(new AsyncResponseTransformerTestSupplier.InputStreamArtSupplier(),