Eclipse pointed out an unused variable, which turned out to be a bad equals implementation. Patch to follow. (This is part of a split up of AVRO-152).
I think this is more right. This is test code (so it doesn't get separately tested), and didn't trigger any real failure, but no reason to keep bad equals() implementations around.
I just committed this. Thanks, Philip!