-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: Add HttpJson CRUD Showcase tests #1589
Conversation
Failure is resolved with this fix: googleapis/gapic-showcase#1263 -- Released in Showcase v0.26.1. Will create a separate PR to upgrade to the latest showcase version to unblock this PR. |
showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITCrud.java
Outdated
Show resolved
Hide resolved
@burkedavison @mpeddada1 @blakeli0 Would you all be able to re-review this PR? |
showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITCrud.java
Outdated
Show resolved
Hide resolved
showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITCrud.java
Outdated
Show resolved
Hide resolved
showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITCrud.java
Show resolved
Hide resolved
showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITCrud.java
Show resolved
Hide resolved
showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITCrud.java
Outdated
Show resolved
Hide resolved
showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITCrud.java
Show resolved
Hide resolved
Thanks all. Will merge tomorrow if unless there are any more concerns. |
showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITCrud.java
Outdated
Show resolved
Hide resolved
[gapic-generator-java-root] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
[java_showcase_integration_tests] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
[java_showcase_unit_tests] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@@ -107,7 +107,7 @@ | |||
<artifactId>fmt-maven-plugin</artifactId> | |||
<version>2.9</version> | |||
<configuration> | |||
<filesNamePattern>(IT.*\.java)|(.*Test.java)</filesNamePattern> | |||
<filesNamePattern>(IT.*\.java)|(.*Test.java)|(TestClientInitializer.java)</filesNamePattern> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder why we are explicitly specifying the files in the fmt plugin, because we want to exclude some files? @mpeddada1
Thank you for opening a Pull Request! For general contributing guidelines, please refer to contributing guide
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
Part of #1439 ☕️