From duke at openjdk.org Fri Oct 11 16:46:18 2024 From: duke at openjdk.org (duke) Date: Fri, 11 Oct 2024 16:46:18 GMT Subject: Withdrawn: 7903767: Feature Tests - Adding five JavaTest GUI legacy automated test scripts In-Reply-To: References: Message-ID: On Fri, 5 Jul 2024 13:22:09 GMT, Golla Yadav 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. KFL1.java > 2. KFL2.java > 3. KFL3.java > 4. KFL4.java > 5. KFL5.java > > Updated 'product.name' in 'build/local.properties' file because it is being used by 'build.xml'. This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.org/jtharness/pull/75 From duke at openjdk.org Wed Oct 16 07:24:07 2024 From: duke at openjdk.org (Golla Yadav) Date: Wed, 16 Oct 2024 07:24:07 GMT Subject: RFR: 7903862: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Message-ID: <7MGMR7jQkujE0vIF5sf2NUyQmqf5S7f3zwQFsPKol60=.0861006b-441b-4ad7-a2be-d5b3b3c49756@github.com> Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. 1. TestTree6.java 2. TestTree7.java 3. TestTree8.java 4. TestTree9.java 5. TestTree10.java ------------- Commit messages: - 7903862: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Changes: https://git.openjdk.org/jtharness/pull/86/files Webrev: https://webrevs.openjdk.org/?repo=jtharness&pr=86&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903862 Stats: 462 lines in 5 files changed: 462 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jtharness/pull/86.diff Fetch: git fetch https://git.openjdk.org/jtharness.git pull/86/head:pull/86 PR: https://git.openjdk.org/jtharness/pull/86 From duke at openjdk.org Wed Oct 16 07:45:54 2024 From: duke at openjdk.org (Golla Yadav) Date: Wed, 16 Oct 2024 07:45:54 GMT Subject: RFR: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Message-ID: Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. 1. TestTree11.java 2. TestTree12.java 3. TestTree13.java 4. TestTree14.java 5. TT_SelectionCE.java 6. TT_SelectionCE1.java ------------- Commit messages: - 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Changes: https://git.openjdk.org/jtharness/pull/87/files Webrev: https://webrevs.openjdk.org/?repo=jtharness&pr=87&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903863 Stats: 538 lines in 6 files changed: 538 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jtharness/pull/87.diff Fetch: git fetch https://git.openjdk.org/jtharness.git pull/87/head:pull/87 PR: https://git.openjdk.org/jtharness/pull/87 From duke at openjdk.org Wed Oct 16 08:07:47 2024 From: duke at openjdk.org (Golla Yadav) Date: Wed, 16 Oct 2024 08:07:47 GMT Subject: RFR: 7903864: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Message-ID: Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. 1. TT_SelectionCE3.java 2. TT_SelectionCE4.java 3. TT_SelectionCE5.java 4. TT_SelectionCE6.java 5. TT_SelectionCE7.java ------------- Commit messages: - 7903864: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Changes: https://git.openjdk.org/jtharness/pull/88/files Webrev: https://webrevs.openjdk.org/?repo=jtharness&pr=88&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903864 Stats: 296 lines in 5 files changed: 296 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jtharness/pull/88.diff Fetch: git fetch https://git.openjdk.org/jtharness.git pull/88/head:pull/88 PR: https://git.openjdk.org/jtharness/pull/88 From duke at openjdk.org Wed Oct 16 08:16:46 2024 From: duke at openjdk.org (Golla Yadav) Date: Wed, 16 Oct 2024 08:16:46 GMT Subject: RFR: 7903865: Feature Tests - Adding two JavaTest GUI legacy automated test scripts Message-ID: Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. 1. TT_SelectionCE8.java 2. TT_SelectionCE9.java ------------- Commit messages: - 7903865: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Changes: https://git.openjdk.org/jtharness/pull/89/files Webrev: https://webrevs.openjdk.org/?repo=jtharness&pr=89&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903865 Stats: 115 lines in 2 files changed: 115 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jtharness/pull/89.diff Fetch: git fetch https://git.openjdk.org/jtharness.git pull/89/head:pull/89 PR: https://git.openjdk.org/jtharness/pull/89 From dbessono at openjdk.org Wed Oct 16 12:21:19 2024 From: dbessono at openjdk.org (Dmitry Bessonov) Date: Wed, 16 Oct 2024 12:21:19 GMT Subject: RFR: 7903767: Feature Tests - Adding five JavaTest GUI legacy automated test scripts [v3] In-Reply-To: References: Message-ID: On Fri, 13 Sep 2024 13:24:49 GMT, Golla Yadav 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. KFL1.java >> 2. KFL2.java >> 3. KFL3.java >> 4. KFL4.java >> 5. KFL5.java >> >> Updated 'product.name' in 'build/local.properties' file because it is being used by 'build.xml'. > > Golla Yadav has updated the pull request incrementally with two additional commits since the last revision: > > - Updated 7903767 > - Updated the 7903767 Marked as reviewed by dbessono (Reviewer). ------------- PR Review: https://git.openjdk.org/jtharness/pull/75#pullrequestreview-2372280924 From dbessono at openjdk.org Wed Oct 16 12:42:18 2024 From: dbessono at openjdk.org (Dmitry Bessonov) Date: Wed, 16 Oct 2024 12:42:18 GMT Subject: RFR: 7903864: Feature Tests - Adding five JavaTest GUI legacy automated test scripts In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 08:03:15 GMT, Golla Yadav wrote: > Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. > > 1. TT_SelectionCE3.java > 2. TT_SelectionCE4.java > 3. TT_SelectionCE5.java > 4. TT_SelectionCE6.java > 5. TT_SelectionCE7.java Thanks for adding more test coverage ------------- Marked as reviewed by dbessono (Reviewer). PR Review: https://git.openjdk.org/jtharness/pull/88#pullrequestreview-2372344574 From dbessono at openjdk.org Wed Oct 16 13:04:20 2024 From: dbessono at openjdk.org (Dmitry Bessonov) Date: Wed, 16 Oct 2024 13:04:20 GMT Subject: RFR: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 07:41:24 GMT, Golla Yadav wrote: > Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. > > 1. TestTree11.java > 2. TestTree12.java > 3. TestTree13.java > 4. TestTree14.java > 5. TT_SelectionCE.java > 6. TT_SelectionCE1.java gui-tests/src/gui/src/jthtest/TestTree/TT_SelectionCE.java line 70: > 68: > 69: // waiter 10 sec > 70: Waiter waiter = new Waiter() { Is there a waiting period of 10 seconds here ? ------------- PR Review Comment: https://git.openjdk.org/jtharness/pull/87#discussion_r1803062094 From duke at openjdk.org Wed Oct 16 14:38:55 2024 From: duke at openjdk.org (Golla Yadav) Date: Wed, 16 Oct 2024 14:38:55 GMT Subject: RFR: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts [v2] In-Reply-To: References: Message-ID: > Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. > > 1. TestTree11.java > 2. TestTree12.java > 3. TestTree13.java > 4. TestTree14.java > 5. TT_SelectionCE.java > 6. TT_SelectionCE1.java Golla Yadav has updated the pull request incrementally with one additional commit since the last revision: 7903863: Feature Tests - Updated the TT_SelectionCE as suggested ------------- Changes: - all: https://git.openjdk.org/jtharness/pull/87/files - new: https://git.openjdk.org/jtharness/pull/87/files/945ff766..0eed033a Webrevs: - full: https://webrevs.openjdk.org/?repo=jtharness&pr=87&range=01 - incr: https://webrevs.openjdk.org/?repo=jtharness&pr=87&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.org/jtharness/pull/87.diff Fetch: git fetch https://git.openjdk.org/jtharness.git pull/87/head:pull/87 PR: https://git.openjdk.org/jtharness/pull/87 From duke at openjdk.org Wed Oct 16 16:04:32 2024 From: duke at openjdk.org (Golla Yadav) Date: Wed, 16 Oct 2024 16:04:32 GMT Subject: RFR: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts [v3] In-Reply-To: References: Message-ID: > Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. > > 1. TestTree11.java > 2. TestTree12.java > 3. TestTree13.java > 4. TestTree14.java > 5. TT_SelectionCE.java > 6. TT_SelectionCE1.java Golla Yadav has updated the pull request incrementally with two additional commits since the last revision: - 7903863: Feature Tests - Updated the test script as suggested - 7903863: Feature Tests - Updated the test script as suggested ------------- Changes: - all: https://git.openjdk.org/jtharness/pull/87/files - new: https://git.openjdk.org/jtharness/pull/87/files/0eed033a..3b2c474a Webrevs: - full: https://webrevs.openjdk.org/?repo=jtharness&pr=87&range=02 - incr: https://webrevs.openjdk.org/?repo=jtharness&pr=87&range=01-02 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jtharness/pull/87.diff Fetch: git fetch https://git.openjdk.org/jtharness.git pull/87/head:pull/87 PR: https://git.openjdk.org/jtharness/pull/87 From duke at openjdk.org Wed Oct 16 16:07:30 2024 From: duke at openjdk.org (Golla Yadav) Date: Wed, 16 Oct 2024 16:07:30 GMT Subject: RFR: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts [v3] In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 13:01:35 GMT, Dmitry Bessonov wrote: > Is there a waiting period of 10 seconds here ? Yes, It is required because it waits for 10 sec by default. public Waiter() { this(true, 10000, 100); } ------------- PR Review Comment: https://git.openjdk.org/jtharness/pull/87#discussion_r1803411101 From dbessono at openjdk.org Wed Oct 16 16:50:20 2024 From: dbessono at openjdk.org (Dmitry Bessonov) Date: Wed, 16 Oct 2024 16:50:20 GMT Subject: RFR: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts [v3] In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 16:04:32 GMT, Golla Yadav wrote: >> Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. >> >> 1. TestTree11.java >> 2. TestTree12.java >> 3. TestTree13.java >> 4. TestTree14.java >> 5. TT_SelectionCE.java >> 6. TT_SelectionCE1.java > > Golla Yadav has updated the pull request incrementally with two additional commits since the last revision: > > - 7903863: Feature Tests - Updated the test script as suggested > - 7903863: Feature Tests - Updated the test script as suggested Marked as reviewed by dbessono (Reviewer). ------------- PR Review: https://git.openjdk.org/jtharness/pull/87#pullrequestreview-2373108094 From duke at openjdk.org Thu Oct 17 06:08:19 2024 From: duke at openjdk.org (Golla Yadav) Date: Thu, 17 Oct 2024 06:08:19 GMT Subject: RFR: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts [v3] In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 16:47:41 GMT, Dmitry Bessonov wrote: >> Golla Yadav has updated the pull request incrementally with two additional commits since the last revision: >> >> - 7903863: Feature Tests - Updated the test script as suggested >> - 7903863: Feature Tests - Updated the test script as suggested > > Marked as reviewed by dbessono (Reviewer). @dbessono Thank you for your time in reviewing the test scripts. ------------- PR Comment: https://git.openjdk.org/jtharness/pull/87#issuecomment-2418582582 From duke at openjdk.org Thu Oct 17 06:08:20 2024 From: duke at openjdk.org (duke) Date: Thu, 17 Oct 2024 06:08:20 GMT Subject: RFR: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts [v3] In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 16:04:32 GMT, Golla Yadav wrote: >> Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. >> >> 1. TestTree11.java >> 2. TestTree12.java >> 3. TestTree13.java >> 4. TestTree14.java >> 5. TT_SelectionCE.java >> 6. TT_SelectionCE1.java > > Golla Yadav has updated the pull request incrementally with two additional commits since the last revision: > > - 7903863: Feature Tests - Updated the test script as suggested > - 7903863: Feature Tests - Updated the test script as suggested @gollayadav Your change (at version 3b2c474aadd1c4460188eee2b8791da1b12041bb) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jtharness/pull/87#issuecomment-2418587046 From duke at openjdk.org Thu Oct 17 11:17:21 2024 From: duke at openjdk.org (Golla Yadav) Date: Thu, 17 Oct 2024 11:17:21 GMT Subject: Integrated: 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 07:41:24 GMT, Golla Yadav wrote: > Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. > > 1. TestTree11.java > 2. TestTree12.java > 3. TestTree13.java > 4. TestTree14.java > 5. TT_SelectionCE.java > 6. TT_SelectionCE1.java This pull request has now been integrated. Changeset: 75a80401 Author: Golla Yadav Committer: Dmitry Bessonov URL: https://git.openjdk.org/jtharness/commit/75a8040193549a73c7adb98f79137f70e5394b86 Stats: 538 lines in 6 files changed: 538 ins; 0 del; 0 mod 7903863: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Reviewed-by: dbessono ------------- PR: https://git.openjdk.org/jtharness/pull/87 From duke at openjdk.org Thu Oct 17 11:53:19 2024 From: duke at openjdk.org (Golla Yadav) Date: Thu, 17 Oct 2024 11:53:19 GMT Subject: RFR: 7903864: Feature Tests - Adding five JavaTest GUI legacy automated test scripts In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 12:39:33 GMT, Dmitry Bessonov wrote: > Thanks for adding more test coverage Thank you @dbessono for your time in reviewing the test scripts. ------------- PR Comment: https://git.openjdk.org/jtharness/pull/88#issuecomment-2419312499 From duke at openjdk.org Thu Oct 17 11:53:19 2024 From: duke at openjdk.org (duke) Date: Thu, 17 Oct 2024 11:53:19 GMT Subject: RFR: 7903864: Feature Tests - Adding five JavaTest GUI legacy automated test scripts In-Reply-To: References: Message-ID: <_cdgM0zwbO7EZ-SKMotJvKbBK4sfCc1eQrMTyI_1j4s=.daf0e76f-6501-41d0-af5e-b230aea9e433@github.com> On Wed, 16 Oct 2024 08:03:15 GMT, Golla Yadav wrote: > Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. > > 1. TT_SelectionCE3.java > 2. TT_SelectionCE4.java > 3. TT_SelectionCE5.java > 4. TT_SelectionCE6.java > 5. TT_SelectionCE7.java @gollayadav Your change (at version 53269557cd739d23a1cdc5cdfe673afd8049e893) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jtharness/pull/88#issuecomment-2419313310 From duke at openjdk.org Thu Oct 17 12:27:18 2024 From: duke at openjdk.org (Golla Yadav) Date: Thu, 17 Oct 2024 12:27:18 GMT Subject: Integrated: 7903864: Feature Tests - Adding five JavaTest GUI legacy automated test scripts In-Reply-To: References: Message-ID: On Wed, 16 Oct 2024 08:03:15 GMT, Golla Yadav wrote: > Adding below automated legacy JavaTest GUI feature Test Scripts to the Jemmy regression suite and tested locally on three platforms(Linux and Mac OS) and working fine. > > 1. TT_SelectionCE3.java > 2. TT_SelectionCE4.java > 3. TT_SelectionCE5.java > 4. TT_SelectionCE6.java > 5. TT_SelectionCE7.java This pull request has now been integrated. Changeset: 7e25220e Author: Golla Yadav Committer: Dmitry Bessonov URL: https://git.openjdk.org/jtharness/commit/7e25220ee3849443fffdb5eb5a77371a05ce71c4 Stats: 296 lines in 5 files changed: 296 ins; 0 del; 0 mod 7903864: Feature Tests - Adding five JavaTest GUI legacy automated test scripts Reviewed-by: dbessono ------------- PR: https://git.openjdk.org/jtharness/pull/88 From duke at openjdk.org Fri Oct 18 17:15:40 2024 From: duke at openjdk.org (Golla Yadav) Date: Fri, 18 Oct 2024 17:15:40 GMT Subject: RFR: 7903869: Feature Tests - Fixing TT_SelectionCE test script Message-ID: Base class "TT_SelectionCE" test script is executing, which is not required. So to avoid this adding below line in build.xml Also, added condition to run the specific tests. ------------- Commit messages: - 7903869: Fixing TT_SelectionCE test script Changes: https://git.openjdk.org/jtharness/pull/90/files Webrev: https://webrevs.openjdk.org/?repo=jtharness&pr=90&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903869 Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jtharness/pull/90.diff Fetch: git fetch https://git.openjdk.org/jtharness.git pull/90/head:pull/90 PR: https://git.openjdk.org/jtharness/pull/90 From duke at openjdk.org Fri Oct 18 17:35:50 2024 From: duke at openjdk.org (Golla Yadav) Date: Fri, 18 Oct 2024 17:35:50 GMT Subject: Withdrawn: 7903869: Feature Tests - Fixing TT_SelectionCE test script In-Reply-To: References: Message-ID: On Fri, 18 Oct 2024 17:07:16 GMT, Golla Yadav wrote: > Base class "TT_SelectionCE" test script is executing, which is not required. So to avoid this adding below line in build.xml > > > > Also, added condition to run the specific tests. This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.org/jtharness/pull/90 From duke at openjdk.org Mon Oct 28 14:47:04 2024 From: duke at openjdk.org (duke) Date: Mon, 28 Oct 2024 14:47:04 GMT Subject: Withdrawn: 7903812: Extract property name added with CODETOOLS-7900303 to a string constant In-Reply-To: References: Message-ID: On Mon, 2 Sep 2024 10:01:56 GMT, Dmitry Bessonov wrote: > A string constant can be used for the property "javatest.desktop.testrunprogressmonitor.hidden" introduced by [CODETOOLS-7900303](https://bugs.openjdk.org/browse/CODETOOLS-7900303) This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.org/jtharness/pull/83