RFR: 8283335 : Add exists and readAttributesIfExists methods to FileSystemProvider [v6]

Lance Andersen lancea at openjdk.org
Tue Jul 5 09:57:47 UTC 2022


On Mon, 4 Jul 2022 19:30:50 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> The updated TestDelegation test is looking a bit better now but I think it would be simplified a lot more by getting rid of the data providers, just aren't needed in this test.

Unless you feel this is a must, I would prefer to keep the DataProviders.  The benefit I see is the test code does not need to be duplicated per parameter, each test scenario can be run as an individual test so that you do not need extra plumbing to run each test scenario in the unlikely event of a failure.

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

PR: https://git.openjdk.org/jdk/pull/9249


More information about the core-libs-dev mailing list