-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Labels
good first issueGood for newcomersGood for newcomersrefactorupdates codebase without any feature addition or bug fixesupdates codebase without any feature addition or bug fixes
Description
Is your feature request related to a problem? Please describe.
We could aim for higher code coverage of 95% for the compass.
Describe the solution you'd like
To achieve it we need to work on hitting all the statements where the error is being caught, and it should help us understand and debug our code in case an unexpected behavior is noticed.
Additional context
- core package (overall coverage - 94.75%):
core/asset/patch.go- 89.8%core/star- no test filescore/tag- few test cases where error handling can be hit.core/user- mostly seems good, opts can be checked inservice.gofile for func NewService()core/validator- 3 error conditions can be hit
- internal package (overall coverage 84.41%):
internal/client- no test files for client.gointernal/server/health- 100% coverageinternal/server/v1beta1- 84.33%internal/store/elasticsearch- discovery_repository, discovery_search_repository andes.gofiles need more coverage on error handling.internal/store/postgres- more coverage for error handling
- pkg package (overall coverage 87.88%):
pkg/statsd- has no test files.
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomersrefactorupdates codebase without any feature addition or bug fixesupdates codebase without any feature addition or bug fixes