Skip to content
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

Spanner test fix #1711

Merged
merged 7 commits into from
Mar 8, 2017
Merged

Spanner test fix #1711

merged 7 commits into from
Mar 8, 2017

Conversation

vkedia
Copy link

@vkedia vkedia commented Mar 8, 2017

Fixed ITDatabaseAdminTest to only drop the databases it created rather than dropping all the databases in the instance.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 8, 2017
@vkedia vkedia closed this Mar 8, 2017
@vkedia vkedia reopened this Mar 8, 2017
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 8c5a6ec on vkedia:test-fix into ** on GoogleCloudPlatform:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling c964be8 on vkedia:test-fix into ** on GoogleCloudPlatform:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling c964be8 on vkedia:test-fix into ** on GoogleCloudPlatform:master**.

@garrettjonesgoogle
Copy link
Member

LGTM. Be sure to use "squash and merge".

@vkedia vkedia merged commit 7716d73 into googleapis:master Mar 8, 2017
rybosome pushed a commit to rybosome/google-cloud-java that referenced this pull request Mar 9, 2017

* Fixed a spanner test to only delete databases it created

* Fix indentation
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.

4 participants