Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS

2023-04-24 Thread Karthik P K
On Thu, 20 Apr 2023 19:21:35 GMT, Andy Goryachev wrote: >> Test was failing because `CountDownLatch` was not used to make sure that new >> date is selected and the delay present was not sufficient. >> If the current date is on the left edge of the date picker, this test would >> fail because of

Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS [v2]

2023-04-24 Thread Kevin Rushforth
On Mon, 24 Apr 2023 11:42:02 GMT, Karthik P K wrote: >> Test was failing because `CountDownLatch` was not used to make sure that new >> date is selected and the delay present was not sufficient. >> If the current date is on the left edge of the date picker, this test would >> fail because of th

Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS [v2]

2023-04-24 Thread Andy Goryachev
On Mon, 24 Apr 2023 11:42:02 GMT, Karthik P K wrote: >> Test was failing because `CountDownLatch` was not used to make sure that new >> date is selected and the delay present was not sufficient. >> If the current date is on the left edge of the date picker, this test would >> fail because of th

Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS

2023-04-24 Thread Karthik P K
On Thu, 20 Apr 2023 13:28:23 GMT, Karthik P K wrote: > Test was failing because `CountDownLatch` was not used to make sure that new > date is selected and the delay present was not sufficient. > If the current date is on the left edge of the date picker, this test would > fail because of the wa

Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS [v2]

2023-04-24 Thread Karthik P K
> Test was failing because `CountDownLatch` was not used to make sure that new > date is selected and the delay present was not sufficient. > If the current date is on the left edge of the date picker, this test would > fail because of the way new date was selected using mouse click. > > Added `

Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS

2023-04-20 Thread Karthik P K
On Thu, 20 Apr 2023 13:41:15 GMT, Kevin Rushforth wrote: > The change from using the mouse to using the keyboard might change the nature > of the test. Have you verified that it would still catch the original bug for > which the test was created? Yes it still catches the original bug. > I'd r

Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS

2023-04-20 Thread Andy Goryachev
On Thu, 20 Apr 2023 13:28:23 GMT, Karthik P K wrote: > Test was failing because `CountDownLatch` was not used to make sure that new > date is selected and the delay present was not sufficient. > If the current date is on the left edge of the date picker, this test would > fail because of the wa

Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS

2023-04-20 Thread Kevin Rushforth
On Thu, 20 Apr 2023 13:28:23 GMT, Karthik P K wrote: > Test was failing because `CountDownLatch` was not used to make sure that new > date is selected and the delay present was not sufficient. > If the current date is on the left edge of the date picker, this test would > fail because of the wa

Re: RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS

2023-04-20 Thread Kevin Rushforth
On Thu, 20 Apr 2023 13:28:23 GMT, Karthik P K wrote: > Test was failing because `CountDownLatch` was not used to make sure that new > date is selected and the delay present was not sufficient. > If the current date is on the left edge of the date picker, this test would > fail because of the wa

RFR: 8306424: [testbug] DatePickerUpdateOnAlertCloseTest fails on MacOS

2023-04-20 Thread Karthik P K
Test was failing because `CountDownLatch` was not used to make sure that new date is selected and the delay present was not sufficient. If the current date is on the left edge of the date picker, this test would fail because of the way new date was selected using mouse click. Added `CountDownLat