search

vocalpy / crowsetta

A tool to work with any format for annotating animal sounds
https://crowsetta.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
51 stars 3 forks source link

rewrite format tests to obey DRY #95

Closed NickleDave closed 2 years ago

NickleDave commented 3 years ago

looks like many of the tests for formats are essentially cut-and-pasted from one module to another

we could replace this with parametrized versions

not quite sure what's the best way -- parameter 'format' that is format name strings + "factory fixtures" that return appropriate data for each format when string is passed in?

NickleDave commented 2 years ago

Closing: tests are still format specific even after #161, but I think there will always be edge cases for specific formats we need to test, so this won't work