[15] RFR(S) : 8249700 : java/io/File/GetXSpace.java should be added to exclude list, and not @ignore-d

Igor Ignatyev igor.ignatyev at oracle.com
Sun Jul 19 04:28:58 UTC 2020


http://cr.openjdk.java.net/~iignatyev//8249700/webrev.01/
> 9 lines changed: 1 ins; 1 del; 7 mod

Hi all,

could you please review this small patch which removes @ignore from java/io/File/GetXSpace.java and instead adds an entry to ProblemList.txt?

from JBS:
> java/io/File/GetXSpace.java is currently @ignore due to JDK-6492634 and JDK-6501010, as running this test isn't harmful and can still be useful to find other bugs, the test should be added to ProblemList.txt and @ignore tag should be removed.
> given JDK-6492634 is solaris-specific, and solaris port has been removed from jdk15, the test will be problem-listed only due to JDK-6501010 and only on windows.
from main issue(8249618):
> although ProblemList and @ignore achieve the same end result (test exclusion), their server different goals and have slightly different meanings, simplified @ignore should be used to exclude useless or harmful tests, and ProblemList in all other cases (see yet-not-integrated `ProblemListing or `@ignore`-ing a Test` section of dev guide, PR -- https://github.com/openjdk/guide/pull/21 for more details). 
> 
> due to different reasons, this hasn't been always followed and some currently @ignore-d tests should rather be ProblemList-ed, and some of ProblemList-ed should be @ignore-d, this issue is to clean up the current state in a hope that this will reduce further confusion. 

the patch also includes minimal changes to make the test runnable on macosx, which reveled that the test might fail on macos. 8249703 has been filed for that issue and the appropriate changes are done in ProblemList.txt.

webrev: http://cr.openjdk.java.net/~iignatyev//8249700/webrev.01/
JBS: https://bugs.openjdk.java.net/browse/JDK-8249700
testing:  java/io/File/GetXSpace.java on linux-x64 100 times, 100% passed; macos-x64 100 times, 71% passed

PS the test is still failing on windows albeit due to the reason other than in 6501010; it seems that the test should be just updated to recognize cygwin's `df` output. I'm currently working on the fix and will send it out for review shortly. 

Thanks,
-- Igor

JDK-6492634: https://bugs.openjdk.java.net/browse/JDK-8249700
JDK-6501010: https://bugs.openjdk.java.net/browse/JDK-6501010
JDK-6492634: https://bugs.openjdk.java.net/browse/JDK-6492634
JDK-8249703: https://bugs.openjdk.java.net/browse/JDK-8249703


More information about the core-libs-dev mailing list