-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add test case issue reporting #207
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.
The code looks good to me, however I left two small notes where I think we can improve the style.
My understanding from your explanation was that we would also have some kind of "confirming issues", so that when someone reports an issue, someone from the team would need to confirm it. Or maybe this was only for reporting issues to environments? I got confused between the two.
You are right. As per spec, confirming issues is only for environments. It's assumed that when users create an issue on test cases, it's already confirmed. But environments are treated differently to allow a way for reviewers to tell certlab that something needs checking (e.g. machine is not provisioning, wifi keeps failing, etc.). Once certlab confirmed the issue they'd update it and mark it as confirmed. |
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.
Thanks for following up on this! Looks really good, excited to see it landed :d
backend/migrations/versions/2024_08_30_1159-6f3eeb521421_add_case_name_to_testcaseissues.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Nadzeya H <[email protected]>
Co-authored-by: Nadzeya H <[email protected]>
Description
Adds model and endpoints to support reporting test case issues.
Resolved issues
Backend part of https://warthogs.atlassian.net/browse/RTW-355
Documentation
Web service API changes
New endpoints:
Tests
Added automated tests.