Fix ciarrow_unittest +style issues

Wrapped lines in ciarrow_unittest.cpp to 80 columns Make use of helper functions for error checks where possible Expect rather than assert where possible Safety checks to prevent buffer overflows/out of bounds access

No longer expecting WEBVTT_ID_TRUNCATED error -- However, we probably do want some way of truncating cue IDs (should happen for IDs which are really long! But I don't see an actual limit in the html5 draft on ID lengths, so...)