Boost C++ Libraries: Ticket #4779: real test expected failure should be marked as such https://svn.boost.org/trac10/ticket/4779 <p> The qi/real.cpp test failure is described as expected in source-code comments, and indeed fails on every platform. </p> <pre class="wiki">..\libs\spirit\test\qi\real.cpp(421): test 'test_attr("123233.4124", float_, f) &amp;&amp; f == 123233.4140625f' failed in function 'int main()' 1 error detected. </pre><p> This failure reduces the value of the other tests in qi/real that do pass, as the whole test is marked failed. </p> <p> Further, this failure causes the entire spirit library to be marked as fail/yellow across every platform. </p> <p> I suggest splitting the expected failure into a separate test and marking it as an expected failure (see the regression test legend at the bottom of the page). This would better reflect the state of both qi/real and the entire spirit library. </p> <p> Also, shouldn't the local compare() function be used here instead of operator==()? </p> en-us Boost C++ Libraries /htdocs/site/boost.png https://svn.boost.org/trac10/ticket/4779 Trac 1.4.3 viboes Mon, 25 Oct 2010 11:18:26 GMT <link>https://svn.boost.org/trac10/ticket/4779#comment:1 </link> <guid isPermaLink="false">https://svn.boost.org/trac10/ticket/4779#comment:1</guid> <description> <p> I don't see this as a showstopper. Please could you set it as a Problem? </p> </description> <category>Ticket</category> </item> <item> <author>Jim Bell <jim@…></author> <pubDate>Mon, 25 Oct 2010 11:38:21 GMT</pubDate> <title>severity changed https://svn.boost.org/trac10/ticket/4779#comment:2 https://svn.boost.org/trac10/ticket/4779#comment:2 <ul> <li><strong>severity</strong> <span class="trac-field-old">Showstopper</span> → <span class="trac-field-new">Problem</span> </li> </ul> <p> Not a showstopper, but I'd like like the next release pass as many tests as possible. </p> Ticket Jim Bell <jim@…> Mon, 25 Oct 2010 13:49:49 GMT severity changed https://svn.boost.org/trac10/ticket/4779#comment:3 https://svn.boost.org/trac10/ticket/4779#comment:3 <ul> <li><strong>severity</strong> <span class="trac-field-old">Problem</span> → <span class="trac-field-new">Regression</span> </li> </ul> Ticket Hartmut Kaiser Wed, 27 Oct 2010 11:29:03 GMT <link>https://svn.boost.org/trac10/ticket/4779#comment:4 </link> <guid isPermaLink="false">https://svn.boost.org/trac10/ticket/4779#comment:4</guid> <description> <p> I rolled back the offending test on the release branch for now. This removes the yellow boxes there. We'll attend to that problem before the next release. </p> </description> <category>Ticket</category> </item> <item> <dc:creator>Joel de Guzman</dc:creator> <pubDate>Sun, 14 Mar 2021 19:47:57 GMT</pubDate> <title>status changed; resolution set https://svn.boost.org/trac10/ticket/4779#comment:5 https://svn.boost.org/trac10/ticket/4779#comment:5 <ul> <li><strong>status</strong> <span class="trac-field-old">new</span> → <span class="trac-field-new">closed</span> </li> <li><strong>resolution</strong> → <span class="trac-field-new">invalid</span> </li> </ul> <p> broken invalid test was removed </p> Ticket