Skip to content
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

Test coverage for Pkg #17996

Closed
davidanthoff opened this issue Aug 12, 2016 · 3 comments
Closed

Test coverage for Pkg #17996

davidanthoff opened this issue Aug 12, 2016 · 3 comments
Labels
domain:packages Package management and loading test This change adds or pertains to unit tests

Comments

@davidanthoff
Copy link
Contributor

I believe I hit at least two repo corrupting bugs in the package manager on julia 0.5 RC2 in the first hour of using RC2 (#17994, #17995). None of what I did seems very exotic/corner cases etc. This does suggest that Pkg could use more comprehensive test coverage that actually exercises typical user scenarios, so that these sorts of bugs can be found in CI builds.

I'm just opening this issue so that such work can be tracked.

@kshyatt kshyatt added test This change adds or pertains to unit tests domain:packages Package management and loading labels Aug 12, 2016
@KristofferC
Copy link
Sponsor Member

I added libgit2 to #11885. Do you feel it is still worth having a separate issue open?

@davidanthoff
Copy link
Contributor Author

Good question, I'm not sure. I do feel that this is more about making sure that common Pkg manager user scenarios are actually tested end-to-end rather than getting high code coverage results.

@KristofferC
Copy link
Sponsor Member

KristofferC commented Aug 12, 2016

It seems sensible to me to have one issue dealing with coverage in general and then specific issues regarding specific bugs. Are there any known bugs with the package manager that we are not tracking?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:packages Package management and loading test This change adds or pertains to unit tests
Projects
None yet
Development

No branches or pull requests

3 participants