alchemistry / alchemtest

the simple alchemistry test set
https://alchemtest.readthedocs.io
BSD 3-Clause "New" or "Revised" License
8 stars 12 forks source link

extended tests #53

Closed orbeckst closed 3 years ago

orbeckst commented 3 years ago
codecov[bot] commented 3 years ago

Codecov Report

Merging #53 (951c8ef) into master (1cb7c12) will increase coverage by 62.52%. The diff coverage is n/a.

:exclamation: Current head 951c8ef differs from pull request most recent head fd5ca13. Consider uploading reports for the commit fd5ca13 to get more accurate results Impacted file tree graph

@@             Coverage Diff             @@
##           master      #53       +/-   ##
===========================================
+ Coverage   25.75%   88.28%   +62.52%     
===========================================
  Files          10        9        -1     
  Lines         132      128        -4     
===========================================
+ Hits           34      113       +79     
+ Misses         98       15       -83     
Flag Coverage Δ
unittests 88.28% <ø> (+62.52%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/alchemtest/_version.py
src/alchemtest/amber/access.py 62.96% <0.00%> (+62.96%) :arrow_up:
src/alchemtest/gmx/access.py 100.00% <0.00%> (+68.62%) :arrow_up:
src/alchemtest/gomc/access.py 100.00% <0.00%> (+100.00%) :arrow_up:
src/alchemtest/namd/access.py 100.00% <0.00%> (+100.00%) :arrow_up:
src/alchemtest/gomc/__init__.py 100.00% <0.00%> (+100.00%) :arrow_up:
src/alchemtest/namd/__init__.py 100.00% <0.00%> (+100.00%) :arrow_up:
src/alchemtest/amber/__init__.py 100.00% <0.00%> (+100.00%) :arrow_up:

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 1cb7c12...fd5ca13. Read the comment docs.

orbeckst commented 3 years ago

not sure why tests are not running??

orbeckst commented 3 years ago

I rewrote the history so that it can me merged (no need to squash).

orbeckst commented 3 years ago

Thanks for the review, @xiki-tempula !