RFR: 8337851: Some tests have name which confuse jtreg [v2]
Alexey Ivanov
aivanov at openjdk.org
Fri Aug 16 11:42:48 UTC 2024
On Fri, 16 Aug 2024 01:01:52 GMT, toshiogata <duke at openjdk.org> wrote:
>> test/langtools/tools/javac/generics/parametricException/ParametricException.java line 1:
>>
>>> 1: /*
>>
>> since you're renaming this file for jtreg testing anyway could you move the jtreg tags down below the imports and expand the wildcard import?
>
> As you mentioned, it's a good time to make a minor improvement, so I modified the test.
I propose not to edit the renamed files.
First of all, the purpose of this PR is to *rename tests* to workaround a _misfeature_ of jtreg.
Git does not support renames natively, therefore by editing the renamed files you could break Git ability to detect renaming.
Additionally, these tests are not part of client and I am not aware of the practices in Java Compiler team. Expanding wildcard imports should be fine, yet I would rather avoid editing the renamed files as I mentioned above.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20475#discussion_r1719731264
More information about the compiler-dev
mailing list