search

Chemellia / ChemistryFeaturization.jl

Interface package for featurizing atomic structures
https://chemistryfeaturization.chemellia.org/dev/
MIT License
41 stars 14 forks source link

test: reorganize tests, and overall structure of test/ directory #56

Closed thazhemadam closed 3 years ago

thazhemadam commented 3 years ago

Restructure the test/ directory module-wise.

Split existing testsets into new, separate testsets if they contain functionally different, and discrete tests. This makes it a little more clear what each test does, reduces size of testsets, and would make it easier to detect and debug regressions.

Signed-off-by: Anant Thazhemadam anant.thazhemadam@gmail.com

codecov-commenter commented 3 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@465797f). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #56   +/-   ##
=======================================
  Coverage        ?   93.36%           
=======================================
  Files           ?        5           
  Lines           ?      407           
  Branches        ?        0           
=======================================
  Hits            ?      380           
  Misses          ?       27           
  Partials        ?        0

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 465797f...b34c380. Read the comment docs.

rkurchin commented 3 years ago

Merged manually, closing (I'm a little confused why GH doesn't realize that this happened but oh well)