Skip to content

tests(vision): normalize VPCSC configuration in systests - #9620

Merged
tseaver merged 4 commits into
googleapis:masterfrom
tseaver:9580-vision-normalize-vpcsc-config
Nov 11, 2019
Merged

tests(vision): normalize VPCSC configuration in systests#9620
tseaver merged 4 commits into
googleapis:masterfrom
tseaver:9580-vision-normalize-vpcsc-config

Conversation

@tseaver

Copy link
Copy Markdown
Contributor

Toward #7580.

- Use 'vpcsc_config' test utility for parsing VPCSC env vars, skipping.
- Use idiomatic 'with pytest.raises' for blocked methods.
- Avoid calling 'unique_id()' more than once.
- Remove test for VPCSC of storage client.
@tseavertseaver added testing api: vision Issues related to the Cloud Vision API. labels Nov 6, 2019
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 6, 2019
Comment threadvision/tests/system.py Outdated
@tseaver
tseaver merged commit 3130f61 into googleapis:masterNov 11, 2019
@tseaver
tseaver deleted the 9580-vision-normalize-vpcsc-config branch November 11, 2019 19:09
crwilcox pushed a commit to googleapis/python-vision that referenced this pull request Feb 1, 2020
* tests(vision): normalize VPCSC configuration in systests
- Use 'vpcsc_config' test utility for parsing VPCSC env vars, skipping.
- Use idiomatic 'with pytest.raises' for blocked methods.
- Avoid calling 'unique_id()' more than once.
- Remove test for VPCSC of storage client.
* fix: blacken
* fix: lint
* fix: update partial error message
Per request from @brendanlundy:
googleapis/google-cloud-python#9620 (comment)
parthea pushed a commit that referenced this pull request Oct 21, 2023
* tests(vision): normalize VPCSC configuration in systests
- Use 'vpcsc_config' test utility for parsing VPCSC env vars, skipping.
- Use idiomatic 'with pytest.raises' for blocked methods.
- Avoid calling 'unique_id()' more than once.
- Remove test for VPCSC of storage client.
* fix: blacken
* fix: lint
* fix: update partial error message
Per request from @brendanlundy:
#9620 (comment)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: visionIssues related to the Cloud Vision API.cla: yesThis human has signed the Contributor License Agreement.testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@tseaver@brendanlundy@busunkim96@googlebot