Remove AmazonWebServiceResponse as base class for transfer utility repsonse objects.

stack-info: PR: #4087, branch: GarrettBeatty/stacked/12

Author: GarrettBeatty

generator/.DevConfigs/c49077d9-90b3-437f-b316-6d8d8833ae75.json

@@ -0,0 +1,11 @@
+{
+  "services": [
+    {
+      "serviceName": "S3",
+      "type": "patch",
+      "changeLogMessages": [
+        "Remove AmazonWebServiceResponse as base class for transfer utility repsonse objects."
+      ]
+    }
+  ]
+}

sdk/src/Services/S3/Custom/Transfer/Internal/ResponseMapper.cs

@@ -61,12 +61,6 @@ internal static TransferUtilityUploadResponse MapPutObjectResponse(PutObjectResp
             response.ServerSideEncryptionKeyManagementServiceKeyId = source.ServerSideEncryptionKeyManagementServiceKeyId;
             response.ServerSideEncryptionMethod = source.ServerSideEncryptionMethod;
             response.VersionId = source.VersionId;
-
-            // Copy response metadata
-            response.ResponseMetadata = source.ResponseMetadata;
-            response.ContentLength = source.ContentLength;
-            response.HttpStatusCode = source.HttpStatusCode;
-
             return response;
         }
 
@@ -97,12 +91,6 @@ internal static TransferUtilityUploadResponse MapCompleteMultipartUploadResponse
             response.ServerSideEncryptionMethod = source.ServerSideEncryptionMethod;
             response.ServerSideEncryptionKeyManagementServiceKeyId = source.ServerSideEncryptionKeyManagementServiceKeyId;
             response.VersionId = source.VersionId;
-
-            // Copy response metadata
-            response.ResponseMetadata = source.ResponseMetadata;
-            response.ContentLength = source.ContentLength;
-            response.HttpStatusCode = source.HttpStatusCode;
-
             return response;
         }
 
@@ -153,12 +141,6 @@ internal static TransferUtilityDownloadResponse MapGetObjectResponse(GetObjectRe
             response.TagCount = source.TagCount;
             response.VersionId = source.VersionId;
             response.WebsiteRedirectLocation = source.WebsiteRedirectLocation;
-
-            // Copy response metadata
-            response.ResponseMetadata = source.ResponseMetadata;
-            response.ContentLength = source.ContentLength;
-            response.HttpStatusCode = source.HttpStatusCode;
-
             return response;
         }
 

sdk/src/Services/S3/Custom/Transfer/TransferUtilityDownloadResponse.cs

@@ -31,7 +31,7 @@ namespace Amazon.S3.Transfer
     /// Response object for Transfer Utility download operations.
     /// Contains response metadata from download operations.
     /// </summary>
-    public class TransferUtilityDownloadResponse : AmazonWebServiceResponse
+    public class TransferUtilityDownloadResponse
     {
         /// <summary>
         /// Gets and sets the AcceptRanges property.

sdk/src/Services/S3/Custom/Transfer/TransferUtilityUploadResponse.cs

@@ -32,7 +32,7 @@ namespace Amazon.S3.Transfer
     /// Contains unified response fields from both simple uploads (PutObjectResponse) 
     /// and multipart uploads (CompleteMultipartUploadResponse).
     /// </summary>
-    public class TransferUtilityUploadResponse : AmazonWebServiceResponse
+    public class TransferUtilityUploadResponse
     {
         private bool? _bucketKeyEnabled;
         private string _checksumCRC32;

sdk/test/Services/S3/UnitTests/Custom/ResponseMapperTests.cs

@@ -149,8 +149,7 @@ public void MapPutObjectResponse_AllMappedProperties_WorkCorrectly()
                 },
                 (sourceResponse, targetResponse) =>
                 {
-                    Assert.AreEqual(sourceResponse.HttpStatusCode, targetResponse.HttpStatusCode, "HttpStatusCode should match");
-                    Assert.AreEqual(sourceResponse.ContentLength, targetResponse.ContentLength, "ContentLength should match");
+                  
                 });
         }
 
@@ -524,12 +523,11 @@ public void MapCompleteMultipartUploadResponse_AllMappedProperties_WorkCorrectly
                 (sourceResponse) =>
                 {
                     sourceResponse.HttpStatusCode = HttpStatusCode.OK;
-                    sourceResponse.ContentLength = 2048;
+                    sourceResponse.ContentLength = 1024;
                 },
                 (sourceResponse, targetResponse) =>
                 {
-                    Assert.AreEqual(sourceResponse.HttpStatusCode, targetResponse.HttpStatusCode, "HttpStatusCode should match");
-                    Assert.AreEqual(sourceResponse.ContentLength, targetResponse.ContentLength, "ContentLength should match");
+                    
                 });
         }
 
@@ -600,12 +598,11 @@ public void MapGetObjectResponse_AllMappedProperties_WorkCorrectly()
                 (sourceResponse) =>
                 {
                     sourceResponse.HttpStatusCode = HttpStatusCode.OK;
-                    sourceResponse.ContentLength = 2048;
+                    sourceResponse.ContentLength = 1024;
                 },
                 (sourceResponse, targetResponse) =>
                 {
-                    Assert.AreEqual(sourceResponse.HttpStatusCode, targetResponse.HttpStatusCode, "HttpStatusCode should match");
-                    Assert.AreEqual(sourceResponse.ContentLength, targetResponse.ContentLength, "ContentLength should match");
+                    
                 });
         }