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