add support for unit testing from building package (including at GitHub) #18

Merged
merged 3 commits into from Jan 17, 2014

Conversation

Projects
None yet
2 participants
Contributor

eddelbuettel commented Jan 6, 2014

Brian,
I am testing most (all?) of my R packages via Travis, and given that Travis relies on Ubuntu, I figured it would useful to have it test against an actual redis server. As I noticed that you already had RUnit tests, I just did the minimal additions to have these tickled -- if and only if an environment variable (currently: RunRRedisTests) is loaded with the right value (currently: yes). So CRAN builds would not get this, heck your own builds would not get it unless you set the variable.
But GitHub commits now trigger checks. I missed the need RUnit at first, but trial 2 succeeded.
Feel free to discard the Pull Request if you find it too invasive, and/or hit me up on Google Chat / email if you have general questions regarding Travis.
Cheers, Dirk

Owner

bwlewis commented Jan 6, 2014

Hi Dirk,

I'll review later this week. I think it will be fine. I like the idea of controlling tests with an environment variable.

bwlewis added a commit that referenced this pull request Jan 17, 2014

Merge pull request #18 from eddelbuettel/master
add support for unit testing from building package (including at GitHub)

@bwlewis bwlewis merged commit 7ba9e2d into bwlewis:master Jan 17, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment