Add RemoteBigQueryHelper and integration tests #492

mziccard · 2015-12-18T21:25:43Z

This PR does the following

Add RemoteBigQueryHelper to help writing tests against BigQuery (with docs and tests)
Add integration tests (ITBigQueryTest)
Add gcloud-java-storage as a test dependency
Update bigquery dependency

Remarks: integration tests might take long to execute depending on how long it takes for jobs to complete. I decided to push them anyway and see how long they take, if we see that they take too much we can avoid merging or remove the tests that involve longer jobs (load?)

- Add RemoteBigQueryHelper to help writing tests against BigQuery (with docs and tests) - Add integration tests (ITBigQueryTest - Add gcloud-java-storage as a test dependency - Update bigquery dependency

gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/testing/RemoteBigQueryHelper.java

+import java.util.logging.Logger;
+
+/**
+ * Utility to create a remote BigQuery configuration for testing.


gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/Acl.java

@@ -163,8 +163,8 @@ Access toPb() {
  }

  /**
-   * Class for a BigQuery Group entity. Objects of this class represent a group to grante access to.
-   * A Group entity can be created given the group's email or can be a special group:
+   * Class for a BigQuery Group entity. Objects of this class represent a group to granted access


aozarov · 2015-12-19T04:32:15Z

Looks good. few comments.

- Remove RemoteBigQueryHelper.create(String, String) - Add javadoc to RemoteBigQueryHelper to document retry options - Add global timeout of 5minutes to each of the BigQuery ITs - Remove print from BigQueryImplTest and other nits

mziccard · 2015-12-21T07:31:21Z

Comments addresses, what is related to storage will come in a separate PR later today

Add RemoteBigQueryHelper and integration tests

🤖 I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

Making CLIRR not required. The version bumps are now controlled by the Release Please and OwlBot. The CL authors create appropriate change description to control major version bumps.

…-info-reports-plugin to v3.4.0 (#492) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [org.apache.maven.plugins:maven-project-info-reports-plugin](https://maven.apache.org/plugins/) | `3.3.0` -> `3.4.0` | [![age](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-project-info-reports-plugin/3.4.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-project-info-reports-plugin/3.4.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-project-info-reports-plugin/3.4.0/compatibility-slim/3.3.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-project-info-reports-plugin/3.4.0/confidence-slim/3.3.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-retail).

…s to v2.3.3 (#492) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-workflows](https://togithub.com/googleapis/java-workflows) | `2.3.2` -> `2.3.3` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-workflows/2.3.3/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-workflows/2.3.3/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-workflows/2.3.3/compatibility-slim/2.3.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-workflows/2.3.3/confidence-slim/2.3.2)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. ⚠ **Warning**: custom changes will be lost. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-workflows).

🤖 I have created a release *beep* *boop* --- ## [0.13.5](https://togithub.com/googleapis/java-analytics-admin/compare/v0.13.4...v0.13.5) (2022-10-05) ### Bug Fixes * update protobuf to v3.21.7 ([68a6a94](https://togithub.com/googleapis/java-analytics-admin/commit/68a6a9412103c1e8fe872200ffc5535a3dd4e2a0)) --- This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).

…workloads to v2.3.0 (#492) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-assured-workloads](https://togithub.com/googleapis/java-assured-workloads) ([source](https://togithub.com/googleapis/java-)) | `2.2.1` -> `2.3.0` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-assured-workloads/2.3.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-assured-workloads/2.3.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-assured-workloads/2.3.0/compatibility-slim/2.2.1)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-assured-workloads/2.3.0/confidence-slim/2.2.1)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. ⚠ **Warning**: custom changes will be lost. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-assured-workloads).

…492) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:google-iam-policy](https://togithub.com/googleapis/java-eventarc) ([source](https://togithub.com/googleapis/java-iam)) | `1.6.5` -> `1.6.6` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-iam-policy/1.6.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-iam-policy/1.6.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-iam-policy/1.6.6/compatibility-slim/1.6.5)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-iam-policy/1.6.6/confidence-slim/1.6.5)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>googleapis/java-iam</summary> ### [`v1.6.6`](https://togithub.com/googleapis/java-iam/blob/HEAD/CHANGELOG.md#166-httpsgithubcomgoogleapisjava-iamcomparev165v166-2022-10-25) [Compare Source](https://togithub.com/googleapis/java-iam/compare/v1.6.5...v1.6.6) ##### Dependencies - Update dependency com.google.cloud:google-iam-policy to v1.6.5 ([#489](https://togithub.com/googleapis/java-iam/issues/489)) ([558cafc](https://togithub.com/googleapis/java-iam/commit/558cafcfc10c7465465499e6df28c66afd57f6bf)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-iam).

Add integration test

6bc4f2c

- Add RemoteBigQueryHelper to help writing tests against BigQuery (with docs and tests) - Add integration tests (ITBigQueryTest - Add gcloud-java-storage as a test dependency - Update bigquery dependency

mziccard added the api: bigquery Issues related to the BigQuery API. label Dec 18, 2015

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 18, 2015

Add BigQuery module to main pom

e46ce4f

ajkannan reviewed Dec 18, 2015
View reviewed changes

gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/testing/RemoteBigQueryHelper.java

import java.util.logging.Logger;

/**

* Utility to create a remote BigQuery configuration for testing.

This comment was marked as spam.

Sign in to view

Few minor fixes and make java8 happy

ba8acf1

aozarov reviewed Dec 19, 2015
View reviewed changes

Minor fixes to RemoteBigQueryHelper

7eee528

- Remove RemoteBigQueryHelper.create(String, String) - Add javadoc to RemoteBigQueryHelper to document retry options - Add global timeout of 5minutes to each of the BigQuery ITs - Remove print from BigQueryImplTest and other nits

mziccard force-pushed the bigquery branch from 4d17ea1 to 7eee528 Compare December 21, 2015 07:00

aozarov added a commit that referenced this pull request Dec 21, 2015

Merge pull request #492 from mziccard/bigquery

3781aa7

Add RemoteBigQueryHelper and integration tests

aozarov merged commit 3781aa7 into googleapis:bigquery Dec 21, 2015

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add RemoteBigQueryHelper and integration tests #492

Add RemoteBigQueryHelper and integration tests #492

mziccard commented Dec 18, 2015

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

aozarov commented Dec 19, 2015

mziccard commented Dec 21, 2015

Add RemoteBigQueryHelper and integration tests #492

Add RemoteBigQueryHelper and integration tests #492

Conversation

mziccard commented Dec 18, 2015

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

aozarov commented Dec 19, 2015

mziccard commented Dec 21, 2015