Update BigTable conformance tests (#5901) by BenWhitehead · Pull Request #6166 · googleapis/google-cloud-java · GitHub
Skip to content

Update BigTable conformance tests (#5901)#6166

Merged
BenWhitehead merged 4 commits into
googleapis:masterfrom
BenWhitehead:bigtable-conformance-tests-update
Aug 26, 2019
Merged

Update BigTable conformance tests (#5901)#6166
BenWhitehead merged 4 commits into
googleapis:masterfrom
BenWhitehead:bigtable-conformance-tests-update

Conversation

@BenWhitehead

Copy link
Copy Markdown
Contributor
  • Refactor ReadRowsMergingAcceptanceTest to use the generated Java
    Protobuf classes from the google-cloud-conformance-test module.
    The format read from is still json.
  • Update to use the latest revision of the conformance-tests data.

* Refactor ReadRowsMergingAcceptanceTest to use the generated Java
  Protobuf classes from the google-cloud-conformance-test module.
  The format read from is still json.
* Update to use the latest revision of the conformance-tests data.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 26, 2019

@igorbernstein2 igorbernstein2 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.

Thanks for getting all of the clients aligned!

Comment thread google-cloud-clients/google-cloud-bigtable/pom.xml
@codecov

codecov Bot commented Aug 26, 2019

Copy link
Copy Markdown

@igorbernstein2 igorbernstein2 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.

LGTM! thanks again for normalizing expectations across clients!

@BenWhitehead

Copy link
Copy Markdown
Contributor Author

@BenWhitehead BenWhitehead merged commit 6de998c into googleapis:master Aug 26, 2019
@BenWhitehead BenWhitehead deleted the bigtable-conformance-tests-update branch August 26, 2019 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants