I'd drop the StringBuilder...I'm not actually convinced it buys you anything (and if it does, it's in a failure case inside test code)