RFR: 7903118: Sanity Tests - Adding four JavaTest GUI newly automated sanity test scripts [v5]

gollayadav duke at openjdk.java.net
Thu Mar 24 21:27:43 UTC 2022


On Fri, 11 Mar 2022 01:35:34 GMT, Dmitry Bessonov <dbessono at openjdk.org> wrote:

>> gollayadav has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   7903118: Sanity Tests - Corrected the trailing whitespace errors in two files
>
> gui-tests/src/gui/src/jthtest/Sanity_Tests/Test_Config_Edit3.java line 65:
> 
>> 63:      * @throws NoSuchMethodException     Occurs when a method is called that exists
>> 64:      *                                   at compile-time, but does not exist at
>> 65:      *                                   runtime.
> 
> What is the idea of copying here a very general description of each exception? This comment seems applicable to all the tests in the PR.

Thanks for reviewing. I think this general description is not required for each exception and I have removed this for all the tests in PR.

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

PR: https://git.openjdk.java.net/jtharness/pull/24


More information about the jtharness-dev mailing list