ropensci / taxlist

Handling taxonomic lists
https://docs.ropensci.org/taxlist/
12 stars 4 forks source link

Fix file paths in a couple tests #5

Closed levisc8 closed 4 years ago

levisc8 commented 4 years ago

As discussed in ropensci/software-review#233, fixes file paths in a few unit tests. I ran devtools::test(), devtools::check(), and devtools::check_built() (from the 0_check.R script) and these all passed w/ no errors, warnings or notes.

Let me know if you have any issues w/ this - I don't think using Eclipse will cause any issues w/ the new code, but I also haven't attempted to test it with a different IDE.

codecov[bot] commented 4 years ago

Codecov Report

Merging #5 into master will increase coverage by 0.09%. The diff coverage is 90.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   94.64%   94.74%   +0.09%     
==========================================
  Files          28       27       -1     
  Lines        1102     1160      +58     
==========================================
+ Hits         1043     1099      +56     
- Misses         59       61       +2     
Impacted Files Coverage Δ
R/levels.R 100.00% <ø> (ø)
R/summary.R 89.90% <83.33%> (-1.27%) :arrow_down:
R/Extract.R 100.00% <100.00%> (ø)
R/subset.R 100.00% <0.00%> (ø)
R/replace_x.R 100.00% <0.00%> (ø)
R/merge_taxa.R 100.00% <0.00%> (ø)
R/get_children.R 100.00% <0.00%> (ø)
R/taxon_relations.R 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2cc4c20...2440227. Read the comment docs.

kamapu commented 4 years ago

Issues with devtools::test() are solved. Thanks!