Skip to content

Commit

Permalink
Test against cli, not cliapp
Browse files Browse the repository at this point in the history
  • Loading branch information
gaborcsardi committed Oct 7, 2020
1 parent 29e6277 commit 5cf65b9
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ Imports:
R6,
utils
Suggests:
cliapp,
cli,
covr,
crayon,
fansi,
Expand Down
1 change: 0 additions & 1 deletion inst/WORDLIST
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
cli
cliapp
cloneable
CMD
Eval
Expand Down
4 changes: 2 additions & 2 deletions tests/testthat/test-messages.R
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
context("messages")

test_that("messages in callr::r do not crash session", {
ret <- r(function() { cliapp::cli_text("fooobar"); 1 + 1 })
ret <- r(function() { cli::cli_text("fooobar"); 1 + 1 })
expect_identical(ret, 2)
gc()
})
Expand All @@ -11,7 +11,7 @@ test_that("messages in callr::r_bg do not crash session", {
skip_in_covr() # TODO: what wrong with this on Windows?
skip_on_cran()

rx <- r_bg(function() { cliapp::cli_text("fooobar"); 1 + 1 })
rx <- r_bg(function() { cli::cli_text("fooobar"); 1 + 1 })
rx$wait(5000)
rx$kill()
expect_equal(rx$get_exit_status(), 0)
Expand Down

0 comments on commit 5cf65b9

Please sign in to comment.