Skip to content

Commit 9ac0ca8

Browse files
authored
fix: fix REST transport client creation generated javadoc sample (#1077)
1 parent da4b130 commit 9ac0ca8

File tree

4 files changed

+6
-29
lines changed

4 files changed

+6
-29
lines changed

src/main/java/com/google/api/generator/gapic/composer/grpcrest/ServiceClientClassComposer.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ protected List<CommentStatement> createClassHeaderComments(
5959
ServiceClientHeaderSampleComposer.composeSetEndpointSample(clientType, settingsType);
6060
Sample transportSampleCode =
6161
ServiceClientHeaderSampleComposer.composeTransportSample(
62-
clientType, settingsType, "defaultHttpJsonTransportProviderBuilder");
62+
clientType, settingsType, "newHttpJsonBuilder");
6363
samples.addAll(
6464
Arrays.asList(
6565
classMethodSampleCode, credentialsSampleCode, endpointSampleCode, transportSampleCode));

src/main/java/com/google/api/generator/gapic/composer/samplecode/ServiceClientHeaderSampleComposer.java

+3-18
Original file line numberDiff line numberDiff line change
@@ -285,7 +285,7 @@ public static Sample composeSetEndpointSample(TypeNode clientType, TypeNode sett
285285
}
286286

287287
public static Sample composeTransportSample(
288-
TypeNode clientType, TypeNode settingsType, String transportProviderMethod) {
288+
TypeNode clientType, TypeNode settingsType, String transportBuilderMethod) {
289289
String settingsName = JavaStyle.toLowerCamelCase(settingsType.reference().name());
290290
String clientName = JavaStyle.toLowerCamelCase(clientType.reference().name());
291291
VariableExpr settingsVarExpr =
@@ -294,26 +294,11 @@ public static Sample composeTransportSample(
294294
MethodInvocationExpr newBuilderMethodExpr =
295295
MethodInvocationExpr.builder()
296296
.setStaticReferenceType(settingsType)
297-
.setMethodName("newBuilder")
298-
.build();
299-
MethodInvocationExpr transportChannelProviderArg =
300-
MethodInvocationExpr.builder()
301-
.setExprReferenceExpr(
302-
MethodInvocationExpr.builder()
303-
.setStaticReferenceType(settingsType)
304-
.setMethodName(transportProviderMethod)
305-
.build())
306-
.setMethodName("build")
307-
.build();
308-
MethodInvocationExpr credentialsMethodExpr =
309-
MethodInvocationExpr.builder()
310-
.setExprReferenceExpr(newBuilderMethodExpr)
311-
.setArguments(transportChannelProviderArg)
312-
.setMethodName("setTransportChannelProvider")
297+
.setMethodName(transportBuilderMethod)
313298
.build();
314299
MethodInvocationExpr buildMethodExpr =
315300
MethodInvocationExpr.builder()
316-
.setExprReferenceExpr(credentialsMethodExpr)
301+
.setExprReferenceExpr(newBuilderMethodExpr)
317302
.setReturnType(settingsType)
318303
.setMethodName("build")
319304
.build();

src/test/java/com/google/api/generator/gapic/composer/grpcrest/goldens/EchoClient.golden

+1-5
Original file line numberDiff line numberDiff line change
@@ -105,11 +105,7 @@ import javax.annotation.Generated;
105105
* // - It may require correct/in-range values for request initialization.
106106
* // - It may require specifying regional endpoints when creating the service client as shown in
107107
* // https://cloud.google.com/java/docs/setup#configure_endpoints_for_the_client_library
108-
* EchoSettings echoSettings =
109-
* EchoSettings.newBuilder()
110-
* .setTransportChannelProvider(
111-
* EchoSettings.defaultHttpJsonTransportProviderBuilder().build())
112-
* .build();
108+
* EchoSettings echoSettings = EchoSettings.newHttpJsonBuilder().build();
113109
* EchoClient echoClient = EchoClient.create(echoSettings);
114110
* }</pre>
115111
*

src/test/java/com/google/api/generator/gapic/composer/grpcrest/goldens/EchoEmpty.golden

+1-5
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,7 @@ import javax.annotation.Generated;
8585
* // - It may require correct/in-range values for request initialization.
8686
* // - It may require specifying regional endpoints when creating the service client as shown in
8787
* // https://cloud.google.com/java/docs/setup#configure_endpoints_for_the_client_library
88-
* EchoEmpySettings echoEmpySettings =
89-
* EchoEmpySettings.newBuilder()
90-
* .setTransportChannelProvider(
91-
* EchoEmpySettings.defaultHttpJsonTransportProviderBuilder().build())
92-
* .build();
88+
* EchoEmpySettings echoEmpySettings = EchoEmpySettings.newHttpJsonBuilder().build();
9389
* EchoEmpyClient echoEmpyClient = EchoEmpyClient.create(echoEmpySettings);
9490
* }</pre>
9591
*

0 commit comments

Comments
 (0)