Update requestcompression codegen name (#4851)

davidh44 · web-flow · commit 9b1f20ac2bd4 · 2024-01-26T13:33:14.000-08:00
* Update codegen to get requestcompression instead of requestCompression

* Update test models
diff --git a/codegen/src/main/java/software/amazon/awssdk/codegen/AddOperations.java b/codegen/src/main/java/software/amazon/awssdk/codegen/AddOperations.java
@@ -176,7 +176,7 @@ public Map<String, OperationModel> constructOperations() {
             operationModel.setEndpointTrait(op.getEndpoint());
             operationModel.setHttpChecksumRequired(op.isHttpChecksumRequired());
             operationModel.setHttpChecksum(op.getHttpChecksum());
-            operationModel.setRequestCompression(op.getRequestCompression());
+            operationModel.setRequestcompression(op.getRequestcompression());
             operationModel.setStaticContextParams(op.getStaticContextParams());
             operationModel.setAuth(getAuthFromOperation(op));
 
diff --git a/codegen/src/main/java/software/amazon/awssdk/codegen/compression/RequestCompression.java b/codegen/src/main/java/software/amazon/awssdk/codegen/compression/RequestCompression.java
@@ -19,7 +19,7 @@
 import software.amazon.awssdk.annotations.SdkInternalApi;
 
 /**
- * Class to map the RequestCompression trait of an operation.
+ * Class to map the requestcompression trait of an operation.
  */
 @SdkInternalApi
 public class RequestCompression {
diff --git a/codegen/src/main/java/software/amazon/awssdk/codegen/model/intermediate/OperationModel.java b/codegen/src/main/java/software/amazon/awssdk/codegen/model/intermediate/OperationModel.java
@@ -78,7 +78,7 @@ public class OperationModel extends DocumentationModel {
 
     private HttpChecksum httpChecksum;
 
-    private RequestCompression requestCompression;
+    private RequestCompression requestcompression;
 
     @JsonIgnore
     private Map<String, StaticContextParam> staticContextParams;
@@ -342,12 +342,12 @@ public void setHttpChecksum(HttpChecksum httpChecksum) {
         this.httpChecksum = httpChecksum;
     }
 
-    public RequestCompression getRequestCompression() {
-        return requestCompression;
+    public RequestCompression getRequestcompression() {
+        return requestcompression;
     }
 
-    public void setRequestCompression(RequestCompression requestCompression) {
-        this.requestCompression = requestCompression;
+    public void setRequestcompression(RequestCompression requestcompression) {
+        this.requestcompression = requestcompression;
     }
 
     public Map<String, StaticContextParam> getStaticContextParams() {
diff --git a/codegen/src/main/java/software/amazon/awssdk/codegen/model/service/Operation.java b/codegen/src/main/java/software/amazon/awssdk/codegen/model/service/Operation.java
@@ -55,7 +55,7 @@ public class Operation {
 
     private HttpChecksum httpChecksum;
 
-    private RequestCompression requestCompression;
+    private RequestCompression requestcompression;
 
     private Map<String, StaticContextParam> staticContextParams;
 
@@ -202,12 +202,12 @@ public void setHttpChecksum(HttpChecksum httpChecksum) {
         this.httpChecksum = httpChecksum;
     }
 
-    public RequestCompression getRequestCompression() {
-        return requestCompression;
+    public RequestCompression getRequestcompression() {
+        return requestcompression;
     }
 
-    public void setRequestCompression(RequestCompression requestCompression) {
-        this.requestCompression = requestCompression;
+    public void setRequestcompression(RequestCompression requestcompression) {
+        this.requestcompression = requestcompression;
     }
 
     public Map<String, StaticContextParam> getStaticContextParams() {
diff --git a/codegen/src/main/java/software/amazon/awssdk/codegen/poet/client/traits/RequestCompressionTrait.java b/codegen/src/main/java/software/amazon/awssdk/codegen/poet/client/traits/RequestCompressionTrait.java
@@ -38,7 +38,7 @@ private RequestCompressionTrait() {
      * return an empty code-block.
      */
     public static CodeBlock create(OperationModel operationModel, IntermediateModel model) {
-        if (operationModel.getRequestCompression() == null) {
+        if (operationModel.getRequestcompression() == null) {
             return CodeBlock.of("");
         }
 
@@ -49,7 +49,7 @@ public static CodeBlock create(OperationModel operationModel, IntermediateModel
             throw new IllegalStateException("Request compression for S3 is not yet supported in the AWS SDK for Java.");
         }
 
-        List<String> encodings = operationModel.getRequestCompression().getEncodings();
+        List<String> encodings = operationModel.getRequestcompression().getEncodings();
 
         return CodeBlock.of(".putExecutionAttribute($T.REQUEST_COMPRESSION, "
                             + "$T.builder().encodings($L).isStreaming($L).build())",
diff --git a/codegen/src/test/resources/software/amazon/awssdk/codegen/poet/client/c2j/json/service-2.json b/codegen/src/test/resources/software/amazon/awssdk/codegen/poet/client/c2j/json/service-2.json
@@ -36,7 +36,7 @@
         "method": "POST",
         "requestUri": "/"
       },
-      "requestCompression": {
+      "requestcompression": {
         "encodings": ["gzip"]
       }
     },
diff --git a/codegen/src/test/resources/software/amazon/awssdk/codegen/poet/client/c2j/query/service-2.json b/codegen/src/test/resources/software/amazon/awssdk/codegen/poet/client/c2j/query/service-2.json
@@ -65,7 +65,7 @@
         "method": "POST",
         "requestUri": "/"
       },
-      "requestCompression": {
+      "requestcompression": {
         "encodings": ["gzip"]
       }
     },
diff --git a/codegen/src/test/resources/software/amazon/awssdk/codegen/poet/client/c2j/rest-json/service-2.json b/codegen/src/test/resources/software/amazon/awssdk/codegen/poet/client/c2j/rest-json/service-2.json
@@ -28,7 +28,7 @@
         "method": "POST",
         "requestUri": "/"
       },
-      "requestCompression": {
+      "requestcompression": {
         "encodings": ["gzip"]
       }
     },
diff --git a/codegen/src/test/resources/software/amazon/awssdk/codegen/poet/client/c2j/xml/service-2.json b/codegen/src/test/resources/software/amazon/awssdk/codegen/poet/client/c2j/xml/service-2.json
@@ -35,7 +35,7 @@
         "method": "POST",
         "requestUri": "/"
       },
-      "requestCompression": {
+      "requestcompression": {
         "encodings": ["gzip"]
       }
     },
diff --git a/test/codegen-generated-classes-test/src/main/resources/codegen-resources/customresponsemetadata/service-2.json b/test/codegen-generated-classes-test/src/main/resources/codegen-resources/customresponsemetadata/service-2.json
@@ -294,7 +294,7 @@
       },
       "input":{"shape":"RequestCompressionStructure"},
       "output":{"shape":"RequestCompressionStructure"},
-      "requestCompression": {
+      "requestcompression": {
         "encodings": ["gzip"]
       }
     },
@@ -306,7 +306,7 @@
       },
       "input":{"shape":"RequestCompressionStructureWithStreaming"},
       "output":{"shape":"RequestCompressionStructureWithStreaming"},
-      "requestCompression": {
+      "requestcompression": {
         "encodings": ["gzip"]
       },
       "authtype":"v4-unsigned-body"

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ public class OperationModel extends DocumentationModel {`
`78`	`78`
`79`	`79`	`private HttpChecksum httpChecksum;`
`80`	`80`
`81`		`- private RequestCompression requestCompression;`
	`81`	`+ private RequestCompression requestcompression;`
`82`	`82`
`83`	`83`	`@JsonIgnore`
`84`	`84`	`private Map<String, StaticContextParam> staticContextParams;`
`@@ -342,12 +342,12 @@ public void setHttpChecksum(HttpChecksum httpChecksum) {`
`342`	`342`	`this.httpChecksum = httpChecksum;`
`343`	`343`	`}`
`344`	`344`
`345`		`- public RequestCompression getRequestCompression() {`
`346`		`- return requestCompression;`
	`345`	`+ public RequestCompression getRequestcompression() {`
	`346`	`+ return requestcompression;`
`347`	`347`	`}`
`348`	`348`
`349`		`- public void setRequestCompression(RequestCompression requestCompression) {`
`350`		`- this.requestCompression = requestCompression;`
	`349`	`+ public void setRequestcompression(RequestCompression requestcompression) {`
	`350`	`+ this.requestcompression = requestcompression;`
`351`	`351`	`}`
`352`	`352`
`353`	`353`	`public Map<String, StaticContextParam> getStaticContextParams() {`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public class Operation {`
`55`	`55`
`56`	`56`	`private HttpChecksum httpChecksum;`
`57`	`57`
`58`		`- private RequestCompression requestCompression;`
	`58`	`+ private RequestCompression requestcompression;`
`59`	`59`
`60`	`60`	`private Map<String, StaticContextParam> staticContextParams;`
`61`	`61`
`@@ -202,12 +202,12 @@ public void setHttpChecksum(HttpChecksum httpChecksum) {`
`202`	`202`	`this.httpChecksum = httpChecksum;`
`203`	`203`	`}`
`204`	`204`
`205`		`- public RequestCompression getRequestCompression() {`
`206`		`- return requestCompression;`
	`205`	`+ public RequestCompression getRequestcompression() {`
	`206`	`+ return requestcompression;`
`207`	`207`	`}`
`208`	`208`
`209`		`- public void setRequestCompression(RequestCompression requestCompression) {`
`210`		`- this.requestCompression = requestCompression;`
	`209`	`+ public void setRequestcompression(RequestCompression requestcompression) {`
	`210`	`+ this.requestcompression = requestcompression;`
`211`	`211`	`}`
`212`	`212`
`213`	`213`	`public Map<String, StaticContextParam> getStaticContextParams() {`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ private RequestCompressionTrait() {`
`38`	`38`	`* return an empty code-block.`
`39`	`39`	`*/`
`40`	`40`	`public static CodeBlock create(OperationModel operationModel, IntermediateModel model) {`
`41`		`- if (operationModel.getRequestCompression() == null) {`
	`41`	`+ if (operationModel.getRequestcompression() == null) {`
`42`	`42`	`return CodeBlock.of("");`
`43`	`43`	`}`
`44`	`44`
`@@ -49,7 +49,7 @@ public static CodeBlock create(OperationModel operationModel, IntermediateModel`
`49`	`49`	`throw new IllegalStateException("Request compression for S3 is not yet supported in the AWS SDK for Java.");`
`50`	`50`	`}`
`51`	`51`
`52`		`- List<String> encodings = operationModel.getRequestCompression().getEncodings();`
	`52`	`+ List<String> encodings = operationModel.getRequestcompression().getEncodings();`
`53`	`53`
`54`	`54`	`return CodeBlock.of(".putExecutionAttribute($T.REQUEST_COMPRESSION, "`
`55`	`55`	`+ "$T.builder().encodings($L).isStreaming($L).build())",`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@`
`36`	`36`	`"method": "POST",`
`37`	`37`	`"requestUri": "/"`
`38`	`38`	`},`
`39`		`- "requestCompression": {`
	`39`	`+ "requestcompression": {`
`40`	`40`	`"encodings": ["gzip"]`
`41`	`41`	`}`
`42`	`42`	`},`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@`
`65`	`65`	`"method": "POST",`
`66`	`66`	`"requestUri": "/"`
`67`	`67`	`},`
`68`		`- "requestCompression": {`
	`68`	`+ "requestcompression": {`
`69`	`69`	`"encodings": ["gzip"]`
`70`	`70`	`}`
`71`	`71`	`},`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@`
`28`	`28`	`"method": "POST",`
`29`	`29`	`"requestUri": "/"`
`30`	`30`	`},`
`31`		`- "requestCompression": {`
	`31`	`+ "requestcompression": {`
`32`	`32`	`"encodings": ["gzip"]`
`33`	`33`	`}`
`34`	`34`	`},`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@`
`35`	`35`	`"method": "POST",`
`36`	`36`	`"requestUri": "/"`
`37`	`37`	`},`
`38`		`- "requestCompression": {`
	`38`	`+ "requestcompression": {`
`39`	`39`	`"encodings": ["gzip"]`
`40`	`40`	`}`
`41`	`41`	`},`