Skip to content

refactor(bigquery): move bigquerystorage integration tests to java-bigquery - #13987

Draft
jinseopkim0 wants to merge 1 commit into
mainfrom
move-bqstorage-it-to-bigquery
Draft

refactor(bigquery): move bigquerystorage integration tests to java-bigquery#13987
jinseopkim0 wants to merge 1 commit into
mainfrom
move-bqstorage-it-to-bigquery

Conversation

@jinseopkim0

Copy link
Copy Markdown
Contributor

Summary

Move the 11 end-to-end integration test classes (95 @Test methods) and helper utilities from java-bigquerystorage into java-bigquery (google-cloud-bigquery/src/test/java/com/google/cloud/bigquery/storage/).

Rationale

  • java-bigquerystorage is a high-throughput streaming data-plane client. It does not have Control Plane RPCs to create or delete BigQuery datasets or tables.
  • The 95 integration tests rely on java-bigquery (com.google.cloud.bigquery.BigQuery) to create temporary test tables in GCP and force-delete them after execution.
  • Since java-bigquery already has a compile dependency on java-bigquerystorage, moving these integration tests into java-bigquery resolves the circular test dependency cleanly while preserving 100% of the end-to-end integration test coverage.
  • java-bigquerystorage retains 398 unit tests (across 37 test classes) that run offline against local mock gRPC servers.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the google-cloud-bigquery test dependency from the java-bigquerystorage/google-cloud-bigquerystorage/pom.xml file. There are no review comments, and I have no feedback to provide.

@jinseopkim0
jinseopkim0 force-pushed the move-bqstorage-it-to-bigquery branch from 0f30c17 to 03eb3fb Compare August 3, 2026 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant