<div dir="ltr">Hi,<div><br></div><div>Yes, I agree the test isn't correct and should be adjusted. I only can speculate why it is the way it is; perhaps we refined the semantics around NONE at some point during the 1.1 work and then didn't adjust that test. Working hours were a bit crazy back then ;)</div><div><br></div><div>I'm curious, though. Apache BVal is 1.1 certified, so you must have passed that one before. Are you re-writing parts of the code base in the course of implementing 2.0?</div><div><br></div><div>Could you also give us a heads-up of where you are in terms of passing the TCK? We'd like to release another TCK version with the changes you pointed out. But if there's the chance that more is coming, we'd wait a bit with that of course.</div><div><br></div><div>Cheers,</div><div><br></div><div>--Gunnar</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-03-22 10:17 GMT+01:00 Guillaume Smet <span dir="ltr"><<a href="mailto:guillaume.smet@gmail.com" target="_blank">guillaume.smet@gmail.com</a>></span>:<br><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 class="gmail_extra"><br><div class="gmail_quote"><span class="">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-exec<wbr>utable</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></span><div>Outch, this one is an old one :).<br><br></div><div>The spec is very clear about the behavior of <span class="m_8806068067157279355gmail-tck-testable m_8806068067157279355gmail-tck-needs-update"><code class="m_8806068067157279355gmail-classname">@ValidateOnExecution</code></span>:<br><span class="m_8806068067157279355gmail-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?<span class="HOEnZb"><font color="#888888"><br><br>-- <br></font></span></div><span class="HOEnZb"><font color="#888888"><div>Guillaume<br></div></font></span></div></div></div>
<br>______________________________<wbr>_________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.<wbr>jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/<wbr>mailman/listinfo/<wbr>beanvalidation-dev</a><br></blockquote></div><br></div>