Re: Change all XFAIL tests to FAIL
| From: | Pierre Joye | Date: | Fri, 30 Mar 2012 07:51:14 +0000 |
| Subject: | Re: Change all XFAIL tests to FAIL | ||
| References: | 1 2 3 4 5 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-59223@lists.php.net to get a copy of this message | ||
hi,
On Fri, Mar 30, 2012 at 8:02 AM, Dirk Haun <dirk@haun-online.de> wrote:
> What's the point of a test that fails (or XFAILs)? Either something is broken - then it
> should be fixed. Or the test makes no sense - then it should be removed.
See the archive about the reasons, it has been discussed to death many times.
To me, the main reason is:
A test fails but a fix is not easy or not possible right now. To keep
it in our radar, it is still added to our tests suite and listed as
"expected failure". This is very common practice.
That being said, as Stas wrote, feel free to fix them all (the bugs) :-)
Cheers,
--
Pierre
@pierrejoye | http://blog.thepimp.net | http://www.libgd.org