On Tue, 19 Sep 2023 21:17:02 GMT, Brian Burkhalter <[email protected]> wrote:
>> test/jdk/java/io/File/TempDirDoesNotExist.java line 118:
>>
>>> 116: }
>>> 117:
>>> 118: public static Stream<Arguments> nonexistentProvider() {
>>
>> The `nonexistentProvider` seems like an odd name. The only provider here is
>> the junit provider (not a FileSystem provider)
>> How about `NoTempDir` and `TempDir` for the two argument streams.
>
> That definitely sounds more reasonable. Unfortunate nomenclature on my part.
I think maybe instead "tempDirSource' and "noTempDirSource."
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15757#discussion_r1330720489
