<div dir="ltr">Hi Matt,<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 21, 2018 at 9:55 PM, Matt Benson <span dir="ltr"><<a href="mailto:mbenson@apache.org" target="_blank">mbenson@apache.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I would challenge<br>
BootstrapConfigurationWithVali<wbr>datedExecutableTypesContaining<wbr>NONETest<br>
as defying the specification. The current version says at<br>
<a href="http://beanvalidation.org/2.0/spec/#integration-general-executable" rel="noreferrer" target="_blank">http://beanvalidation.org/2.0/<wbr>spec/#integration-general-<wbr>executable</a> and<br>
in the code for ExecutableType that NONE is essentially ignored in the<br>
presence of other types (and effectively, always). I see nothing to<br>
dictate that NONE should be treated differently in the XML descriptor<br>
than elsewhere.<br></blockquote><div><br></div><div>Outch, this one is an old one :).<br><br></div><div>The spec is very clear about the behavior of <span class="gmail-tck-testable gmail-tck-needs-update"><code class="gmail-classname">@ValidateOnExecution</code></span>:<br><span class="gmail-tck-testable">"A list containing <code>NONE</code> and other types of executables is equivalent to a list containing the types of executables without <code>NONE</code>.</span>"<br><br></div><div>Same in the @ExecutableType javadoc:<br> /**<br> * None of the executables.<br> * <p><br> * Note that this option is equivalent to an empty list of executable types<br> * and is present to improve readability. If {@code NONE} and other types of executables<br> * are present in a list, {@code NONE} is ignored.<br> */<br></div><div><br></div><div>I can't find anything specific to the XML configuration so I suppose we should follow the same rule.<br></div><div><br></div><div>@Gunnar does old Gunnar have any recollection of what young Gunnar did back at the time?<br><br>-- <br></div><div>Guillaume<br></div></div></div></div>