search

almartin82 / mapvizieR

visualizations and reports for the NWEA MAP assessment in R
Other
17 stars 6 forks source link

improved test coverage on new historic recap reports #231

Closed almartin82 closed 8 years ago

almartin82 commented 8 years ago

hey @chrishaid - there are documentation improvements and better test coverage here. nothing new, just cleanup. for your review!

almartin82 commented 8 years ago

per codecov this improves coverage to ~ 97%

codecov-io commented 8 years ago

Current coverage is 96.49%

Merging #231 into master will increase coverage by +5.14% as of 6ef41b6

@@            master    #231   diff @@
======================================
  Files           51      51       
  Stmts         6259    6217    -42
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit           5718    5999   +281
  Partial          0       0       
+ Missed         541     218   -323

Review entire Coverage Diff as of 6ef41b6

Powered by Codecov. Updated on successful CI builds.

almartin82 commented 8 years ago

@chrishaid tests passed! :dancers: