Skip to content

Clean up Vision unit tests. #3082

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 28, 2017
Merged

Clean up Vision unit tests. #3082

merged 1 commit into from
Feb 28, 2017

Conversation

daspecster
Copy link
Contributor

@daspecster daspecster commented Feb 28, 2017 •
edited
Loading

Some cleanup from some original unit test inconsistencies.

@daspecster daspecster added the api: vision Issues related to the Cloud Vision API. label Feb 28, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 28, 2017
dhermes
dhermes approved these changes Feb 28, 2017
View reviewed changes
Copy link
Contributor

@dhermes dhermes left a comment

Choose a reason for hiding this comment

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

LGTM if CI passes

@daspecster daspecster merged commit c090fb5 into googleapis:master Feb 28, 2017
@daspecster daspecster deleted the vision-test-updates branch February 28, 2017 15:08
richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
@daspecster
Merge pull request googleapis#3082 from daspecster/vision-test-updates
8b576ad 
Clean up Vision unit tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@dhermes dhermes dhermes approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@lukesneeringer lukesneeringer

@tseaver tseaver

Labels

api: vision Issues related to the Cloud Vision API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@daspecster @dhermes @lukesneeringer @tseaver @googlebot