Update unknown content length check

The original assumption was brittle and missed some types. Make the
assumption check simpler and less error prone.

Author: dagnir

test/s3-tests/src/it/java/software/amazon/awssdk/services/s3/regression/upload/UploadAsyncRegressionTesting.java

@@ -64,11 +64,9 @@ void putObject(UploadConfig config) throws Exception {
                                 || config.getBucketType() == BucketType.MRAP,
                                 "Async payload signing doesn't work with Java based clients");
 
-        // For testing purposes, ContentProvider is Publisher<ByteBuffer> for async clients
-        // Async java based clients don't currently support unknown content-length bodies
-        Assumptions.assumeFalse(config.getBodyType() == BodyType.CONTENT_PROVIDER_NO_LENGTH
-                                || config.getBodyType() == BodyType.INPUTSTREAM_NO_LENGTH,
-                                "Async Java based support unknown content length");
+        // Async java based clients don't support unknown content-length bodies
+        Assumptions.assumeTrue(config.getBodyType().isContentLengthAvailable(),
+                               "Async Java based does not support unknown content length");
 
         LOG.info(() -> "Running UploadAsyncRegressionTesting putObject with config: " + config);
 

test/s3-tests/src/it/java/software/amazon/awssdk/services/s3/regression/upload/UploadStreamingRegressionTesting.java

@@ -397,29 +397,40 @@ void performWriteIfNeeded(TestAsyncBody requestBody) throws IOException {
     }
 
     protected enum BodyType {
-        INPUTSTREAM_RESETABLE,
-        INPUTSTREAM_NOT_RESETABLE,
-        INPUTSTREAM_NO_LENGTH,
+        INPUTSTREAM_RESETABLE(true),
+        INPUTSTREAM_NOT_RESETABLE(true),
+        INPUTSTREAM_NO_LENGTH(false),
 
-        STRING,
+        STRING(true),
 
-        FILE,
+        FILE(true),
 
-        CONTENT_PROVIDER_WITH_LENGTH,
+        CONTENT_PROVIDER_WITH_LENGTH(true),
 
-        CONTENT_PROVIDER_NO_LENGTH,
+        CONTENT_PROVIDER_NO_LENGTH(false),
 
-        BYTES,
-        BYTE_BUFFER,
-        REMAINING_BYTE_BUFFER,
+        BYTES(true),
+        BYTE_BUFFER(true),
+        REMAINING_BYTE_BUFFER(true),
 
-        BUFFERS,
-        BUFFERS_REMAINING,
+        BUFFERS(true),
+        BUFFERS_REMAINING(true),
 
-        BLOCKING_INPUT_STREAM,
-        BLOCKING_OUTPUT_STREAM,
-        BUFFERED_SPLITTABLE_KNOWN_CONTENT_LENGTH,
-        BUFFERED_SPLITTABLE_UNKNOWN_CONTENT_LENGTH
+        BLOCKING_INPUT_STREAM(true),
+        BLOCKING_OUTPUT_STREAM(true),
+        BUFFERED_SPLITTABLE_KNOWN_CONTENT_LENGTH(true),
+        BUFFERED_SPLITTABLE_UNKNOWN_CONTENT_LENGTH(false)
+        ;
+
+        private final boolean contentLengthAvailable;
+
+        BodyType(boolean contentLengthAvailable) {
+            this.contentLengthAvailable = contentLengthAvailable;
+        }
+
+        public boolean isContentLengthAvailable() {
+            return contentLengthAvailable;
+        }
     }
 
     protected enum ContentSize {