SyncedVariantReader/Iterator

Also some variant_reader_test cleanup
I don't understand the first part of this comment.

For the second: it's true that I am testing more than necessary.  Is that a big concern?  The run time remains very fast.