Updating tests

[zachmayer]

No description provided.

[coveralls]

Coverage decreased (-1.28%) to 10.94% when pulling c1f14eb on tests into 8233d15 on master.

[zachmayer]

Ok, so the new tests for model trimming are what's failing here. The weird thing is, they run fine locally, and R CMD CHECK passes. Are there any special packages I may need to install to run the trim tests.

[coveralls]

Coverage decreased (-1.28%) to 10.94% when pulling 4a00ae8 on tests into 8233d15 on master.

[zachmayer]

The error I'm getting is really weird:

* checking tests ...
  Running ‘testthat.R’ [12s/12s]
 ERROR
Running the tests in ‘tests/testthat.R’ failed.
Last 13 lines of output:
         trim = TRUE))
  6: train(x, y, weights = w, ...)
  7: train.default(x, y, weights = w, ...)
  8: checkInstall(models$library)
  9: stop()

  testthat results ================================================================
  OK: 89 SKIPPED: 0 FAILED: 2
  1. Error: bayesglm classification 
  2. Error: bayesglm regression 

  Error: testthat unit tests failed
  Execution halted
* checking for unstated dependencies in vignettes ... OK
* checking package vignettes in ‘inst/doc’ ... OK
* checking running R code from vignettes ...
   ‘caret.Rnw’ ... OK
 OK
* checking re-building of vignette outputs ... OK
* checking PDF version of manual ... OK
* DONE
Status: 1 ERROR, 1 WARNING, 1 NOTE
See
  ‘/home/travis/build/topepo/caret/pkg/caret/caret.Rcheck/00check.log’
for details.

[zachmayer]

I might have figured it out— explicitly loading the package seems to work

[coveralls]

Coverage decreased (-1.28%) to 10.94% when pulling 8369f59 on tests into 8233d15 on master.

[coveralls]

Coverage decreased (-1.28%) to 10.94% when pulling 52fce53 on tests into 8233d15 on master.

[coveralls]

Coverage decreased (-1.28%) to 10.94% when pulling c652729 on tests into 8233d15 on master.

[coveralls]

Coverage decreased (-1.28%) to 10.94% when pulling c652729 on tests into 8233d15 on master.

[coveralls]

Coverage increased (+1.65%) to 13.88% when pulling c652729 on tests into 8233d15 on master.

[coveralls]

Coverage increased (+1.65%) to 13.88% when pulling 675215a on tests into 8233d15 on master.

[coveralls]

Coverage increased (+1.65%) to 13.88% when pulling 675215a on tests into 8233d15 on master.

[topepo]

Should I merge this in?

[zachmayer]

@topepo Let me squash it down to a single commit.

Are you ok with adding more packages to the "suggests" field? That's the only way I could get the trim tests to work.

[topepo]

On Tue, May 26, 2015 at 9:26 PM, Zach Mayer notifications@github.com
wrote:

@topepo https://github.com/topepo Let me squash it down to a single
commit.

Sounds good.

Are you ok with adding more packages to the "suggests" field? That's the
only way I could get the trim tests to work.

I'd like to avoid it if at all possible. Which ones?

[zachmayer]

kknn
arm
glmnet
rpart
C50,
ipred
plyr
earth

I'll figure out how to install them on travis manually (it should be easy, but it's not!)

[coveralls]

Coverage decreased (-1.28%) to 10.94% when pulling 598a68f on tests into 8233d15 on master.