RFR: 8327748: Convert javax/swing/JFileChooser/6798062/bug6798062.java applet test to main [v4]

Alexey Ivanov aivanov at openjdk.org
Tue Mar 12 11:43:14 UTC 2024


On Tue, 12 Mar 2024 11:39:46 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> I think it's needed to have some files in the directory..
>> This test is problemlisted actually and as per [this comment](https://bugs.openjdk.org/browse/JDK-8146446?focusedId=13884125&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13884125) it needs to be tested on a directory with some files in it but sometimes, in CI system temp folder may have thousand of files so I think we should revert back to "java.io.tmpdir" for this test sprint and figure out the fix for the tmpdir in JDK-8146446...Any objection?
>
> One way to handle it is to create a number of files in the current directory, the `scratch` directory that jtreg provides and then rely on jtreg to remove it. Or to create a temp directory in the current directory and create files there.
> 
> We have no control over how many files are in the temporary directory, however, in most cases it's not empty.

> I think we should revert back to "java.io.tmpdir" for this test sprint and figure out the fix for the tmpdir in JDK-8146446...Any objection?

Sounds reasonable.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/18180#discussion_r1521314423


More information about the client-libs-dev mailing list