Add integration testing loading real files

scipp / scippnexus

h5py-like utility for NeXus files with seamless scipp integration

https://scipp.github.io/scippnexus/

BSD 3-Clause "New" or "Revised" License

3 stars 3 forks source link

Add integration testing loading real files #131

Open SimonHeybrock opened 2 years ago

SimonHeybrock commented 2 years ago

~Load with both scn.load and scn.load_nexus.~ scippnexus.v2
Existing files (various facilities).
Files generated by McStas (+Mantid)
Can we use Nexus constructor? https://github.com/ess-dmsc/nexus-constructor
Can this run on GitHub? Or too expensive / too much data?

SimonHeybrock commented 2 years ago

One way to verify results could be to save a "reference" using scipp's HDF5 I/O. If we add more features to the loader we may need to update it, but at least it should catch regressions (e.g., failing to load logs that were previously loaded).

SimonHeybrock commented 1 year ago

Note that test that load files without verification have recently been added under pytest -m externalfile.

~Blocked by scipp/scipp#3056.~

SimonHeybrock commented 1 year ago

I have begun adding some manual tests in #133. The approach could be extended.