Concatenate error messages for the same path

[msmych]

When there are multiple validation errors for the same path, the response error messages get overrided

This PR fixes that by concatenating error messages for the same path

[msmych]

Don't know how to check that error was raised and verify error details in one run 🤔

[sunu]

You can store the context in a variable if you want. Something like this should work:

 with self.assertRaises(BadRequest) as ctx:
            validate(data, schema)

self.assertEqual(ctx.exception.response.get_json().get("errors"), {"name": "'Bob' is too short", "password": "'1234' is too short"})

[sunu]

Here's the relevant section in the Python docs for reference: https://docs.python.org/3/library/unittest.html#:~:text=The%20context%20manager%20will%20store%20the%20caught%20exception%20object%20in%20its%20exception%20attribute.

[msmych]

@sunu thanks, that worked!

[Rosencrantz]

Thanks for raising this pull request @msmych we are taking a look now and we'll get back to you with any feedback. Needless to say we really appreciate you taking the time to work on this.

[sunu]

Hi @msmych, apart from the issue you've mentioned about the tests, this looks great! We can merge this in once you remove the duplicate calls in the tests.

[sunu]

You can store the context in a variable if you want. Something like this should work:

 with self.assertRaises(BadRequest) as ctx:
            validate(data, schema)

self.assertEqual(ctx.exception.response.get_json().get("errors"), {"name": "'Bob' is too short", "password": "'1234' is too short"})

[sunu]

Here's the relevant section in the Python docs for reference: https://docs.python.org/3/library/unittest.html#:~:text=The%20context%20manager%20will%20store%20the%20caught%20exception%20object%20in%20its%20exception%20attribute.

[msmych]

@sunu updated

[sunu]

Thank you @msmych!