Handled Review commments from Zoe

Author: joviegas

.changes/next-release/bugfix-AmazonS3-bb097c9.json

@@ -2,5 +2,5 @@
     "type": "bugfix",
     "category": "Amazon S3",
     "contributor": "",
-    "description": "Fix StreamingRequestInterceptor to skip Expect: 100-continue header when PutObject or UploadPart requests have zero content length."
+    "description": "Skip Expect: 100-continue header for PutObject and UploadPart requests with zero content length"
 }

services/s3/src/test/java/software/amazon/awssdk/services/s3/functionaltests/Expect100ContinueHeaderTest.java

@@ -69,7 +69,7 @@
  */
 @WireMockTest(httpsEnabled = true)
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
-public class Expect100ContinueHeaderTest {
+class Expect100ContinueHeaderTest {
 
     private static final String BUCKET = "test-bucket";
     private static final String KEY = "test-key";
@@ -216,6 +216,7 @@ public void request(long n) {
 
                 @Override
                 public void cancel() {
+                    // No action.
                 }
             });
         };
@@ -246,6 +247,7 @@ public void request(long n) {
 
                 @Override
                 public void cancel() {
+                    // No action.
                 }
             });
         };

services/s3/src/test/java/software/amazon/awssdk/services/s3/internal/handlers/StreamingRequestInterceptorTest.java

@@ -67,7 +67,7 @@ public void modifyHttpRequest_doesNotSetExpect_whenSdkRequestIsNotPutObject() {
 
     @ParameterizedTest(name = "{0} with {1}={2} should not set Expect header")
     @MethodSource("zeroContentLengthProvider")
-    public void modifyHttpRequest_doesNotSetExpect_whenContentLengthIsZero(
+    void modifyHttpRequest_doesNotSetExpect_whenContentLengthIsZero(
         String requestType, String headerName, String headerValue, SdkRequest sdkRequest) {
 
         SdkHttpRequest httpRequest = buildHttpRequest(headerName, headerValue);
@@ -83,7 +83,7 @@ public void modifyHttpRequest_doesNotSetExpect_whenContentLengthIsZero(
 
     @ParameterizedTest(name = "{0} with {1}={2} should set Expect header")
     @MethodSource("nonZeroContentLengthProvider")
-    public void modifyHttpRequest_setsExpect_whenContentLengthIsNonZero(
+    void modifyHttpRequest_setsExpect_whenContentLengthIsNonZero(
         String requestType, String headerName, String headerValue, SdkRequest sdkRequest) {
 
         SdkHttpRequest httpRequest = buildHttpRequest(headerName, headerValue);
@@ -96,7 +96,7 @@ public void modifyHttpRequest_setsExpect_whenContentLengthIsNonZero(
     }
 
     @Test
-    public void modifyHttpRequest_prioritizesDecodedContentLength_overContentLength() {
+    void modifyHttpRequest_prioritizesDecodedContentLength_overContentLength() {
         SdkHttpRequest httpRequest = SdkHttpFullRequest.builder()
                                                        .uri(URI.create("http://localhost:8080"))
                                                        .method(SdkHttpMethod.PUT)