Skip to content

Commit

Permalink
Fixes \" issue in tests I hope!
Browse files Browse the repository at this point in the history 
The most ridiculous solution ever!!!! @dill but should work across
machines hopefully. Your previous ‘fix’ broke it on both my machines.
  • Loading branch information
@LHMarshall
LHMarshall committed Aug 10, 2016
1 parent ad7157b commit 4dcfdef
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/testthat/test_ds.R
View file
Expand Up @@ -27,11 +27,11 @@ test_that("Input errors are thrown correctly",{

# incorrect key definition?
expect_error(ds(egdata,4,key="bananas"),
"'arg' should be one of \"hn\", \"hr\", \"unif\"")
suppressMessages(message("'arg' should be one of \"hn\", \"hr\", \"unif\"")))

# incorrect adjustment definition?
expect_error(ds(egdata,4,key="hn",adjustment="bananas"),
"'arg' should be one of \"cos\", \"herm\", \"poly\"")
suppressMessages(message("'arg' should be one of \"cos\", \"herm\", \"poly\"")))

# first cutpoint not zero when no left truncation
expect_error(ds(egdata,4,cutpoints=c(2,3,4)),
Expand Down

2 comments on commit 4dcfdef

@dill
Copy link
Contributor

@dill dill commented on 4dcfdef Aug 10, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, this is super-weird. I have no idea what's going on though I have a suspicion that it is a Mac/RStudio thing... If this works then it works I guess... ¯_(ツ)_/¯

@LHMarshall
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I googled and print apparently doesn't recognise " but cat does, couldn't supress cat but message uses can and can supress. My machines (one mac, one windows, both using R studio) were comparing using print I guess.

Please sign in to comment.