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

New test failures on r-devel #106

Closed
cpsievert opened this issue Nov 15, 2021 · 0 comments · Fixed by #107
Closed

New test failures on r-devel #106

cpsievert opened this issue Nov 15, 2021 · 0 comments · Fixed by #107

Comments

@cpsievert
Copy link
Collaborator

From https://cran.r-project.org/web/checks/check_results_shinymeta.html

(Most likely a result of wch/r-source@3a3343e)

Check: tests
Result: ERROR
     Running ‘testthat.R’ [10s/17s]
    Running the tests in ‘tests/testthat.R’ failed.
    Complete output:
     > library(testthat)
     > library(shinymeta)
     >
     > test_check("shinymeta")
     adding: template.html (deflated 64%)
     adding: template.Rmd (deflated 21%)
     ══ Skipped tests ═══════════════════════════════════════════════════════════════
     • On CRAN (4)
    
     ══ Failed tests ════════════════════════════════════════════════════════════════
     ── Failure (test-expandchain.R:97:5): expansion context: can substitute ────────
     `x` (`actual`) not identical to c(...) (`expected`).
    
     actual vs expected
     "# Add one hundred and two hundred"
     "mr2 <- 100 + 200"
     "mrFour <- 4"
     - "mr2 + 1000 + 2000 + mrFour"
     + "mr2 + (1000 + 2000) + mrFour"
    
     [ FAIL 1 | WARN 22 | SKIP 4 | PASS 204 ]
     Deleting unused snapshots:
     * report/template.Rmd
     Error: Test failures
     Execution halted
Flavor: r-devel-linux-x86_64-debian-gcc

Version: 0.2.0.2
Check: tests
Result: ERROR
     Running ‘testthat.R’ [15s/22s]
    Running the tests in ‘tests/testthat.R’ failed.
    Complete output:
     > library(testthat)
     > library(shinymeta)
     >
     > test_check("shinymeta")
     adding: template.html (deflated 64%)
     adding: template.Rmd (deflated 21%)
     ══ Skipped tests ═══════════════════════════════════════════════════════════════
     • On CRAN (4)
    
     ══ Failed tests ════════════════════════════════════════════════════════════════
     ── Failure (test-expandchain.R:97:5): expansion context: can substitute ────────
     `x` (`actual`) not identical to c(...) (`expected`).
    
     actual vs expected
     "# Add one hundred and two hundred"
     "mr2 <- 100 + 200"
     "mrFour <- 4"
     - "mr2 + 1000 + 2000 + mrFour"
     + "mr2 + (1000 + 2000) + mrFour"
    
     [ FAIL 1 | WARN 22 | SKIP 4 | PASS 204 ]
     Deleting unused snapshots:
     * report/template.Rmd
     Error: Test failures
     Execution halted
Flavor: r-devel-linux-x86_64-fedora-clang

Version: 0.2.0.2
Check: tests
Result: ERROR
     Running ‘testthat.R’ [14s/16s]
    Running the tests in ‘tests/testthat.R’ failed.
    Complete output:
     > library(testthat)
     > library(shinymeta)
     >
     > test_check("shinymeta")
     adding: template.html (deflated 64%)
     adding: template.Rmd (deflated 21%)
     ══ Skipped tests ═══════════════════════════════════════════════════════════════
     • On CRAN (4)
    
     ══ Failed tests ════════════════════════════════════════════════════════════════
     ── Failure (test-expandchain.R:97:5): expansion context: can substitute ────────
     `x` (`actual`) not identical to c(...) (`expected`).
    
     actual vs expected
     "# Add one hundred and two hundred"
     "mr2 <- 100 + 200"
     "mrFour <- 4"
     - "mr2 + 1000 + 2000 + mrFour"
     + "mr2 + (1000 + 2000) + mrFour"
    
     [ FAIL 1 | WARN 22 | SKIP 4 | PASS 204 ]
     Deleting unused snapshots:
     * report/template.Rmd
     Error: Test failures
     Execution halted
Flavor: r-devel-linux-x86_64-fedora-gcc
cpsievert added a commit that referenced this issue Nov 15, 2021
(It appears that recent changes to base::deparse() fix that fact that no parentheses were needed/requested)
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

Successfully merging a pull request may close this issue.

1 participant