I agree with the tests for PersonIdent.  But we also need these same tests for RawParseUtils.parsePersonIdent().  That constructor is used for RevCommit's getAuthor() method, which is probably what most display tools will be relying on.  And I'm reasonably certain we aren't testing for the multiple author case there, even if it does work.