fix: cleanup temporary files

[LDeakin]

tempfile.mktemp() does not clean up after itself, and I could see my /tmp growing as I ran a few tests

[flying-sheep]

For pytest, you want to use the fixtures for this.

I’ll help

[flying-sheep]

OK, here we go. Ended up being a major restructuring, since to do this right, we need to override the format and dimensionality fixtures, which means moving the versions of them used in the non-roundtrip tests into conftest. That’ll cause the arr fixture to pick these values from the overridden fixtures, resulting in the exact same set of tests as before.

But now all the scoping is done by pytest, so it cleans up, and the arrays aren’t created in pytests’ collection phase anymore (should speed that up by a lot and therefore help with IDE performance as well)

[ilan-gold]

Awesome paradigm to override, I have found myself in this tricky situation of param dependencies before

[flying-sheep]

Yeah, that’s how pytest is intended to work.

Isaac always wanted flat test file directories (I don’t think I’ve ever heard a reason) so we don’t use it much.

[d-v-b]

would in-memory storage also work here?

[ilan-gold]

Don't think so since this package doesn't support that kind of store crossing into rust