Remove "original_header" from write_vcf and improve test coverage

The logic around writing headers is tricky at the moment, and it's mainly because of the lingering usage of the "original_header" if it exists. As we're removing this from the spec, we should stop using it.

We're also currently missing some test coverage around corner cases with type numbers etc, which would be good to round out here.