[Read Helpers IIb] Add read_nwbfile with Pytest fixtures #388
Conversation
for more information, see https://pre-commit.ci
…outborders/nwbinspector into add_read_nwbfile_2
|
The CI has trouble untangling the HDMF versioning, but manual local dev installation of each package allows the tests to run Getting some actual legit errors now though, beginning debugging now |
for more information, see https://pre-commit.ci
|
Update: the However the other tests related to deletion are legitimately failing locally, so hdmf-dev/hdmf#882 might need some more investigation |
|
Notes from convo: *add tests for fsspec
|
…atawithoutborders/nwbinspector into add_read_nwbfile_fixtures
for more information, see https://pre-commit.ci
…atawithoutborders/nwbinspector into add_read_nwbfile_fixtures
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
…atawithoutborders/nwbinspector into add_read_nwbfile_fixtures
…atawithoutborders/nwbinspector into add_read_nwbfile_fixtures
|
@bendichter All tests green now |
| - name: Install HDMF-zarr # temporary | ||
| run: pip install hdmf-zarr | ||
| - name: Install latest HDMF # temporary | ||
| run: pip install -U hdmf |
There was a problem hiding this comment.
It looks like it might be better to switch to a requirements-tests.txt file
There was a problem hiding this comment.
This is temporary b/c of hdmf-zarr's dependencies - it's normally not needed at all for that workflow
There was a problem hiding this comment.
Also the goal here is not to make such things directly be base requirements of nwbinspector (instead relying on pynwb to handle it); this is only temporary stuff for running the tests that will be ported to PyNWB after thorough evaluation using the NWB Inspector
Analog of #386 but using pytest fixtures instead of
unittest.TestCase