more tests, and some #nocov tags

[eddelbuettel]

No description provided.

[codecov-io]

Current coverage is 59.76% (diff: 100%)

Merging #601 into master will increase coverage by 1.22%

@@             master       #601   diff @@
==========================================
  Files            76         74     -2   
  Lines          5075       4988    -87   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           2971       2981    +10   
+ Misses         2104       2007    -97   
  Partials          0          0          

Powered by Codecov. Last update 0c3d0b1...5620275

[eddelbuettel]

Anyone?

[thirdwing]

I have to say this is something I know really little, but LGTM.

[eddelbuettel]

I am not expert on the covr package either but slowly getting a feel for it via a few other (smaller!!) packages. It wraps very cleverly around calls to code and 'counts' -- for R by modifying the code, for compiled code via gcov. It helps some -- became aware of a regression on the weekend (in another and smaller package) where a line all of sudden was no longer covered when I changed code, and indeed, an argument was ignored when it should not have been...

[eddelbuettel]

Also: if you use Chrome, the plugin is neat as you then see the code at GitHub itself "coloured" green and red depending on whether it has coverage or not.

[eddelbuettel]

No further complaints to will merge.