diff --git a/google-cloud-core-grpc/src/main/java/com/google/cloud/grpc/BaseGrpcServiceException.java b/google-cloud-core-grpc/src/main/java/com/google/cloud/grpc/BaseGrpcServiceException.java index 4f50fe330dc6..50d917619634 100644 --- a/google-cloud-core-grpc/src/main/java/com/google/cloud/grpc/BaseGrpcServiceException.java +++ b/google-cloud-core-grpc/src/main/java/com/google/cloud/grpc/BaseGrpcServiceException.java @@ -19,7 +19,8 @@ import com.google.api.client.http.HttpResponseException; import com.google.api.core.BetaApi; import com.google.api.core.InternalApi; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.grpc.GrpcStatusCode; +import com.google.api.gax.rpc.ApiException; import com.google.cloud.BaseServiceException; import com.google.common.base.MoreObjects; import java.io.IOException; @@ -66,13 +67,13 @@ private static ExceptionData makeExceptionData(IOException exception, boolean id } @BetaApi - public BaseGrpcServiceException(GrpcApiException apiException) { + public BaseGrpcServiceException(ApiException apiException) { super(ExceptionData.newBuilder() .setMessage(apiException.getMessage()) .setCause(apiException) .setRetryable(apiException.isRetryable()) - .setCode(apiException.getStatusCode().getCode().value()) - .setReason(apiException.getStatusCode().getCode().name()) + .setCode(((GrpcStatusCode) apiException.getStatusCode()).getCode().value()) + .setReason(((GrpcStatusCode) apiException.getStatusCode()).getCode().name()) .setLocation(null) .setDebugInfo(null) .build()); diff --git a/google-cloud-core-grpc/src/test/java/com/google/cloud/grpc/BaseGrpcServiceExceptionTest.java b/google-cloud-core-grpc/src/test/java/com/google/cloud/grpc/BaseGrpcServiceExceptionTest.java index ec9b051c0071..e35a3a620878 100644 --- a/google-cloud-core-grpc/src/test/java/com/google/cloud/grpc/BaseGrpcServiceExceptionTest.java +++ b/google-cloud-core-grpc/src/test/java/com/google/cloud/grpc/BaseGrpcServiceExceptionTest.java @@ -25,7 +25,9 @@ import static org.junit.Assert.assertNull; import static org.junit.Assert.assertTrue; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.grpc.GrpcStatusCode; +import com.google.api.gax.rpc.ApiException; +import com.google.api.gax.rpc.InternalException; import com.google.cloud.BaseServiceException; import com.google.cloud.RetryHelper; import io.grpc.Status.Code; @@ -73,8 +75,8 @@ public void testBaseServiceException() { assertNull(serviceException.getDebugInfo()); Exception cause = new IllegalArgumentException("bad arg"); - GrpcApiException apiException = - new GrpcApiException(MESSAGE, cause, Code.INTERNAL, NOT_RETRYABLE); + InternalException apiException = + new InternalException(MESSAGE, cause, GrpcStatusCode.of(Code.INTERNAL), NOT_RETRYABLE); serviceException = new BaseGrpcServiceException(apiException); assertFalse(serviceException.isRetryable()); assertEquals(MESSAGE, serviceException.getMessage()); diff --git a/google-cloud-dlp/src/test/java/com/google/cloud/dlp/v2beta1/DlpServiceClientTest.java b/google-cloud-dlp/src/test/java/com/google/cloud/dlp/v2beta1/DlpServiceClientTest.java index b7a03c0e474d..d753e83298be 100644 --- a/google-cloud-dlp/src/test/java/com/google/cloud/dlp/v2beta1/DlpServiceClientTest.java +++ b/google-cloud-dlp/src/test/java/com/google/cloud/dlp/v2beta1/DlpServiceClientTest.java @@ -16,10 +16,11 @@ package com.google.cloud.dlp.v2beta1; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.grpc.GrpcStatusCode; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.longrunning.Operation; import com.google.privacy.dlp.v2beta1.ContentItem; import com.google.privacy.dlp.v2beta1.CreateInspectOperationRequest; @@ -125,8 +126,8 @@ public void inspectContentExceptionTest() throws Exception { client.inspectContent(inspectConfig, items); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -166,8 +167,8 @@ public void redactContentExceptionTest() throws Exception { client.redactContent(inspectConfig, items, replaceConfigs); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -217,10 +218,11 @@ public void createInspectOperationExceptionTest() throws Exception { client.createInspectOperationAsync(inspectConfig, storageConfig, outputConfig).get(); Assert.fail("No exception raised"); } catch (ExecutionException e) { - Assert.assertEquals(GrpcApiException.class, e.getCause().getClass()); - GrpcApiException apiException = (GrpcApiException) e.getCause(); + Assert.assertEquals(InvalidArgumentException.class, e.getCause().getClass()); + InvalidArgumentException apiException = (InvalidArgumentException) e.getCause(); Assert.assertEquals( - Status.INVALID_ARGUMENT.getCode(), apiException.getStatusCode().getCode()); + Status.INVALID_ARGUMENT.getCode(), + ((GrpcStatusCode) apiException.getStatusCode()).getCode()); } } @@ -255,8 +257,8 @@ public void listInspectFindingsExceptionTest() throws Exception { client.listInspectFindings(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -292,8 +294,8 @@ public void listInfoTypesExceptionTest() throws Exception { client.listInfoTypes(category, languageCode); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -326,8 +328,8 @@ public void listRootCategoriesExceptionTest() throws Exception { client.listRootCategories(languageCode); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ErrorGroupServiceClientTest.java b/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ErrorGroupServiceClientTest.java index b5a507c22c94..f1ddd1159d31 100644 --- a/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ErrorGroupServiceClientTest.java +++ b/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ErrorGroupServiceClientTest.java @@ -16,10 +16,10 @@ package com.google.cloud.errorreporting.v1beta1; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.devtools.clouderrorreporting.v1beta1.ErrorGroup; import com.google.devtools.clouderrorreporting.v1beta1.GetGroupRequest; import com.google.devtools.clouderrorreporting.v1beta1.GroupName; @@ -114,8 +114,8 @@ public void getGroupExceptionTest() throws Exception { client.getGroup(groupName); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -151,8 +151,8 @@ public void updateGroupExceptionTest() throws Exception { client.updateGroup(group); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ErrorStatsServiceClientTest.java b/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ErrorStatsServiceClientTest.java index 10db186b1a5d..4f48c5146f1b 100644 --- a/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ErrorStatsServiceClientTest.java +++ b/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ErrorStatsServiceClientTest.java @@ -19,10 +19,10 @@ import static com.google.cloud.errorreporting.v1beta1.PagedResponseWrappers.ListGroupStatsPagedResponse; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.common.collect.Lists; import com.google.devtools.clouderrorreporting.v1beta1.DeleteEventsRequest; import com.google.devtools.clouderrorreporting.v1beta1.DeleteEventsResponse; @@ -134,8 +134,8 @@ public void listGroupStatsExceptionTest() throws Exception { client.listGroupStats(projectName, timeRange); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -181,8 +181,8 @@ public void listEventsExceptionTest() throws Exception { client.listEvents(projectName, groupId); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -215,8 +215,8 @@ public void deleteEventsExceptionTest() throws Exception { client.deleteEvents(projectName); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ReportErrorsServiceClientTest.java b/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ReportErrorsServiceClientTest.java index 98a14d189498..730fa6e37f8e 100644 --- a/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ReportErrorsServiceClientTest.java +++ b/google-cloud-errorreporting/src/test/java/com/google/cloud/errorreporting/v1beta1/ReportErrorsServiceClientTest.java @@ -16,10 +16,10 @@ package com.google.cloud.errorreporting.v1beta1; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.devtools.clouderrorreporting.v1beta1.ProjectName; import com.google.devtools.clouderrorreporting.v1beta1.ReportErrorEventRequest; import com.google.devtools.clouderrorreporting.v1beta1.ReportErrorEventResponse; @@ -114,8 +114,8 @@ public void reportErrorEventExceptionTest() throws Exception { client.reportErrorEvent(projectName, event); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-examples/src/test/java/com/google/cloud/examples/pubsub/snippets/ITSubscriptionAdminClientSnippets.java b/google-cloud-examples/src/test/java/com/google/cloud/examples/pubsub/snippets/ITSubscriptionAdminClientSnippets.java index 1f71b7ac6003..5beb95de1c89 100644 --- a/google-cloud-examples/src/test/java/com/google/cloud/examples/pubsub/snippets/ITSubscriptionAdminClientSnippets.java +++ b/google-cloud-examples/src/test/java/com/google/cloud/examples/pubsub/snippets/ITSubscriptionAdminClientSnippets.java @@ -20,7 +20,7 @@ import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.rpc.ApiException; import com.google.cloud.Identity; import com.google.cloud.Role; import com.google.cloud.pubsub.v1.PagedResponseWrappers.ListSubscriptionsPagedResponse; @@ -124,7 +124,7 @@ public void listSubscriptionsRetrievesAllAddedSubscriptions() throws Exception { assertTrue(subFound[0] && subFound[1]); } - @Test(expected = GrpcApiException.class) + @Test(expected = ApiException.class) public void deleteSubscriptionThrowsExceptionWhenRetrieved() throws Exception { String topicName = topics[0]; String subscriptionName = subscriptions[0]; diff --git a/google-cloud-examples/src/test/java/com/google/cloud/examples/pubsub/snippets/ITTopicAdminClientSnippets.java b/google-cloud-examples/src/test/java/com/google/cloud/examples/pubsub/snippets/ITTopicAdminClientSnippets.java index 3cde185e5560..2bc8683e5089 100644 --- a/google-cloud-examples/src/test/java/com/google/cloud/examples/pubsub/snippets/ITTopicAdminClientSnippets.java +++ b/google-cloud-examples/src/test/java/com/google/cloud/examples/pubsub/snippets/ITTopicAdminClientSnippets.java @@ -20,7 +20,7 @@ import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.rpc.ApiException; import com.google.cloud.Identity; import com.google.cloud.Role; import com.google.cloud.pubsub.v1.PagedResponseWrappers.ListTopicSubscriptionsPagedResponse; @@ -136,7 +136,7 @@ public void listTopicSubscriptionsRetrievesAddedSubscriptions() throws Exception assertTrue(subFound[0] && subFound[1]); } - @Test(expected = GrpcApiException.class) + @Test(expected = ApiException.class) public void deletedTopicIsNotRetrievableAndThrowsException() throws Exception { String topicName = topics[0]; Topic topicAdded = topicAdminClientSnippets.createTopic(topicName); diff --git a/google-cloud-language/src/test/java/com/google/cloud/language/v1/LanguageServiceClientTest.java b/google-cloud-language/src/test/java/com/google/cloud/language/v1/LanguageServiceClientTest.java index da4812868b4b..51695d725171 100644 --- a/google-cloud-language/src/test/java/com/google/cloud/language/v1/LanguageServiceClientTest.java +++ b/google-cloud-language/src/test/java/com/google/cloud/language/v1/LanguageServiceClientTest.java @@ -16,10 +16,10 @@ package com.google.cloud.language.v1; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.cloud.language.v1.AnnotateTextRequest.Features; import com.google.protobuf.GeneratedMessageV3; import io.grpc.Status; @@ -103,8 +103,8 @@ public void analyzeSentimentExceptionTest() throws Exception { client.analyzeSentiment(document); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -142,8 +142,8 @@ public void analyzeEntitiesExceptionTest() throws Exception { client.analyzeEntities(document, encodingType); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -181,8 +181,8 @@ public void analyzeSyntaxExceptionTest() throws Exception { client.analyzeSyntax(document, encodingType); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -223,8 +223,8 @@ public void annotateTextExceptionTest() throws Exception { client.annotateText(document, features, encodingType); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-language/src/test/java/com/google/cloud/language/v1beta2/LanguageServiceClientTest.java b/google-cloud-language/src/test/java/com/google/cloud/language/v1beta2/LanguageServiceClientTest.java index b2d64c6d150f..4ccec2289495 100644 --- a/google-cloud-language/src/test/java/com/google/cloud/language/v1beta2/LanguageServiceClientTest.java +++ b/google-cloud-language/src/test/java/com/google/cloud/language/v1beta2/LanguageServiceClientTest.java @@ -16,10 +16,10 @@ package com.google.cloud.language.v1beta2; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.cloud.language.v1beta2.AnnotateTextRequest.Features; import com.google.protobuf.GeneratedMessageV3; import io.grpc.Status; @@ -103,8 +103,8 @@ public void analyzeSentimentExceptionTest() throws Exception { client.analyzeSentiment(document); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -142,8 +142,8 @@ public void analyzeEntitiesExceptionTest() throws Exception { client.analyzeEntities(document, encodingType); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -183,8 +183,8 @@ public void analyzeEntitySentimentExceptionTest() throws Exception { client.analyzeEntitySentiment(document, encodingType); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -222,8 +222,8 @@ public void analyzeSyntaxExceptionTest() throws Exception { client.analyzeSyntax(document, encodingType); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -264,8 +264,8 @@ public void annotateTextExceptionTest() throws Exception { client.annotateText(document, features, encodingType); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-logging/src/main/java/com/google/cloud/logging/LoggingException.java b/google-cloud-logging/src/main/java/com/google/cloud/logging/LoggingException.java index 127bfc3a35eb..8f1a375b065c 100644 --- a/google-cloud-logging/src/main/java/com/google/cloud/logging/LoggingException.java +++ b/google-cloud-logging/src/main/java/com/google/cloud/logging/LoggingException.java @@ -17,7 +17,7 @@ package com.google.cloud.logging; import com.google.api.core.BetaApi; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.rpc.ApiException; import com.google.cloud.grpc.BaseGrpcServiceException; import java.io.IOException; @@ -33,7 +33,7 @@ public LoggingException(IOException ex, boolean idempotent) { } @BetaApi - public LoggingException(GrpcApiException apiException) { + public LoggingException(ApiException apiException) { super(apiException); } } diff --git a/google-cloud-logging/src/main/java/com/google/cloud/logging/spi/v2/GrpcLoggingRpc.java b/google-cloud-logging/src/main/java/com/google/cloud/logging/spi/v2/GrpcLoggingRpc.java index 78f22541335e..205f756acb7d 100644 --- a/google-cloud-logging/src/main/java/com/google/cloud/logging/spi/v2/GrpcLoggingRpc.java +++ b/google-cloud-logging/src/main/java/com/google/cloud/logging/spi/v2/GrpcLoggingRpc.java @@ -21,9 +21,10 @@ import com.google.api.core.ApiFutures; import com.google.api.gax.core.BackgroundResource; import com.google.api.gax.grpc.ChannelProvider; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.grpc.GrpcStatusCode; import com.google.api.gax.grpc.GrpcTransport; import com.google.api.gax.grpc.GrpcTransportProvider; +import com.google.api.gax.rpc.ApiException; import com.google.api.gax.rpc.ClientContext; import com.google.api.gax.rpc.Transport; import com.google.api.gax.rpc.UnaryCallSettings; @@ -151,11 +152,11 @@ private static ApiFuture translate( } return ApiFutures.catching( from, - GrpcApiException.class, - new ApiFunction() { + ApiException.class, + new ApiFunction() { @Override - public V apply(GrpcApiException exception) { - if (returnNullOnSet.contains(exception.getStatusCode().getCode())) { + public V apply(ApiException exception) { + if (returnNullOnSet.contains(((GrpcStatusCode) exception.getStatusCode()).getCode())) { return null; } throw new LoggingException(exception); diff --git a/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/ConfigClientTest.java b/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/ConfigClientTest.java index 798d3910bb05..dcd2e29fac0a 100644 --- a/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/ConfigClientTest.java +++ b/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/ConfigClientTest.java @@ -18,10 +18,10 @@ import static com.google.cloud.logging.v2.PagedResponseWrappers.ListSinksPagedResponse; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.api.resourcenames.ResourceName; import com.google.common.collect.Lists; import com.google.logging.v2.CreateSinkRequest; @@ -130,8 +130,8 @@ public void listSinksExceptionTest() throws Exception { client.listSinks(parent); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -176,8 +176,8 @@ public void getSinkExceptionTest() throws Exception { client.getSink(sinkName); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -225,8 +225,8 @@ public void createSinkExceptionTest() throws Exception { client.createSink(parent, sink); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -274,8 +274,8 @@ public void updateSinkExceptionTest() throws Exception { client.updateSink(sinkName, sink); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -307,8 +307,8 @@ public void deleteSinkExceptionTest() throws Exception { client.deleteSink(sinkName); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/LoggingClientTest.java b/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/LoggingClientTest.java index b5da1e4e7ecc..e75cf39f66ab 100644 --- a/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/LoggingClientTest.java +++ b/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/LoggingClientTest.java @@ -20,10 +20,10 @@ import com.google.api.MonitoredResource; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.common.collect.Lists; import com.google.logging.v2.DeleteLogRequest; import com.google.logging.v2.ListLogEntriesRequest; @@ -127,8 +127,8 @@ public void deleteLogExceptionTest() throws Exception { client.deleteLog(logName); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -171,8 +171,8 @@ public void writeLogEntriesExceptionTest() throws Exception { client.writeLogEntries(logName, resource, labels, entries); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -222,8 +222,8 @@ public void listLogEntriesExceptionTest() throws Exception { client.listLogEntries(resourceNames, filter, orderBy); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -266,8 +266,8 @@ public void listLogsExceptionTest() throws Exception { client.listLogs(parent); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/MetricsClientTest.java b/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/MetricsClientTest.java index aec978430537..4eaf993f47e3 100644 --- a/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/MetricsClientTest.java +++ b/google-cloud-logging/src/test/java/com/google/cloud/logging/v2/MetricsClientTest.java @@ -18,10 +18,10 @@ import static com.google.cloud.logging.v2.PagedResponseWrappers.ListLogMetricsPagedResponse; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.common.collect.Lists; import com.google.logging.v2.CreateLogMetricRequest; import com.google.logging.v2.DeleteLogMetricRequest; @@ -132,8 +132,8 @@ public void listLogMetricsExceptionTest() throws Exception { client.listLogMetrics(parent); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -170,8 +170,8 @@ public void getLogMetricExceptionTest() throws Exception { client.getLogMetric(metricName); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -211,8 +211,8 @@ public void createLogMetricExceptionTest() throws Exception { client.createLogMetric(parent, metric); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -252,8 +252,8 @@ public void updateLogMetricExceptionTest() throws Exception { client.updateLogMetric(metricName, metric); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -285,8 +285,8 @@ public void deleteLogMetricExceptionTest() throws Exception { client.deleteLogMetric(metricName); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-monitoring/src/test/java/com/google/cloud/monitoring/v3/GroupServiceClientTest.java b/google-cloud-monitoring/src/test/java/com/google/cloud/monitoring/v3/GroupServiceClientTest.java index 04f34d55e29d..5463a435fa62 100644 --- a/google-cloud-monitoring/src/test/java/com/google/cloud/monitoring/v3/GroupServiceClientTest.java +++ b/google-cloud-monitoring/src/test/java/com/google/cloud/monitoring/v3/GroupServiceClientTest.java @@ -19,10 +19,10 @@ import com.google.api.MonitoredResource; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.common.collect.Lists; import com.google.monitoring.v3.CreateGroupRequest; import com.google.monitoring.v3.DeleteGroupRequest; @@ -129,8 +129,8 @@ public void getGroupExceptionTest() throws Exception { client.getGroup(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -178,8 +178,8 @@ public void createGroupExceptionTest() throws Exception { client.createGroup(name, group); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -224,8 +224,8 @@ public void updateGroupExceptionTest() throws Exception { client.updateGroup(group); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -257,8 +257,8 @@ public void deleteGroupExceptionTest() throws Exception { client.deleteGroup(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -303,8 +303,8 @@ public void listGroupMembersExceptionTest() throws Exception { client.listGroupMembers(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-monitoring/src/test/java/com/google/cloud/monitoring/v3/MetricServiceClientTest.java b/google-cloud-monitoring/src/test/java/com/google/cloud/monitoring/v3/MetricServiceClientTest.java index ce5d6955ea64..73af5ba498a7 100644 --- a/google-cloud-monitoring/src/test/java/com/google/cloud/monitoring/v3/MetricServiceClientTest.java +++ b/google-cloud-monitoring/src/test/java/com/google/cloud/monitoring/v3/MetricServiceClientTest.java @@ -22,10 +22,10 @@ import com.google.api.MetricDescriptor; import com.google.api.MonitoredResourceDescriptor; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.common.collect.Lists; import com.google.monitoring.v3.CreateMetricDescriptorRequest; import com.google.monitoring.v3.CreateTimeSeriesRequest; @@ -144,8 +144,8 @@ public void listMonitoredResourceDescriptorsExceptionTest() throws Exception { client.listMonitoredResourceDescriptors(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -191,8 +191,8 @@ public void getMonitoredResourceDescriptorExceptionTest() throws Exception { client.getMonitoredResourceDescriptor(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -236,8 +236,8 @@ public void listMetricDescriptorsExceptionTest() throws Exception { client.listMetricDescriptors(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -282,8 +282,8 @@ public void getMetricDescriptorExceptionTest() throws Exception { client.getMetricDescriptor(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -332,8 +332,8 @@ public void createMetricDescriptorExceptionTest() throws Exception { client.createMetricDescriptor(name, metricDescriptor); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -366,8 +366,8 @@ public void deleteMetricDescriptorExceptionTest() throws Exception { client.deleteMetricDescriptor(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -420,8 +420,8 @@ public void listTimeSeriesExceptionTest() throws Exception { client.listTimeSeries(name, filter, interval, view); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -456,8 +456,8 @@ public void createTimeSeriesExceptionTest() throws Exception { client.createTimeSeries(name, timeSeries); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/SubscriptionAdminClientTest.java b/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/SubscriptionAdminClientTest.java index b6ed8e0c9c71..a82cac92ef3f 100644 --- a/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/SubscriptionAdminClientTest.java +++ b/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/SubscriptionAdminClientTest.java @@ -19,13 +19,13 @@ import static com.google.cloud.pubsub.v1.PagedResponseWrappers.ListSubscriptionsPagedResponse; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; import com.google.api.gax.grpc.testing.MockStreamObserver; import com.google.api.gax.rpc.ApiStreamObserver; import com.google.api.gax.rpc.BidiStreamingCallable; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.common.collect.Lists; import com.google.iam.v1.GetIamPolicyRequest; import com.google.iam.v1.Policy; @@ -166,8 +166,8 @@ public void createSubscriptionExceptionTest() throws Exception { client.createSubscription(name, topic, pushConfig, ackDeadlineSeconds); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -210,8 +210,8 @@ public void getSubscriptionExceptionTest() throws Exception { client.getSubscription(subscription); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -254,8 +254,8 @@ public void listSubscriptionsExceptionTest() throws Exception { client.listSubscriptions(project); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -287,8 +287,8 @@ public void deleteSubscriptionExceptionTest() throws Exception { client.deleteSubscription(subscription); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -326,8 +326,8 @@ public void modifyAckDeadlineExceptionTest() throws Exception { client.modifyAckDeadline(subscription, ackIds, ackDeadlineSeconds); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -362,8 +362,8 @@ public void acknowledgeExceptionTest() throws Exception { client.acknowledge(subscription, ackIds); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -402,8 +402,8 @@ public void pullExceptionTest() throws Exception { client.pull(subscription, returnImmediately, maxMessages); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -501,8 +501,8 @@ public void modifyPushConfigExceptionTest() throws Exception { client.modifyPushConfig(subscription, pushConfig); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -545,8 +545,8 @@ public void listSnapshotsExceptionTest() throws Exception { client.listSnapshots(project); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -585,8 +585,8 @@ public void createSnapshotExceptionTest() throws Exception { client.createSnapshot(name, subscription); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -618,8 +618,8 @@ public void deleteSnapshotExceptionTest() throws Exception { client.deleteSnapshot(snapshot); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -657,8 +657,8 @@ public void setIamPolicyExceptionTest() throws Exception { client.setIamPolicy(formattedResource, policy); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -693,8 +693,8 @@ public void getIamPolicyExceptionTest() throws Exception { client.getIamPolicy(formattedResource); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -731,8 +731,8 @@ public void testIamPermissionsExceptionTest() throws Exception { client.testIamPermissions(formattedResource, permissions); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/TopicAdminClientTest.java b/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/TopicAdminClientTest.java index 9fe3cfd26f5f..f1d19e82ee3e 100644 --- a/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/TopicAdminClientTest.java +++ b/google-cloud-pubsub/src/test/java/com/google/cloud/pubsub/v1/TopicAdminClientTest.java @@ -19,10 +19,10 @@ import static com.google.cloud.pubsub.v1.PagedResponseWrappers.ListTopicsPagedResponse; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.common.collect.Lists; import com.google.iam.v1.GetIamPolicyRequest; import com.google.iam.v1.Policy; @@ -132,8 +132,8 @@ public void createTopicExceptionTest() throws Exception { client.createTopic(name); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -176,8 +176,8 @@ public void publishExceptionTest() throws Exception { client.publish(topic, messages); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -211,8 +211,8 @@ public void getTopicExceptionTest() throws Exception { client.getTopic(topic); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -255,8 +255,8 @@ public void listTopicsExceptionTest() throws Exception { client.listTopics(project); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -305,8 +305,8 @@ public void listTopicSubscriptionsExceptionTest() throws Exception { client.listTopicSubscriptions(topic); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -338,8 +338,8 @@ public void deleteTopicExceptionTest() throws Exception { client.deleteTopic(topic); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -377,8 +377,8 @@ public void setIamPolicyExceptionTest() throws Exception { client.setIamPolicy(formattedResource, policy); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -413,8 +413,8 @@ public void getIamPolicyExceptionTest() throws Exception { client.getIamPolicy(formattedResource); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -451,8 +451,8 @@ public void testIamPermissionsExceptionTest() throws Exception { client.testIamPermissions(formattedResource, permissions); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-speech/src/test/java/com/google/cloud/speech/v1/SpeechClientTest.java b/google-cloud-speech/src/test/java/com/google/cloud/speech/v1/SpeechClientTest.java index da3572ea6c2a..2ddefed03ff8 100644 --- a/google-cloud-speech/src/test/java/com/google/cloud/speech/v1/SpeechClientTest.java +++ b/google-cloud-speech/src/test/java/com/google/cloud/speech/v1/SpeechClientTest.java @@ -16,13 +16,14 @@ package com.google.cloud.speech.v1; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.grpc.GrpcStatusCode; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; import com.google.api.gax.grpc.testing.MockStreamObserver; import com.google.api.gax.rpc.ApiStreamObserver; import com.google.api.gax.rpc.BidiStreamingCallable; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.cloud.speech.v1.RecognitionConfig.AudioEncoding; import com.google.longrunning.Operation; import com.google.protobuf.Any; @@ -128,8 +129,8 @@ public void recognizeExceptionTest() throws Exception { client.recognize(config, audio); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -192,10 +193,11 @@ public void longRunningRecognizeExceptionTest() throws Exception { client.longRunningRecognizeAsync(config, audio).get(); Assert.fail("No exception raised"); } catch (ExecutionException e) { - Assert.assertEquals(GrpcApiException.class, e.getCause().getClass()); - GrpcApiException apiException = (GrpcApiException) e.getCause(); + Assert.assertEquals(InvalidArgumentException.class, e.getCause().getClass()); + InvalidArgumentException apiException = (InvalidArgumentException) e.getCause(); Assert.assertEquals( - Status.INVALID_ARGUMENT.getCode(), apiException.getStatusCode().getCode()); + Status.INVALID_ARGUMENT.getCode(), + ((GrpcStatusCode) apiException.getStatusCode()).getCode()); } } diff --git a/google-cloud-speech/src/test/java/com/google/cloud/speech/v1beta1/SpeechClientTest.java b/google-cloud-speech/src/test/java/com/google/cloud/speech/v1beta1/SpeechClientTest.java index ccf8a4e78f5d..ae012c5148d4 100644 --- a/google-cloud-speech/src/test/java/com/google/cloud/speech/v1beta1/SpeechClientTest.java +++ b/google-cloud-speech/src/test/java/com/google/cloud/speech/v1beta1/SpeechClientTest.java @@ -16,13 +16,14 @@ package com.google.cloud.speech.v1beta1; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.grpc.GrpcStatusCode; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; import com.google.api.gax.grpc.testing.MockStreamObserver; import com.google.api.gax.rpc.ApiStreamObserver; import com.google.api.gax.rpc.BidiStreamingCallable; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.cloud.speech.v1beta1.RecognitionConfig.AudioEncoding; import com.google.longrunning.Operation; import com.google.protobuf.Any; @@ -118,8 +119,8 @@ public void syncRecognizeExceptionTest() throws Exception { client.syncRecognize(config, audio); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -170,10 +171,11 @@ public void asyncRecognizeExceptionTest() throws Exception { client.asyncRecognizeAsync(config, audio).get(); Assert.fail("No exception raised"); } catch (ExecutionException e) { - Assert.assertEquals(GrpcApiException.class, e.getCause().getClass()); - GrpcApiException apiException = (GrpcApiException) e.getCause(); + Assert.assertEquals(InvalidArgumentException.class, e.getCause().getClass()); + InvalidArgumentException apiException = (InvalidArgumentException) e.getCause(); Assert.assertEquals( - Status.INVALID_ARGUMENT.getCode(), apiException.getStatusCode().getCode()); + Status.INVALID_ARGUMENT.getCode(), + ((GrpcStatusCode) apiException.getStatusCode()).getCode()); } } diff --git a/google-cloud-trace/src/test/java/com/google/cloud/trace/v1/TraceServiceClientTest.java b/google-cloud-trace/src/test/java/com/google/cloud/trace/v1/TraceServiceClientTest.java index 4e21f777d26f..27e28b69b86c 100644 --- a/google-cloud-trace/src/test/java/com/google/cloud/trace/v1/TraceServiceClientTest.java +++ b/google-cloud-trace/src/test/java/com/google/cloud/trace/v1/TraceServiceClientTest.java @@ -18,10 +18,10 @@ import static com.google.cloud.trace.v1.PagedResponseWrappers.ListTracesPagedResponse; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.common.collect.Lists; import com.google.devtools.cloudtrace.v1.GetTraceRequest; import com.google.devtools.cloudtrace.v1.ListTracesRequest; @@ -112,8 +112,8 @@ public void patchTracesExceptionTest() throws Exception { client.patchTraces(projectId, traces); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -152,8 +152,8 @@ public void getTraceExceptionTest() throws Exception { client.getTrace(projectId, traceId); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } @@ -196,8 +196,8 @@ public void listTracesExceptionTest() throws Exception { client.listTraces(projectId); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/google-cloud-video-intelligence/src/test/java/com/google/cloud/videointelligence/v1beta1/VideoIntelligenceServiceClientTest.java b/google-cloud-video-intelligence/src/test/java/com/google/cloud/videointelligence/v1beta1/VideoIntelligenceServiceClientTest.java index c8eada856021..d0a926e648ba 100644 --- a/google-cloud-video-intelligence/src/test/java/com/google/cloud/videointelligence/v1beta1/VideoIntelligenceServiceClientTest.java +++ b/google-cloud-video-intelligence/src/test/java/com/google/cloud/videointelligence/v1beta1/VideoIntelligenceServiceClientTest.java @@ -16,10 +16,11 @@ package com.google.cloud.videointelligence.v1beta1; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; +import com.google.api.gax.grpc.GrpcStatusCode; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.longrunning.Operation; import com.google.protobuf.Any; import com.google.protobuf.GeneratedMessageV3; @@ -125,10 +126,11 @@ public void annotateVideoExceptionTest() throws Exception { client.annotateVideoAsync(inputUri, features, videoContext, outputUri, locationId).get(); Assert.fail("No exception raised"); } catch (ExecutionException e) { - Assert.assertEquals(GrpcApiException.class, e.getCause().getClass()); - GrpcApiException apiException = (GrpcApiException) e.getCause(); + Assert.assertEquals(InvalidArgumentException.class, e.getCause().getClass()); + InvalidArgumentException apiException = (InvalidArgumentException) e.getCause(); Assert.assertEquals( - Status.INVALID_ARGUMENT.getCode(), apiException.getStatusCode().getCode()); + Status.INVALID_ARGUMENT.getCode(), + ((GrpcStatusCode) apiException.getStatusCode()).getCode()); } } } diff --git a/google-cloud-vision/src/test/java/com/google/cloud/vision/v1/ImageAnnotatorClientTest.java b/google-cloud-vision/src/test/java/com/google/cloud/vision/v1/ImageAnnotatorClientTest.java index 769a952a7a0f..6b94b3a6e2be 100644 --- a/google-cloud-vision/src/test/java/com/google/cloud/vision/v1/ImageAnnotatorClientTest.java +++ b/google-cloud-vision/src/test/java/com/google/cloud/vision/v1/ImageAnnotatorClientTest.java @@ -16,10 +16,10 @@ package com.google.cloud.vision.v1; import com.google.api.gax.core.NoCredentialsProvider; -import com.google.api.gax.grpc.GrpcApiException; import com.google.api.gax.grpc.GrpcTransportProvider; import com.google.api.gax.grpc.testing.MockGrpcService; import com.google.api.gax.grpc.testing.MockServiceHelper; +import com.google.api.gax.rpc.InvalidArgumentException; import com.google.protobuf.GeneratedMessageV3; import io.grpc.Status; import io.grpc.StatusRuntimeException; @@ -101,8 +101,8 @@ public void batchAnnotateImagesExceptionTest() throws Exception { client.batchAnnotateImages(requests); Assert.fail("No exception raised"); - } catch (GrpcApiException e) { - Assert.assertEquals(Status.INVALID_ARGUMENT.getCode(), e.getStatusCode().getCode()); + } catch (InvalidArgumentException e) { + // Expected exception } } } diff --git a/pom.xml b/pom.xml index a77b5ca29531..6f4ea168c894 100644 --- a/pom.xml +++ b/pom.xml @@ -131,8 +131,8 @@ 1.22.0 1.1.0 - 1.6.0 - 0.23.0 + 1.7.0 + 0.24.0 0.1.15 0.7.0 1.4.0