We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Let this be a lesson to you kids about writing tests based on expectations of messages from other packages.
Consider this test:
sandpaper/tests/testthat/test-build_lesson.R
Lines 7 to 15 in 6211631
We expect knitr to produce output that says "ordinary text without R code" like this:
> knitr::knit(text = "hello there!\n\n```{r rversion}\nR.version.string\n```\n") |> writeLines () |............................ | 33% ordinary text without R code |........................................................ | 67% label: rversion |....................................................................................| 100% ordinary text without R code hello there! ```r R.version.string ``` ``` ## [1] "R version 4.2.2 Patched (2022-11-10 r83330)" ```
Well.... I should have really paid attention because this progress bar has been a thorn in Hadley's craw for a WHILE: yihui/knitr#1880
With {knitr} version 1.42, the printing is now completely changed:
> knitr::knit(text = "hello there!\n\n```{r rversion}\nR.version.string\n```\n") |> writeLines () | | 0% |........................ | 33% |................................................. | 67% (rversion) |.........................................................................| 100% hello there! ```r R.version.string ``` ``` ## [1] "R version 4.2.2 Patched (2022-11-10 r83330)" ```
Which means that, while there is output, it's no longer the output we expect and thus it breaks our tests 😢
The text was updated successfully, but these errors were encountered:
The good thing is that I now have a pathway to improve the progress bars if I wanted to
Sorry, something went wrong.
Successfully merging a pull request may close this issue.
Let this be a lesson to you kids about writing tests based on expectations of messages from other packages.
Consider this test:
sandpaper/tests/testthat/test-build_lesson.R
Lines 7 to 15 in 6211631
We expect knitr to produce output that says "ordinary text without R code" like this:
Well.... I should have really paid attention because this progress bar has been a thorn in Hadley's craw for a WHILE: yihui/knitr#1880
With {knitr} version 1.42, the printing is now completely changed:
Which means that, while there is output, it's no longer the output we expect and thus it breaks our tests 😢
The text was updated successfully, but these errors were encountered: