fix(test): added missing h5py context managers - Githubissues

dattalab / moseq2-model

Python scripts for linking kinect-extraction to pybasicbayes

Other

1 stars 2 forks source link

fix(test): added missing h5py context managers #73

Closed aymanzay closed 3 years ago

aymanzay commented 3 years ago

Issue being fixed or feature implemented

test_dict_to_h5() was opening h5py files without using a context manager.

How Has This Been Tested?

Locally and Travis

What Was Done?

Replaced vanilla h5 file open with context manager

Breaking Changes

None

Checklists

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation