chore: cleanup

Author: diegomarquezp

sdk-platform-java/gapic-generator-java/src/main/java/com/google/api/generator/gapic/composer/rest/HttpJsonServiceStubClassComposer.java

@@ -925,10 +925,7 @@ private Expr createFieldsExtractorClassInstance(
 
       paramsPutArgs.add(
           ValueExpr.withValue(
-              StringObjectValue.withValue(
-                  (httpBindingFieldName.jsonName() != null)
-                      ? httpBindingFieldName.jsonName()
-                      : JavaStyle.toLowerCamelCase(httpBindingFieldName.name()))));
+              StringObjectValue.withValue(httpBindingFieldName.jsonName())));
       paramsPutArgs.add(requestBuilderExpr);
 
       Expr paramsPutExpr =

sdk-platform-java/gapic-generator-java/src/main/java/com/google/api/generator/gapic/protoparser/HttpRuleParser.java

@@ -147,7 +147,7 @@ private static Set<HttpBinding> validateAndConstructHttpBindings(
         continue;
       }
       Message nestedMessage = inputMessage;
-      List<String> jsonNameParts = new ArrayList();
+      List<String> jsonNameParts = new ArrayList<>();
       for (int i = 0; i < subFields.length; i++) {
         String subFieldName = subFields[i];
         Field field = nestedMessage.fieldMap().get(subFieldName);

sdk-platform-java/gapic-generator-java/src/test/java/com/google/api/generator/gapic/composer/rest/goldens/ComplianceClientTest.golden

@@ -66,6 +66,7 @@ public class ComplianceClientTest {
             .setName("name3373707")
             .setInfo(ComplianceData.newBuilder().build())
             .setServerVerify(true)
+            .setCustomKebabName("customKebabName-2062111197")
             .build();
 
     RepeatResponse actualResponse = client.repeatDataBody(request);
@@ -99,6 +100,7 @@ public class ComplianceClientTest {
               .setName("name3373707")
               .setInfo(ComplianceData.newBuilder().build())
               .setServerVerify(true)
+              .setCustomKebabName("customKebabName-2062111197")
               .build();
       client.repeatDataBody(request);
       Assert.fail("No exception raised");

sdk-platform-java/gapic-generator-java/src/test/java/com/google/api/generator/gapic/protoparser/HttpRuleParserTest.java

@@ -80,21 +80,7 @@ void parseHttpAnnotation_multipleBindings() {
         .containsExactly("answer", "foo", "name", "test_to_verify.name", "type");
   }
 
-  @Test
-  void parseHttpAnnotation_missingFieldFromMessage() {
-    FileDescriptor testingFileDescriptor = TestingOuterClass.getDescriptor();
-    ServiceDescriptor testingService = testingFileDescriptor.getServices().get(0);
-    assertEquals("Testing", testingService.getName());
-
-    Map<String, Message> messages = Parser.parseMessages(testingFileDescriptor);
 
-    // VerityTest method.
-    MethodDescriptor rpcMethod =
-        testingService.getMethods().get(testingService.getMethods().size() - 1);
-    Message inputMessage = messages.get("com.google.showcase.v1beta1.CreateSessionRequest");
-    assertThrows(
-        IllegalStateException.class, () -> HttpRuleParser.parse(rpcMethod, inputMessage, messages));
-  }
 
   @Test
   void parseHttpAnnotation_shouldPutAllFieldsIntoQueryParamsIfPathParamAndBodyAreNotConfigured() {

sdk-platform-java/gapic-generator-java/src/test/java/com/google/api/generator/gapic/protoparser/ParserTest.java

@@ -102,10 +102,15 @@ void parseMessages_basic() {
 
     String echoResponseName = "EchoResponse";
     Field echoResponseContentField =
-        Field.builder().setName("content").setType(TypeNode.STRING).build();
+        Field.builder()
+            .setName("content")
+            .setJsonName("content")
+            .setType(TypeNode.STRING)
+            .build();
     Field echoResponseSeverityField =
         Field.builder()
             .setName("severity")
+            .setJsonName("severity")
             .setType(
                 TypeNode.withReference(
                     VaporReference.builder().setName("Severity").setPakkage(ECHO_PACKAGE).build()))