feat: Parser to consume the api-versioning value from proto

gapic-generator-java/src/main/java/com/google/api/generator/gapic/composer/Composer.java

@@ -207,7 +207,7 @@ static List<GapicClass> prepareExecutableSamples(List<GapicClass> clazzes) {
                   sample ->
                       samples.add(
                           addRegionTagAndHeaderToSample(
-                              sample, gapicClass.apiShortName(), gapicClass.apiVersion())));
+                              sample, gapicClass.apiShortName(), gapicClass.apiMajorVersion())));
           clazzesWithSamples.add(gapicClass.withSamples(samples));
         });
     return clazzesWithSamples;

gapic-generator-java/src/main/java/com/google/api/generator/gapic/composer/common/AbstractServiceClientClassComposer.java

@@ -173,7 +173,7 @@ public GapicClass generate(GapicContext context, Service service) {
     updateGapicMetadata(context, service, className, grpcRpcsToJavaMethodNames);
     return GapicClass.create(kind, classDef, SampleComposerUtil.handleDuplicateSamples(samples))
         .withApiShortName(service.apiShortName())
-        .withApiVersion(service.apiVersion());
+        .withApiMajorVersion(service.apiMajorVersion());
   }
 
   private static List<AnnotationNode> createClassAnnotations(Service service, TypeStore typeStore) {

gapic-generator-java/src/main/java/com/google/api/generator/gapic/composer/common/AbstractServiceSettingsClassComposer.java

@@ -128,7 +128,7 @@ public GapicClass generate(GapicContext context, Service service) {
             .build();
     return GapicClass.create(kind, classDef, SampleComposerUtil.handleDuplicateSamples(samples))
         .withApiShortName(service.apiShortName())
-        .withApiVersion(service.apiVersion());
+        .withApiMajorVersion(service.apiMajorVersion());
   }
 
   private static List<CommentStatement> createClassHeaderComments(

gapic-generator-java/src/main/java/com/google/api/generator/gapic/composer/common/AbstractServiceStubSettingsClassComposer.java

@@ -204,7 +204,7 @@ public GapicClass generate(GapicContext context, Service service) {
     return GapicClass.create(
             GapicClass.Kind.STUB, classDef, SampleComposerUtil.handleDuplicateSamples(samples))
         .withApiShortName(service.apiShortName())
-        .withApiVersion(service.apiVersion());
+        .withApiMajorVersion(service.apiMajorVersion());
   }
 
   protected MethodDefinition createDefaultCredentialsProviderBuilderMethod() {

gapic-generator-java/src/main/java/com/google/api/generator/gapic/model/GapicClass.java

@@ -43,7 +43,7 @@ public enum Kind {
   public abstract String apiShortName();
 
   // Only used for generating the region tag for samples; therefore only used in select Composers.
-  public abstract String apiVersion();
+  public abstract String apiMajorVersion();
 
   /**
    * Create a GapicClass with minimal information. This is intended to be used for GapicClasses that
@@ -76,7 +76,7 @@ static Builder builder() {
     return new AutoValue_GapicClass.Builder()
         .setSamples(Collections.emptyList())
         .setApiShortName("")
-        .setApiVersion("");
+        .setApiMajorVersion("");
   }
 
   abstract Builder toBuilder();
@@ -89,8 +89,8 @@ public final GapicClass withApiShortName(String apiShortName) {
     return toBuilder().setApiShortName(apiShortName).build();
   }
 
-  public final GapicClass withApiVersion(String apiVersion) {
-    return toBuilder().setApiVersion(apiVersion).build();
+  public final GapicClass withApiMajorVersion(String apiMajorVersion) {
+    return toBuilder().setApiMajorVersion(apiMajorVersion).build();
   }
 
   @AutoValue.Builder
@@ -103,7 +103,7 @@ abstract static class Builder {
 
     abstract Builder setApiShortName(String apiShortName);
 
-    abstract Builder setApiVersion(String apiVersion);
+    abstract Builder setApiMajorVersion(String apiMajorVersion);
 
     abstract GapicClass build();
   }

gapic-generator-java/src/main/java/com/google/api/generator/gapic/model/Service.java

@@ -27,6 +27,9 @@
 public abstract class Service {
   public abstract String name();
 
+  @Nullable
+  public abstract String apiVersion();
+
   public abstract String defaultHost();
 
   public abstract ImmutableList<String> oauthScopes();
@@ -52,6 +55,10 @@ public boolean hasDescription() {
     return !Strings.isNullOrEmpty(description());
   }
 
+  public boolean hasApiVersion() {
+    return !Strings.isNullOrEmpty(apiVersion());
+  }
+
   public String hostServiceName() {
     // Host Service Name is guaranteed to exist and be non-null and non-empty
     // Parser will fail if the default host is not supplied
@@ -65,9 +72,9 @@ public String apiShortName() {
     return "";
   }
 
-  public String apiVersion() {
+  public String apiMajorVersion() {
     if (!Strings.isNullOrEmpty(protoPakkage())) {
-      return parseApiVersion(protoPakkage());
+      return parseApiMajorVersion(protoPakkage());
     }
     return "";
   }
@@ -158,6 +165,8 @@ public abstract static class Builder {
 
     public abstract Builder setOverriddenName(String overriddenName);
 
+    public abstract Builder setApiVersion(String apiVersion);
+
     public abstract Builder setDefaultHost(String defaultHost);
 
     public abstract Builder setOauthScopes(List<String> oauthScopes);
@@ -177,17 +186,17 @@ public abstract static class Builder {
     public abstract Service build();
   }
 
-  private static String parseApiVersion(String protoPackage) {
-    //  parse protoPackage for apiVersion
+  private static String parseApiMajorVersion(String protoPackage) {
+    //  parse protoPackage for apiMajorVersion
     String[] pakkage = protoPackage.split("\\.");
-    String apiVersion;
+    String apiMajorVersion;
     //  e.g. v1, v2, v1beta1
     if (pakkage[pakkage.length - 1].matches("v[0-9].*")) {
-      apiVersion = pakkage[pakkage.length - 1];
+      apiMajorVersion = pakkage[pakkage.length - 1];
     } else {
-      apiVersion = "";
+      apiMajorVersion = "";
     }
-    return apiVersion;
+    return apiMajorVersion;
   }
 
   // Parse the service name from the default host configured in the protos

gapic-generator-java/src/main/java/com/google/api/generator/gapic/protoparser/Parser.java

@@ -483,26 +483,33 @@ public static List<Service> parseService(
                 overriddenServiceName =
                     languageSettings.getJavaServiceName(fileDescriptor.getPackage(), s.getName());
               }
-              return serviceBuilder
-                  .setName(serviceName)
-                  .setOverriddenName(overriddenServiceName)
-                  .setDefaultHost(defaultHost)
-                  .setOauthScopes(oauthScopes)
-                  .setPakkage(pakkage)
-                  .setOriginalJavaPackage(originalJavaPackage)
-                  .setProtoPakkage(fileDescriptor.getPackage())
-                  .setIsDeprecated(isDeprecated)
-                  .setMethods(
-                      parseMethods(
-                          s,
-                          pakkage,
-                          messageTypes,
-                          resourceNames,
-                          serviceConfigOpt,
-                          serviceYamlProtoOpt,
-                          outputArgResourceNames,
-                          transport))
-                  .build();
+              Service.Builder service =
+                  serviceBuilder
+                      .setName(serviceName)
+                      .setOverriddenName(overriddenServiceName)
+                      .setDefaultHost(defaultHost)
+                      .setOauthScopes(oauthScopes)
+                      .setPakkage(pakkage)
+                      .setOriginalJavaPackage(originalJavaPackage)
+                      .setProtoPakkage(fileDescriptor.getPackage())
+                      .setIsDeprecated(isDeprecated)
+                      .setMethods(
+                          parseMethods(
+                              s,
+                              pakkage,
+                              messageTypes,
+                              resourceNames,
+                              serviceConfigOpt,
+                              serviceYamlProtoOpt,
+                              outputArgResourceNames,
+                              transport));
+
+              String apiVersion = null;
+              if (serviceOptions.hasExtension(ClientProto.apiVersion)) {
+                apiVersion = serviceOptions.getExtension(ClientProto.apiVersion);
+                service.setApiVersion(apiVersion);
+              }
+              return service.build();
             })
         .collect(Collectors.toList());
   }

gapic-generator-java/src/test/java/com/google/api/generator/gapic/composer/ComposerTest.java

@@ -45,7 +45,7 @@ public class ComposerTest {
           GrpcServiceCallableFactoryClassComposer.instance()
               .generate(context, echoProtoService)
               .withApiShortName(echoProtoService.apiShortName())
-              .withApiVersion(echoProtoService.apiVersion()));
+              .withApiMajorVersion(echoProtoService.apiMajorVersion()));
   private final Sample sample =
       Sample.builder()
           .setRegionTag(
@@ -160,7 +160,7 @@ private List<GapicClass> getTestClassListFromService(Service testService) {
             .generate(context, testService)
             .withSamples(ListofSamples)
             .withApiShortName(testService.apiShortName())
-            .withApiVersion(testService.apiVersion());
+            .withApiMajorVersion(testService.apiMajorVersion());
     List<GapicClass> testClassList = Arrays.asList(new GapicClass[] {testClass});
     return testClassList;
   }

gapic-generator-java/src/test/java/com/google/api/generator/gapic/composer/grpc/ServiceClientClassComposerTest.java

@@ -83,6 +83,6 @@ public void generateServiceClientClasses() {
     Assert.assertGoldenSamples(
         this.getClass(), name, clazz.classDefinition().packageString(), clazz.samples());
     Assert.assertCodeEquals(clazz.apiShortName(), apiShortNameExpected);
-    Assert.assertCodeEquals(clazz.apiVersion(), apiVersionExpected);
+    Assert.assertCodeEquals(clazz.apiMajorVersion(), apiVersionExpected);
   }
 }

gapic-generator-java/src/test/java/com/google/api/generator/gapic/composer/grpc/ServiceSettingsClassComposerTest.java

@@ -69,6 +69,6 @@ public void generateServiceSettingsClasses() {
         clazz.classDefinition().packageString(),
         clazz.samples());
     Assert.assertCodeEquals(clazz.apiShortName(), apiShortNameExpected);
-    Assert.assertCodeEquals(clazz.apiVersion(), apiVersionExpected);
+    Assert.assertCodeEquals(clazz.apiMajorVersion(), apiVersionExpected);
   }
 }

gapic-generator-java/src/test/java/com/google/api/generator/gapic/composer/grpc/ServiceStubClassComposerTest.java

@@ -55,6 +55,6 @@ public void generateServiceStubClasses() {
     Assert.assertGoldenClass(this.getClass(), clazz, name + ".golden");
     Assert.assertEmptySamples(clazz.samples());
     Assert.assertCodeEquals(clazz.apiShortName(), apiShortNameExpected);
-    Assert.assertCodeEquals(clazz.apiVersion(), apiVersionExpected);
+    Assert.assertCodeEquals(clazz.apiMajorVersion(), apiVersionExpected);
   }
 }

gapic-generator-java/src/test/java/com/google/api/generator/gapic/composer/grpc/ServiceStubSettingsClassComposerTest.java

@@ -81,6 +81,6 @@ public void generateServiceStubSettingsClasses() {
         clazz.classDefinition().packageString(),
         clazz.samples());
     Assert.assertCodeEquals(clazz.apiShortName(), apiShortNameExpected);
-    Assert.assertCodeEquals(clazz.apiVersion(), apiVersionExpected);
+    Assert.assertCodeEquals(clazz.apiMajorVersion(), apiVersionExpected);
   }
 }

gapic-generator-java/src/test/java/com/google/api/generator/gapic/model/ServiceTest.java

@@ -16,24 +16,20 @@
 
 import static com.google.common.truth.Truth.assertThat;
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
 
 import com.google.api.generator.engine.ast.TypeNode;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
 import java.util.Arrays;
+import org.junit.Before;
 import org.junit.Test;
 
 public class ServiceTest {
   private static final String SHOWCASE_PACKAGE_NAME = "com.google.showcase.v1beta1";
-  private static final Service.Builder testServiceBuilder =
-      Service.builder()
-          .setName("Echo")
-          .setDefaultHost("localhost:7469")
-          .setOauthScopes(Arrays.asList("https://www.googleapis.com/auth/cloud-platform"))
-          .setPakkage(SHOWCASE_PACKAGE_NAME)
-          .setProtoPakkage(SHOWCASE_PACKAGE_NAME)
-          .setOriginalJavaPackage(SHOWCASE_PACKAGE_NAME)
-          .setOverriddenName("Echo");
+  private Service.Builder testServiceBuilder;
 
   private static final Method.Builder testMethodBuilder =
       Method.builder()
@@ -50,6 +46,19 @@ public class ServiceTest {
           .setIsAsteriskBody(false)
           .setHttpVerb(HttpBindings.HttpVerb.GET);
 
+  @Before
+  public void init() {
+    testServiceBuilder =
+        Service.builder()
+            .setName("Echo")
+            .setDefaultHost("localhost:7469")
+            .setOauthScopes(Arrays.asList("https://www.googleapis.com/auth/cloud-platform"))
+            .setPakkage(SHOWCASE_PACKAGE_NAME)
+            .setProtoPakkage(SHOWCASE_PACKAGE_NAME)
+            .setOriginalJavaPackage(SHOWCASE_PACKAGE_NAME)
+            .setOverriddenName("Echo");
+  }
+
   @Test
   public void apiShortName_shouldReturnApiShortNameIfHostContainsRegionalEndpoint() {
     String defaultHost = "us-east1-pubsub.googleapis.com";
@@ -82,14 +91,29 @@ public void apiShortName_shouldReturnHostIfNoPeriods() {
   public void apiVersion_shouldReturnVersionIfMatch() {
     String protoPackage = "com.google.showcase.v1";
     Service testService = testServiceBuilder.setProtoPakkage(protoPackage).build();
-    assertEquals("v1", testService.apiVersion());
+    assertEquals("v1", testService.apiMajorVersion());
+  }
+
+  @Test
+  public void apiVersion_shouldReturnApiVersion() {
+    String apiVersion = "v1_20230601";
+    Service testService = testServiceBuilder.setApiVersion(apiVersion).build();
+    assertTrue(testService.hasApiVersion());
+    assertEquals(apiVersion, testService.apiVersion());
+  }
+
+  @Test
+  public void apiVersion_shouldReturnNullApiVersion() {
+    Service testService = testServiceBuilder.build();
+    assertNull(testService.apiVersion());
+    assertFalse(testService.hasApiVersion());
   }
 
   @Test
   public void apiVersion_shouldReturnEmptyIfNoMatch() {
     String protoPackage = "com.google.showcase";
     Service testService = testServiceBuilder.setProtoPakkage(protoPackage).build();
-    assertEquals("", testService.apiVersion());
+    assertEquals("", testService.apiMajorVersion());
   }
 
   @Test

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

@@ -17,6 +17,7 @@
 import static com.google.common.truth.Truth.assertThat;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
 
@@ -607,6 +608,29 @@ public void parseNestedProtoTypeName() {
             "google.ads.googleads.v3.resources.MutateJob.MutateJobMetadata"));
   }
 
+  @Test
+  public void testServiceApiVersionParsed() {
+    Map<String, Message> messageTypes = Parser.parseMessages(echoFileDescriptor);
+    Map<String, ResourceName> resourceNames = Parser.parseResourceNames(echoFileDescriptor);
+    List<com.google.api.generator.gapic.model.Service> services =
+        Parser.parseService(
+            echoFileDescriptor, messageTypes, resourceNames, Optional.empty(), new HashSet<>());
+    com.google.api.generator.gapic.model.Service parsedEchoService = services.get(0);
+    assertEquals("v1_20230601", parsedEchoService.apiVersion());
+  }
+
+  @Test
+  public void testServiceWithoutApiVersionParsed() {
+    FileDescriptor bookshopFileDescriptor = BookshopProto.getDescriptor();
+    Map<String, Message> messageTypes = Parser.parseMessages(bookshopFileDescriptor);
+    Map<String, ResourceName> resourceNames = Parser.parseResourceNames(bookshopFileDescriptor);
+    List<com.google.api.generator.gapic.model.Service> services =
+        Parser.parseService(
+            bookshopFileDescriptor, messageTypes, resourceNames, Optional.empty(), new HashSet<>());
+    com.google.api.generator.gapic.model.Service parsedBookshopService = services.get(0);
+    assertNull(parsedBookshopService.apiVersion());
+  }
+
   private void assertMethodArgumentEquals(
       String name, TypeNode type, List<TypeNode> nestedFields, MethodArgument argument) {
     assertEquals(name, argument.name());

gapic-generator-java/src/test/proto/echo.proto

@@ -44,6 +44,7 @@ service Echo {
   // This service is meant to only run locally on the port 7469 (keypad digits
   // for "show").
   option (google.api.default_host) = "localhost:7469";
+  option (google.api.api_version) = "v1_20230601";
   option (google.api.oauth_scopes) =
       "https://www.googleapis.com/auth/cloud-platform";