)]}'
{"keystone/common/validation/validators.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"496c2005a74c610b4f94db0b1ba0d056054b445d","unresolved":false,"context_lines":[{"line_number":52,"context_line":"                # fails validation."},{"line_number":53,"context_line":"                detail \u003d _(\"Invalid input for field \u0027%(path)s\u0027: \""},{"line_number":54,"context_line":"                           \"%(message)s\") % {\u0027path\u0027: ex.path.pop(),"},{"line_number":55,"context_line":"                                             \u0027message\u0027: ex.message}"},{"line_number":56,"context_line":"            else:"},{"line_number":57,"context_line":"                detail \u003d ex.message"},{"line_number":58,"context_line":"            raise exception.SchemaValidationError(detail\u003ddetail)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba0121b8_314cc29d","line":55,"updated":"2016-03-30 15:56:14.000000000","message":"We should add a test that confirms the message we set is passed along to the user.","commit_id":"815a9247758a79551cb106b09e2f5a7c1bd61cfc"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"b7b06748f31d5260252bbd9bca47d90e647157a9","unresolved":false,"context_lines":[{"line_number":52,"context_line":"                # fails validation."},{"line_number":53,"context_line":"                detail \u003d _(\"Invalid input for field \u0027%(path)s\u0027: \""},{"line_number":54,"context_line":"                           \"%(message)s\") % {\u0027path\u0027: ex.path.pop(),"},{"line_number":55,"context_line":"                                             \u0027message\u0027: ex.message}"},{"line_number":56,"context_line":"            else:"},{"line_number":57,"context_line":"                detail \u003d ex.message"},{"line_number":58,"context_line":"            raise exception.SchemaValidationError(detail\u003ddetail)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba0121b8_c53c9dca","line":55,"in_reply_to":"ba0121b8_314cc29d","updated":"2016-03-30 17:46:10.000000000","message":"ex.message is set by jsonschema, we don\u0027t have a lot of control over it.\n\nWe already test that SchemaValidationError is raised. Testing the value of the exception message would mean changing every assertRaises() in keystone/tests/unit/test_validation.py to assertRaisesRegexp(). Is that what you have in mind or is there a better way to test the message?","commit_id":"815a9247758a79551cb106b09e2f5a7c1bd61cfc"}]}
