sciencehistory / chf-sufia

sufia-based hydra app
Other
9 stars 4 forks source link

Ingest audio derivatives; correct derivatives are created on ingest. #1236

Closed eddierubeiz closed 5 years ago

jrochkind commented 5 years ago

I'm not a huge fan of the architectural choices (inconsistency with how audio is handled compared to other derivs), but fitting into our technical debt in this app is challenging, and we don't want to spend lots of time fixing technical debt in this app we're trying to leave, so I'll say thumbs up anyway.

We should figure out how to test on travis one way or another, either get ffmpeg installed, or add additional tests that just test the ffmpeg command lines are as expected, maybe with a mock that returns a mock file as output.