remove() is documented to throw IllegalArgumentException

[gavinking]

The tests for remove contradict the Javadoc for remove().

If I'm understanding the code correctly, it looks like ReactiveExceptionConverter is converting the IllegalArgumentException to a HibernateException.

But I'm not sure why we even need ReactiveExceptionConverter. Why is the converter from ORM not good enough?

[DavideD]

ReactiveExceptionConverter was introduced 5 years go to solve this issue.

A lot has changed since then, I guess this fell behind. Thanks for noticing it. I can update the test and backport to the other branches tomorrow morning.

[DavideD]

(Also, I probably haven't checked the javadoc when I wrote the test and just assumed that the error was fine)