RFR: 7903177: Feature Tests - Adding five JavaTest GUI legacy automated test scripts [v2]
gollayadav
duke at openjdk.java.net
Fri May 20 12:12:39 UTC 2022
On Thu, 19 May 2022 21:23:24 GMT, Dmitry Bessonov <dbessono at openjdk.org> wrote:
>> gollayadav has updated the pull request incrementally with four additional commits since the last revision:
>>
>> - Update Config_LoadEdit3.java
>>
>> Updated/Modified the Config_LoadEdit3.java as per review comments
>> - Update Config_LoadEdit2.java
>>
>> Updated/Modified the Config_LoadEdit2.java as per review comments
>> - Update Config_LoadEdit1.java
>>
>> Updated/Modified the Config_LoadEdit1.java as per the review comments
>> - Updated the script as per the review.
>
> gui-tests/src/gui/src/jthtest/Config_LoadEdit/Config_LoadEdit.java line 37:
>
>> 35: public class Config_LoadEdit extends ConfigTools {
>> 36:
>> 37: }
>
> During the first iteration missed that this test is empty. It it really needed at all?
This empty test script is not required and removing it.
-------------
PR: https://git.openjdk.java.net/jtharness/pull/29
More information about the jtharness-dev
mailing list