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

errors from R CMD check #636

Open
aronatkins opened this issue Aug 12, 2021 · 2 comments
Open

errors from R CMD check #636

aronatkins opened this issue Aug 12, 2021 · 2 comments

Comments

@aronatkins
Copy link
Contributor

══ Warnings ════════════════════════════════════════════════════════════════════
── Warning (test-local-repositories.R:8:5): init fails when package not found in any repo ──
path[1]="repo-empty": No such file or directory
Backtrace:
 1. base::options(repos = c(CRAN = paste0("file:///", normalizePath("repo-empty")))) test-local-repositories.R:8:4
 3. base::normalizePath("repo-empty")

══ Failed tests ════════════════════════════════════════════════════════════════
── Failure (test-packrat.R:95:5): snapshot captures new dependencies ───────────
"bread" %in% pkgs is not TRUE

`actual`:   FALSE
`expected`: TRUE 
── Failure (test-packrat.R:96:5): snapshot captures new dependencies ───────────
"toast" %in% pkgs is not TRUE

`actual`:   FALSE
`expected`: TRUE 
── Failure (test-packrat.R:120:5): snapshot captures only installed dependecies butwhen infer.dependencies is FALSE ──
"bread" %in% pkgs is not TRUE

`actual`:   FALSE
`expected`: TRUE 

[ FAIL 3 | WARN 1 | SKIP 5 | PASS 134 ]
Error: Test failures
Execution halted

1 error x | 0 warnings ✓ | 0 notes ✓
Error: R CMD check found ERRORs
@kevinushey
Copy link
Contributor

kevinushey commented Aug 12, 2021

══ Warnings ════════════════════════════════════════════════════════════════════
── Warning (test-local-repositories.R:8:5): init fails when package not found in any repo ──
path[1]="repo-empty": No such file or directory
Backtrace:
 1. base::options(repos = c(CRAN = paste0("file:///", normalizePath("repo-empty")))) test-local-repositories.R:8:4
 3. base::normalizePath("repo-empty")

This probably happens because R removes empty directories during build. We could probably subvert it by adding a single empty file or something similar? Or by creating that directory on demand rather than expecting it to exist.

I don't have a great idea on the other issues, though. :-/

@aronatkins
Copy link
Contributor Author

The other failures had skip_on_travis but that doesn't work in GHA. They now have skip_on_ci as of #638.

Sadly, we now see:

══ Failed tests ════════════════════════════════════════════════════════════════
── Failure (test-dependencies.R:58:3): dependencies are discovered in inline R code ──
"emo" %in% packrat:::fileDependencies(emojiRmd) is not TRUE

`actual`:   FALSE
`expected`: TRUE 
── Failure (test-rmarkdown.R:22:3): We can discover dependencies with an evaluate hook ──
`deps` not equal to c("abc", "def", "ghi", "jkl").
target is NULL, current is character

Unlike the others, these failures do not happen in local "check" runs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants