Add sanity check tests for all REST structs
Open
Nobody has claimed this yet.
area/techdebt
component/restapi
lifecycle/frozen
lifecycle/stale
priority/P2
- Dominant language
- Go
- Stars
- 8.7k
- Forks
- 4.8k
- Avg merge
- 4d 10h
- Merged PRs (30d)
- 53
Description
- Make sure they use the default
ObjectNameFuncunless they have fancy logic to avoid #13124 - Make sure they have functional
GetAttrsto avoid #14853 - ...
These would be upstream tests.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the REST structs and existing upstream tests in the repository. Check how ObjectNameFunc and GetAttrs are implemented, including the exceptions for fancy logic, then add coverage for all applicable structs and confirm the checks catch the issues described in #13124 and #14853.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- api, testing-qa
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100