Skip to content
This repository was archived by the owner on Jul 20, 2023. It is now read-only.

docs(samples): add try/catch to delete operations in testing#19

Merged
irataxy merged 4 commits intogoogleapis:mainfrom
irataxy:main
Jun 23, 2022
Merged

docs(samples): add try/catch to delete operations in testing#19
irataxy merged 4 commits intogoogleapis:mainfrom
irataxy:main

Conversation

@irataxy
Copy link
Copy Markdown
Contributor

@irataxy irataxy commented Jun 15, 2022

Fixes b:202972969
Fixes: #17
Fixes #18

@irataxy irataxy requested a review from a team June 15, 2022 00:56
@product-auto-label product-auto-label Bot added size: s Pull request size is small. samples Issues that are directly related to samples. labels Jun 15, 2022
@irataxy irataxy added the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 15, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 15, 2022
Comment thread samples/test/stitcher.test.js Outdated
Comment thread samples/test/stitcher.test.js Outdated
@sofisl
Copy link
Copy Markdown
Contributor

sofisl commented Jun 15, 2022

@irataxy, will these close #17 and #18?

…ces over an hour old, move create and delete operations to before and after hooks in each test suite
@product-auto-label product-auto-label Bot added size: l Pull request size is large. and removed size: s Pull request size is small. labels Jun 16, 2022
@irataxy irataxy added the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 16, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 16, 2022
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. and removed size: l Pull request size is large. labels Jun 16, 2022
@irataxy irataxy requested review from alclee-google and sofisl June 16, 2022 15:48
@product-auto-label product-auto-label Bot added the api: videostitcher Issues related to the googleapis/nodejs-video-stitcher API. label Jun 17, 2022
}
);
} catch (err) {
if (err.message.includes('NOT_FOUND')) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Does the error object have a status to check the http status code? Might be cleaner than checking the message.

@sofisl sofisl requested a review from alclee-google June 23, 2022 18:06
@irataxy irataxy merged commit f2cc376 into googleapis:main Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: videostitcher Issues related to the googleapis/nodejs-video-stitcher API. samples Issues that are directly related to samples. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CDN key functions: should delete a Google CDN key failed CDN key functions: should get a Google CDN key failed

3 participants