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.

Sign up for GitHub

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

Jump to bottom

Getting test coverage for the current package seems to require the package to be installed? #2535

Closed
jennybc opened this issue Sep 17, 2023 · 2 comments
Closed

Getting test coverage for the current package seems to require the package to be installed? #2535

jennybc opened this issue Sep 17, 2023 · 2 comments

Comments

@jennybc
Copy link
Member

jennybc commented Sep 17, 2023 •
edited
Loading

Observation from conf workshop:

Several folks, all on Windows I believe (that could be a red herring yeah that is not important I can reproduce this), seem to need to install the current package before they can get test coverage for it.
@jennybc
Copy link
Member Author

jennybc commented Sep 17, 2023 •
edited
Loading

Error actually occurs in testthat::local_test_directory(pkg$path, pkg$package), because of local_edition(find_edition(path, package), .env = .env).

@jennybc jennybc mentioned this issue Sep 17, 2023
Closed
@jennybc
Copy link
Member Author

jennybc commented Sep 17, 2023

I think it's better to resolve this in testthat. I'm going to open an issue there.

@jennybc jennybc closed this as not planned Won't fix, can't repro, duplicate, stale Sep 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't pass the package name r-lib/devtools
1 participant
@jennybc