Fixes a bug in the previous PR and also cleans up the existing tests. Previously, many tests would fail and it was difficult to determine whether the test was just out of date or whether the PR broke a new test.
Now, tests that currently fail are marked with xfail so that new PRs can see whether any existing passing tests fail.
Checklist:
[x] All tests pass on catscan: run pytest --basetemp=tmp -sv -n 8 tests on catscan from the root directory
[ ] If needed, docs have been update: docs/source has been updated for any added, moved, or removed files
[x] Docs build with no errors: run make clean & make html from the docs folder
[x] No python formatting errors: run flake8 nsds_lab_to_nwb tests from the root directory
Description and related issues
Fixes a bug in the previous PR and also cleans up the existing tests. Previously, many tests would fail and it was difficult to determine whether the test was just out of date or whether the PR broke a new test.
Now, tests that currently fail are marked with xfail so that new PRs can see whether any existing passing tests fail.
Checklist:
pytest --basetemp=tmp -sv -n 8 tests
on catscan from the root directorydocs/source
has been updated for any added, moved, or removed filesmake clean & make html
from thedocs
folderflake8 nsds_lab_to_nwb tests
from the root directory