-
Notifications
You must be signed in to change notification settings - Fork 2
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
Cancel tower analysis #286
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice 👍 Was a little confused by some stuff, nothing about the logic though
tests passing |
Kudos, SonarCloud Quality Gate passed!
|
Description
Closes: #226
Changed
cancel
supports NF-Tower casesHow to prepare for test
us
paxa
How to test
trailblazer cancel 279435
. It should raise the following error:TrailblazerError: Analysis 279435 is not running
trailblazer cancel 279435
. It should raise the following error:Request failed for url {url}: Error: {error}
trailblazer cancel 271983
. It complete successfully with the following message:Analysis 279435: cancelled successfully!
Review
Thanks for filling in who performed the code review and the test!
This version is a