golem icon indicating copy to clipboard operation
golem copied to clipboard

More tests for better code coverage

Open ColinFay opened this issue 5 years ago • 3 comments

> Sys.time()
[1] "2019-11-28 21:42:52 CET"
> covr::package_coverage()
golem Coverage: 70.97%
R/addins.R: 0.00%
R/get_sysreqs.R: 0.00%
R/use_files.R: 0.00%
R/with_opt.R: 0.00%
R/utils.R: 38.67%
R/js.R: 46.67%
R/config.R: 51.47%
R/add_ressource_path.R: 60.00%
R/reload.R: 62.86%
R/add_deploy_helpers.R: 74.64%
R/create_golem.R: 78.85%
R/add_files.R: 82.78%
R/use_favicon.R: 83.78%
R/add_r_files.R: 86.05%
R/add_modules.R: 86.11%
R/make_dev.R: 90.00%
R/use_utils.R: 96.00%
R/options.R: 99.31%
R/browser_button.R: 100.00%
R/desc.R: 100.00%
R/test_helpers.R: 100.00%
R/use_recommended.R: 100.00%

We might be missing some corner cases in the file manipulation, would be nice to cover these.

See also :

https://codecov.io/gh/ThinkR-open/golem

ColinFay avatar Nov 28 '19 20:11 ColinFay

Current state of affairs:

> Sys.time()
[1] "2020-09-05 00:21:02 CEST"
> covr::package_coverage()
golem Coverage: 65.90%
R/addins.R: 0.00%
R/bundle_resources.R: 0.00%
R/get_sysreqs.R: 0.00%
R/gobals.R: 0.00%
R/run_dev.R: 0.00%
R/use_files.R: 0.00%
R/with_opt.R: 0.00%
R/yesno.R: 0.00%
R/js.R: 43.75%
R/utils.R: 51.75%
R/config.R: 57.69%
R/test_helpers.R: 63.89%
R/add_dockerfiles.R: 70.47%
R/reload.R: 71.74%
R/create_golem.R: 74.71%
R/use_recommended.R: 80.65%
R/use_favicon.R: 85.39%
R/add_ressource_path.R: 85.71%
R/add_files.R: 86.07%
R/add_modules.R: 87.69%
R/add_r_files.R: 88.57%
R/add_rstudio_files.R: 89.74%
R/make_dev.R: 90.00%
R/use_utils.R: 92.59%
R/options.R: 98.82%
R/browser_button.R: 100.00%
R/desc.R: 100.00%
R/is_running.R: 100.00%

ColinFay avatar Sep 04 '20 22:09 ColinFay

> Sys.time()
[1] "2021-10-01 12:43:59 CEST"
> covr::package_coverage()
golem Coverage: 58.50%
R/addins.R: 0.00%
R/bundle_resources.R: 0.00%
R/enable_roxygenize.R: 0.00%
R/get_sysreqs.R: 0.00%
R/gobals.R: 0.00%
R/run_dev.R: 0.00%
R/sanity_check.R: 0.00%
R/use_files.R: 0.00%
R/with_opt.R: 0.00%
R/test_helpers.R: 28.12%
R/utils.R: 39.78%
R/modules_fn.R: 43.12%
R/js.R: 43.75%
R/config.R: 57.69%
R/use_recommended.R: 63.33%
R/create_golem.R: 65.49%
R/reload.R: 68.13%
R/add_files.R: 77.54%
R/use_favicon.R: 85.56%
R/add_dockerfiles.R: 87.41%
R/desc.R: 87.50%
R/add_r_files.R: 88.33%
R/add_resource_path.R: 88.89%
R/make_dev.R: 90.00%
R/use_utils.R: 92.31%
R/add_rstudio_files.R: 92.59%
R/options.R: 98.85%
R/browser_button.R: 100.00%
R/disable_autoload.R: 100.00%
R/is_running.R: 100.00%
R/templates.R: 100.00%

ColinFay avatar Oct 01 '21 10:10 ColinFay

Test files should also be renamed for consistency between R & tests

ColinFay avatar Oct 01 '21 10:10 ColinFay