PR #708 relaxed the requirement at the C++ level that the number of frames and the duration of the video or audio file must be present. We are doing this in eventual support of live streaming (see #695). The tests pass, and we checked that when we hack the C++ code to remove obtaining the values, we don't run into any segfaults. But, we don't have any tests that ensure the correct behavior with the values missing.
We should:
- Check in a small test video that has missing metadata for number of frames and duration.
- Use it for some of our ops tests - in particular, range queries in both indices and time. Tests that use the Python
VideoDeocder will always fail because we assert the presence of these values at the Python level.