btw - I'm not proposing we actually change toString, I'm just saying don't include it in the test. It's inclusion here implies it's part of the contract of a BlockId, which I'm saying it shouldn't be.