Skip to content

test: option to run integration tests with modified dependencies #2392

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

suztomo
Copy link
Member

@suztomo suztomo commented Apr 19, 2025

This pull request explores the idea of running the integration
tests with modified dependency versions.

First, the integraiton test script (build.sh) compiles the project
with normal dependencies (the latest shared dependencies BOM).
After that, when it runs "mvn verify", it uses slightly modified
dependencies (-Dgrpc.version=1.72.0 in this case), without
re-compiling the main source code by -Dmaven.main.skip argument.

This runtime is similar to what our library users would have when
they upgrade the 3rd-party dependencies in their builds, because
they do not recompile GCP client libraries.

This pull request explores the idea of running the integration
tests with modified dependency versions.

First, the integraiton test script (build.sh) compiles the project
with normal dependencies (the latest shared dependencies BOM).
After that, when it runs "mvn verify", it uses slightly modified
dependencies (`-Dgrpc.version=1.72.0` in this case), without
re-compiling the main source code by `-Dmaven.main.skip` argument.

This runtime is similar to what our library users would have when
they upgrade the 3rd-party dependencies in their builds, because
they do not recompile GCP client libraries.
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: pubsub Issues related to the googleapis/java-pubsub API. labels Apr 19, 2025
@suztomo suztomo force-pushed the additional_class_path branch from fb860d7 to 31d6733 Compare April 19, 2025 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsub Issues related to the googleapis/java-pubsub API. size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant