Your word on messaging code is canon to me. Merging with thanks!

Finding myself now wishing we had covered this with tests last year.

Oh you know what... the branch for the v2 API, with the condition if hasattr(result, 'group') actually can never be taken as far as I can see. That's because the attribute is actually result.groups and not result.group...

Moral of the story, let's cover all this stuff with tests :-)

The result.job attribute also hasn't existed since the commit introducing API v2. Seems like the create_fedmsg() function regressed in that commit (both copies of it). As did the ability to POST /api/v1.0/jobs which I also discovered when trying to write a test for it. Yikes...