Merge pull request #8 from gaborcsardi/fix/tryerrorintest

Get rid of the annoying try() error in the tests
jeroen · Mar 11, 2017 · ac7cd9a · ac7cd9a
2 parents c6941eb + 3ca4224
commit ac7cd9a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/tests/testthat/test-forking.R b/tests/testthat/test-forking.R
@@ -29,8 +29,8 @@ test_that("eval_fork gives errors", {
   expect_error(eval_fork(Sys.sleep(10), timeout = 2), "timeout")
 
   # Test that tryCatch works
-  expect_equal(eval_fork(try(pi)), pi)
-  expect_is(eval_fork(try(blabla())), "try-error")
+  expect_equal(eval_fork(try(pi, silent = TRUE)), pi)
+  expect_is(eval_fork(try(blabla(), silent = TRUE)), "try-error")
   expect_is(eval_fork(tryCatch(blabla(), error = identity)), "simpleError")
 })