apr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From <...@apache.org>
Subject Re: cvs commit: apr/test testfile.c
Date Thu, 02 Jan 2003 16:46:05 GMT

On 2 Jan 2003, Karl Fogel wrote:

> <rbb@apache.org> writes:
> > There are two ways to resolve this bug.  You can create a feature macro
> > APR_FILE_OPEN_DIRS, which means that apr_file_open will work on
> > directories, or you can make apr_file_open fail to open directories on
> > Unix.  In the meantime, leave the failure in the test suite.  It is
> > highlighting a problem.
>
> I'm not very familiar with the APR test framework, so pardon me if the
> answer to this is obvious: do we have an XFAIL mode available?  If so,
> we should be using it here.
>
> If a test failure reveals a bug, the bug should be recorded in the
> appropriate place (um, "the issue tracker"... :-) ).  If we then
> switch the test to "eXpected Fail", that reduces the noise level so
> that test won't interfere with our seeing other failures.  (And in the
> issue, just record that there is a test, and that it needs to be
> changed from XFAIL to PASS when the bug is fixed.)

We don't currently have an XFAIL mode, although it would be possible to
add one.  I will try to add that feature tonight.

Ryan


Mime
View raw message