RFR: 7903374: Feature Tests - Adding six JavaTest GUI legacy automated test scripts [v3]

[Dmitry Bessonov]

On Thu, 3 Nov 2022 18:15:54 GMT, gollayadav <duke at openjdk.org> wrote:

>> Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux, Windows, Mac OS) and working fine. Also verified with JDK8 on macos.
>> 
>> 1.CustomReport6.java
>> 2.CustomReport7.java
>> 3.ReportCreate2.java
>> 4.ReportCreate3.java
>> 5.Markers2.java
>> 6.Markers3.java
>
> gollayadav has updated the pull request incrementally with one additional commit since the last revision:
> 
>   7903374: Corrected the tab space errors

gui-tests/src/gui/src/jthtest/CustomReport/CustomReport7.java line 33:

> 31:  * text format.
> 32:  */
> 33: 

This line should be also better removed. The same applies to CustomReport6.

gui-tests/src/gui/src/jthtest/Markers/Markers.java line 44:

> 42: import org.netbeans.jemmy.operators.JPopupMenuOperator;
> 43: 
> 44: public class Markers extends ConfigTools {

This class seem to contain a collection of static methods. Why is it inherited?

gui-tests/src/gui/src/jthtest/Markers/Markers3.java line 31:

> 29: /**
> 30:  * This test case verifies that a marked question has a small icon to indicate that it has been marked.
> 31:  */

Please place this javadoc for the class correctly. This applied to other classes.

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

PR: https://git.openjdk.org/jtharness/pull/41