Setup testthat

[dan-knight]

Description

Previously, package tests were contained in a single file under tests/test_plotting_functions.R. These tests were run simply by sourcing the file, which violates testing best practices with numerous obvious drawbacks. To conform to both R and Boutros Lab standards, the package should use testthat instead.

Checklist

• This PR does NOT contain PHI or germline genetic data. A repo may need to be deleted if such data is uploaded. Disclosing PHI is a major problem.

• This PR does NOT contain molecular files, compressed files, output files such as images (e.g. .png, .jpeg), .pdf, .RData, .xlsx, .doc, .ppt, or other non-plain-text files. To automatically exclude such files using a .gitignore file, see here for example.

• I have read the code review guidelines and the code review best practice on GitHub check-list.

• I have set up or verified the main branch protection rule following the github standards before opening this pull request.

• The name of the branch is meaningful and well formatted following the standards, using [AD_username (or 5 letters of AD if AD is too long)]-[brief_description_of_branch].

• I have added the major changes included in this pull request to the NEWS under the next release version or unreleased, and updated the date. I have also updated the version number in DESCRIPTION according to semantic versioning rules.

• Both R CMD build and R CMD check run successfully.

Testing Results

> devtools::test()
ℹ Testing BoutrosLab.plotting.general
✔ | F W S  OK | Context
✔ |         2 | plotting.functions                                                      

══ Results ═════════════════════════════════════════════════════════════════════════════
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 2 ]

[dan-knight]

Closing, as these changes were also included in #165