Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Investigate gaps in existing Application Set unit tests (via coverage stats), and open issues for them #82

Closed
jgwest opened this issue Dec 17, 2020 · 2 comments
Assignees

Comments

@jgwest
Copy link
Member

jgwest commented Dec 17, 2020

Look through the coverage report and look for (significant) areas of the code that are not thoroughly tested. For each identified issue (or set of related issues), open a new GitHub issue. Re: definition of 'significant', part of this issue is developing an understanding/model for what can currently be considered 'significant' 😄 .

If #64 has merged, you can run make test, otherwise:

go test -race -count=1 -coverprofile=coverage.out `go list ./...`

to generate the coverage.out data.

To use this coverage file, generate an HTML report and open in the browser:

go tool cover -html=coverage.out
@jgwest
Copy link
Member Author

jgwest commented Jan 11, 2021

Working on this.

@jgwest jgwest self-assigned this Jan 12, 2021
@jgwest jgwest added this to the Milestone 1 milestone Jan 12, 2021
@jgwest
Copy link
Member Author

jgwest commented Jan 26, 2021

Done, new issues opened:

Write additional tests for repo_service.go:
#102

Write tests for pkg/utils/util.go:
#100

Write tests for clustereventhandler.go:
#99

Write tests for list.go generator:
#98

@jgwest jgwest closed this as completed Jan 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant