|
Kevin Sutter I would prefer to keep the tests in to trigger such discussions. If we hadn't left it in, we would not be discussing here and Apache Bean Validation would not be aware of that case. If all BV impls can agree to treat Weld as a special exception and raise a meaningful exception for the normal cases i.e. user code. Everyone wins. I also suspect that not handing Weld proxies explicitly can lead to actual incoherent behavior of the Bean Validation implementation, so better be aware of that case (I did not think it through but from the HV code and how in handles Weld proxies, I think I am right).
If you really really really insist that Apache Bean Validation does not want to implement this user experience improvement, then I will disable the tests. But I think everyone would lose. In practice, there are only two implementations on the market, you and us, I hope we can come to a common ground.
|