address PR feedback

googleapis · Jan 21, 2022 · 3f692cb · 3f692cb
1 parent d3dca27
commit 3f692cb
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 13 deletions.
diff --git a/...om/google/api/generator/gapic/composer/common/AbstractServiceClientTestClassComposer.java b/...om/google/api/generator/gapic/composer/common/AbstractServiceClientTestClassComposer.java
@@ -470,16 +470,16 @@ private MethodDefinition createRpcTestMethod(
               .setValueExpr(valExpr)
               .build());
     } else {
+      Map<String, String> valuePatterns = Collections.emptyMap();
+      if (getTransportContext().useValuePatterns() && method.hasHttpBindings()) {
+        valuePatterns = method.httpBindings().getPathParametersValuePatterns();
+      }
       for (MethodArgument methodArg : methodSignature) {
         String methodArgName = JavaStyle.toLowerCamelCase(methodArg.name());
         VariableExpr varExpr =
             VariableExpr.withVariable(
                 Variable.builder().setType(methodArg.type()).setName(methodArgName).build());
         argExprs.add(varExpr);
-        Map<String, String> valuePatterns = Collections.emptyMap();
-        if (getTransportContext().useValuePatterns() && method.hasHttpBindings()) {
-          valuePatterns = method.httpBindings().getPathParametersValuePatterns();
-        }
         Expr valExpr =
             DefaultValueComposer.createMethodArgValue(
                 methodArg, resourceNames, messageTypes, valuePatterns);
@@ -777,17 +777,16 @@ protected List<Statement> createRpcExceptionTestStatements(
               .setValueExpr(valExpr)
               .build());
     } else {
+      Map<String, String> valuePatterns = Collections.emptyMap();
+      if (getTransportContext().useValuePatterns() && method.hasHttpBindings()) {
+        valuePatterns = method.httpBindings().getPathParametersValuePatterns();
+      }
       for (MethodArgument methodArg : methodSignature) {
         String methodArgName = JavaStyle.toLowerCamelCase(methodArg.name());
         VariableExpr varExpr =
             VariableExpr.withVariable(
                 Variable.builder().setType(methodArg.type()).setName(methodArgName).build());
         argVarExprs.add(varExpr);
-        Map<String, String> valuePatterns = Collections.emptyMap();
-        if (getTransportContext().useValuePatterns() && method.hasHttpBindings()) {
-          valuePatterns = method.httpBindings().getPathParametersValuePatterns();
-        }
-
         Expr valExpr =
             DefaultValueComposer.createMethodArgValue(
                 methodArg, resourceNames, messageTypes, valuePatterns);

diff --git a/src/main/java/com/google/api/generator/gapic/composer/defaultvalue/DefaultValueComposer.java b/src/main/java/com/google/api/generator/gapic/composer/defaultvalue/DefaultValueComposer.java
@@ -600,16 +600,14 @@ private static String constructValueMatchingPattern(String fieldName, String pat
 
     final String suffix = "-" + (Math.abs((fieldName + pattern).hashCode()) % 10000);
 
-    String value = pattern;
-    value = value.replace("**", "*");
+    String value = pattern.replace("**", "*");
 
     String prevTempl = null;
     while (!value.equals(prevTempl)) {
       prevTempl = value;
       value = REPLACER_PATTERN.matcher(value).replaceFirst("$1$2$1" + suffix);
     }
 
-    value = value.replace("*", fieldName + suffix);
-    return value;
+    return value.replace("*", fieldName + suffix);
   }
 }