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...)
Merge pull request #342 from caitp/ciarrow_unittest

Fix ciarrow_unittest +style issues