Code review comment for lp:~ev/canonical-identity-provider/error-messages

I agree with the outcome of the discussion, and wanted to add that tests were missing, so please consider adding the proper suite for the new approach :-)