<div dir="auto"><div style="font-family:sans-serif;font-size:12.8px" dir="auto"><div style="margin:16px 0px"><div dir="auto"><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">Okay, after digging a bit deeper I have come up with the following list of affected tests:</span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">org.hibernate.beanvalidation.tck.tests.metadata.PropertyDescriptorTest:</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetGroupConversionsForField</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetGroupConversionsForProperty</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetGroupConversionsReturnsEmptySet</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testIsCascaded</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testPropertyName</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-size:12pt"><u></u> <u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">org.hibernate.beanvalidation.tck.tests.metadata.ConstraintDescriptorTest:</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testAnnotationAndMapParametersReflectParameterOverriding</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testComposingConstraints</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testComposingConstraintsPayload</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetGroups</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetGroupsOnInterface</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetGroupsWithImplicitGroup</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetMessageTemplate</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetValidationAppliesTo</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetValidationAppliesToFromComposingConstraint</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetValidationAppliesToReturnsNull</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testPayload</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testReportAsSingleViolation</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-size:12pt"><u></u> <u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">org.hibernate.beanvalidation.tck.tests.metadata.BeanDescriptorTest:</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetConstraintsForUnConstrainedProperty</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testGetElementClassReturnsBeanClass</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testIsBeanConstrainedDueToConstraintOnInterface</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">#testIsBeanConstrainedDueToValidAnnotation</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-size:12pt"><u></u> <u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">These tests have in common the fact that they use bean classes derived from the org.hibernate.beanvalidation.tck.tests.metadata.Person interface, whose #getAge() method declares @CustomConstraint, which inherits @Size as a composing constraint via @CustomComposingConstraint.</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-size:12pt"><u></u> <u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">Section 3.3 of the spec says:</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">The type upon which composed constraint is placed must be compatible with all constraints (composing and composed).</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-size:12pt"><u></u> <u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">I therefore conclude that the constraint applied to Person#getAge() is incorrectly composed.</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-size:12pt"><u></u> <u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">Thanks,</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:arial,sans-serif">Matt</span><span style="font-size:12pt"><u></u><u></u></span></p><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><u></u> <u></u></p></div><u></u><p style="margin-right:0in;margin-left:0in;font-size:12pt;font-family:'times new roman',serif"><span style="font-family:'calibri'"><span style="color:rgb(128,128,128)"><span style="font-size:13px"><em><strong>CONFIDENTIALITY NOTICE:</strong> This e-mail, including any attachments, may contain confidential, privileged and/or proprietary information which is solely for the use of the intended recipient(s). Any review, use, disclosure or retention by others is strictly prohibited. If you are not an intended recipient, please contact the sender and delete this e-mail, any attachments and all copies.</em></span><br><span style="font-size:12.25px"><br>Permanent General Assurance Corporation | Permanent General Assurance Corporation of Ohio | The General Automobile Insurance Company, Inc. | Old American County Mutual Fire Insurance Company | Home Office: 2636 Elm Hill Pike, Nashville, TN 37214</span></span></span></p></div><div style="height:0px"></div></div><div style="font-family:sans-serif;font-size:12.8px;height:172px" dir="auto"></div><br></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Mar 23, 2018, 3:35 AM Guillaume Smet <<a href="mailto:guillaume.smet@gmail.com">guillaume.smet@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Matt,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 22, 2018 at 11:12 PM, Matt Benson <span dir="ltr"><<a href="mailto:mbenson@apache.org" target="_blank" rel="noreferrer">mbenson@apache.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div dir="auto">Yeah, this is weird. Elsewhere there is a test that specifically looks for there *not* being a Size validator for int. I will do some more research on this and report back tomorrow.</div></div></blockquote><div><br></div><div>FWIW, we don't have one in HV so the TCK cannot expect the impl to have one.<br><br>-- <br></div><div>Guillaume<br></div></div></div></div></div>
_______________________________________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org" target="_blank" rel="noreferrer">beanvalidation-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" rel="noreferrer noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a></blockquote></div>