[ggj][codegen] fix: ServiceClientTest settings init and tearDown method (#335)

* feat: add protobuf comment parser util

* fix: add basic proto build rules

* feat: add header comments to ServiceClient

* fix: build protoc at test time

* fix!: wrap protobuf location and process comments

* feat: add comment parsing to methods and fields

* fix: test

* feat: add protobuf comments to ServiceClient

* fix: solidify codegen method order with TypeNode/MethodArg and Comparable

* fix: clean up tests

* fix: ServiceClient member variables and method calls

* fix: ServiceStubSettings builder type

* fix: ServiceSettings Builder construction

* fix: ServiceStub callable types

* feat: java_gapic_library rule impl

* fix: remove debugging comments

* feat: add gradle assembly Bazel rules

* feat: add java_gapic_test Bazel rule

* fix: use Java packages for resname codegen

* fix: build resnames separately and extract into proto/ dir

* fix: remove debug printf

* feat: add ServiceClient.MethodPagedResponse inner class

* feat: add ServiceClient.MethodPage inner class

* feat: add ServiceClient.MethodFixedSizeCollection innser class

* fix: clean up resname codegen, lower_snake varnames

* fix: init remaining resname tokenVars to null

* fix: ServiceStubSettings paged descriptor rep. field getter name

* fix: handle map/list args in ServiceClient

* fix: ServiceSTubSettings batching request type

* fix: GrpcServiceStub createPagedCallable

* fix: remove @OverRide from MockServiceImpl

* fix: sServiceClientTest ettings var init

* fix: tearDown method in test

Author: miraleung

src/main/java/com/google/api/generator/gapic/composer/ServiceClientTestClassComposer.java

@@ -326,7 +326,7 @@ private static MethodDefinition createSetUpMethod(
 
     Expr initLocalSettingsExpr =
         AssignmentExpr.builder()
-            .setVariableExpr(localSettingsVarExpr)
+            .setVariableExpr(localSettingsVarExpr.toBuilder().setIsDecl(true).build())
             .setValueExpr(settingsBuilderExpr)
             .build();
 
@@ -365,7 +365,6 @@ private static MethodDefinition createTearDownMethod(
     return MethodDefinition.builder()
         .setAnnotations(Arrays.asList(AnnotationNode.withType(staticTypes.get("After"))))
         .setScope(ScopeNode.PUBLIC)
-        .setIsStatic(true)
         .setReturnType(TypeNode.VOID)
         .setName("tearDown")
         .setThrowsExceptions(

src/test/java/com/google/api/generator/gapic/composer/ServiceClientTestClassComposerTest.java

@@ -102,7 +102,7 @@ public void generateServiceClasses() {
           + "  public void setUp() throws IOException {\n"
           + "    mockServiceHelper.reset();\n"
           + "    channelProvider = mockServiceHelper.createChannelProvider();\n"
-          + "    settings =\n"
+          + "    EchoSettings settings =\n"
           + "        EchoSettings.newBuilder()\n"
           + "            .setTransportChannelProvider(channelProvider)\n"
           + "            .setCredentialsProvider(NoCredentialsProvider.create())\n"
@@ -111,7 +111,7 @@ public void generateServiceClasses() {
           + "  }\n"
           + "\n"
           + "  @After\n"
-          + "  public static void tearDown() throws Exception {\n"
+          + "  public void tearDown() throws Exception {\n"
           + "    client.close();\n"
           + "  }\n"
           + "}\n";