Comment out entire test

Author: alextwoods

services-custom/s3-transfer-manager/src/it/java/software/amazon/awssdk/transfer/s3/S3TransferManagerDownloadPauseResumeIntegrationTest.java

@@ -131,36 +131,36 @@ void pauseAndResume_ObjectEtagChange_shouldRestartDownload(S3TransferManager tm)
         }
     }
 
-    @ParameterizedTest
-    @MethodSource("transferManagers")
-    void pauseAndResume_ObjectNotChanged_shouldResumeDownload(S3TransferManager tm) {
-        Path path = RandomTempFile.randomUncreatedFile().toPath();
-        TestDownloadListener testDownloadListener = new TestDownloadListener();
-        DownloadFileRequest request = DownloadFileRequest.builder()
-                                                         .getObjectRequest(b -> b.bucket(BUCKET).key(KEY))
-                                                         .destination(path)
-                                                         .addTransferListener(testDownloadListener)
-                                                         .build();
-        FileDownload download = tm.downloadFile(request);
-        waitUntilFirstByteBufferDelivered(download);
-
-        ResumableFileDownload resumableFileDownload = download.pause();
-        long bytesTransferred = resumableFileDownload.bytesTransferred();
-        log.debug(() -> "Paused: " + resumableFileDownload);
-        assertEqualsBySdkFields(resumableFileDownload.downloadFileRequest(), request);
-        assertThat(testDownloadListener.getObjectResponse).isNotNull();
-        assertThat(resumableFileDownload.s3ObjectLastModified()).hasValue(testDownloadListener.getObjectResponse.lastModified());
-        assertThat(bytesTransferred).isEqualTo(path.toFile().length());
-        assertThat(resumableFileDownload.totalSizeInBytes()).hasValue(sourceFile.length());
-
-        //TODO: Fix this test to ensure that pause happens after bytes written but never before complete.
-        // depending on the timing of waitUntilFirstByteBufferDelivered, the entire file may have been downloaded
-        assertThat(bytesTransferred).isLessThanOrEqualTo(sourceFile.length());
-        assertThat(download.completionFuture()).isCancelled();
-
-        log.debug(() -> "Resuming download ");
-        verifyFileDownload(path, resumableFileDownload, OBJ_SIZE - bytesTransferred, tm);
-    }
+    //TODO: This test currently flakey. Fix this test to ensure that pause happens after bytes written but never before complete.
+    // depending on the timing of waitUntilFirstByteBufferDelivered, the entire file may have been downloaded
+    //
+    // @ParameterizedTest
+    // @MethodSource("transferManagers")
+    // void pauseAndResume_ObjectNotChanged_shouldResumeDownload(S3TransferManager tm) {
+    //     Path path = RandomTempFile.randomUncreatedFile().toPath();
+    //     TestDownloadListener testDownloadListener = new TestDownloadListener();
+    //     DownloadFileRequest request = DownloadFileRequest.builder()
+    //                                                      .getObjectRequest(b -> b.bucket(BUCKET).key(KEY))
+    //                                                      .destination(path)
+    //                                                      .addTransferListener(testDownloadListener)
+    //                                                      .build();
+    //     FileDownload download = tm.downloadFile(request);
+    //     waitUntilFirstByteBufferDelivered(download);
+    //
+    //     ResumableFileDownload resumableFileDownload = download.pause();
+    //     long bytesTransferred = resumableFileDownload.bytesTransferred();
+    //     log.debug(() -> "Paused: " + resumableFileDownload);
+    //     assertEqualsBySdkFields(resumableFileDownload.downloadFileRequest(), request);
+    //     assertThat(testDownloadListener.getObjectResponse).isNotNull();
+    //     assertThat(resumableFileDownload.s3ObjectLastModified()).hasValue(testDownloadListener.getObjectResponse.lastModified());
+    //     assertThat(bytesTransferred).isEqualTo(path.toFile().length());
+    //     assertThat(resumableFileDownload.totalSizeInBytes()).hasValue(sourceFile.length());
+    //     assertThat(bytesTransferred).isLessThan(sourceFile.length());
+    //     assertThat(download.completionFuture()).isCancelled();
+    //
+    //     log.debug(() -> "Resuming download ");
+    //     verifyFileDownload(path, resumableFileDownload, OBJ_SIZE - bytesTransferred, tm);
+    // }
 
     private void assertEqualsBySdkFields(DownloadFileRequest actual, DownloadFileRequest expected) {
         // Transfer manager adds an execution attribute to the GetObjectRequest, so both objects are different.