catalystneuro / nwb-conversion-tools

Create NWB files by converting and combining neural data in proprietary formats and adding essential metadata.
https://nwb-conversion-tools.readthedocs.io/en/main/
BSD 3-Clause "New" or "Revised" License
25 stars 12 forks source link

[Bug] Only run DANDI test if key is present #513

Closed CodyCBakerPhD closed 2 years ago

CodyCBakerPhD commented 2 years ago

fix #497

codecov[bot] commented 2 years ago

Codecov Report

Merging #513 (9190b0b) into main (ca0e0de) will increase coverage by 1.31%. The diff coverage is n/a.

:exclamation: Current head 9190b0b differs from pull request most recent head 72175c9. Consider uploading reports for the commit 72175c9 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #513      +/-   ##
==========================================
+ Coverage   87.64%   88.96%   +1.31%     
==========================================
  Files          58       54       -4     
  Lines        2971     2682     -289     
==========================================
- Hits         2604     2386     -218     
+ Misses        367      296      -71     
Flag Coverage Δ
unittests 88.96% <ø> (+1.31%) :arrow_up:

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

Impacted Files Coverage Δ
src/nwb_conversion_tools/utils/json_schema.py 94.59% <0.00%> (-0.37%) :arrow_down:
src/nwb_conversion_tools/tools/__init__.py 100.00% <0.00%> (ø)
src/nwb_conversion_tools/utils/__init__.py 100.00% <0.00%> (ø)
...rc/nwb_conversion_tools/datainterfaces/__init__.py 100.00% <0.00%> (ø)
...rfaces/ecephys/neuralynx/neuralynxdatainterface.py 96.42% <0.00%> (ø)
src/nwb_conversion_tools/tools/neo/neo.py
...ols/datainterfaces/icephys/abf/abfdatainterface.py
src/nwb_conversion_tools/tools/neo/__init__.py
...ols/datainterfaces/icephys/baseicephysinterface.py
src/nwb_conversion_tools/tools/nwb_helpers.py 92.85% <0.00%> (+5.05%) :arrow_up: