We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Related to #2861
We need to add bunch of fixtures for more robust tests. However, this is difficult now because the current sample code often does something like this:
try: response = client.create_device_registry(parent, body) print('Created registry') return response except HttpError: print('Error, registry not created') return "" except AlreadyExists: print('Error, registry already exists') return ""
This is bad because of the following reasons:
I think we should re-raise the exception (it's good to have these except clauses to show how to handle exceptions).
except
I'm going to fix it first before building the fixture for #2861
@gguuss FYI
The text was updated successfully, but these errors were encountered:
[iot] testing: re-raise the exception
a878861
fixes GoogleCloudPlatform#3424
[iot] testing: re-raise the exception (#3425)
b57b562
fixes #3424
tmatsuo
Successfully merging a pull request may close this issue.
Related to #2861
We need to add bunch of fixtures for more robust tests. However, this is difficult now because the current sample code often does something like this:
This is bad because of the following reasons:
I think we should re-raise the exception (it's good to have these
except
clauses to show how to handle exceptions).I'm going to fix it first before building the fixture for #2861
@gguuss FYI
The text was updated successfully, but these errors were encountered: