[icedtea-web] RFC: disable tests that are known to fail

[Omair Majid]

Hi,

IcedTea-Web contains a number of unit tests. Their usefulness is
lessened because some of the tests always fail and so many (most?)
developers will not actually run them. Even if developers do run the
tests, they may miss a regression or ignore it ("X other tests fail,
what's the big deal with one more failure"). I would like to have all
the unit tests pass. One way is to fix the underlying problem, of
course. The other is to disable the test temporarily.

The attached patch disables a few parser tests that we know are broken
and we wont be fixing soon. The disabled tests all check if the parser
can handle mixed content in xml files (the embedded nanoxml cant) and if
the parser can handle malformed xml files (pretty much all xml parsers
wont).

I would like to keep these tests around (because they do represent
actual bugs) but disable them to give developers greater incentive to
run all tests and always check that they pass.

Any thoughts or comments?

Thanks,
Omair
-------------- next part --------------
A non-text attachment was scrubbed...
Name: disable-broken-unit-tests-01.patch
Type: text/x-patch
Size: 3627 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20120321/567047b0/disable-broken-unit-tests-01.patch