On Wed, 20 Sep 2023 21:29:21 GMT, Phil Race <[email protected]> wrote:
> How did this get through ? Was it not tested before pushing ? And if a test > needs "Robot" it surely expects a headful envt ? In fact I am going to say > that the problem here is using Robot at all. You don't need to add headful, > you just need to get rid of Robot which is just being used to do unnecessary > Thread.sleep() calls. I believe a review comment was suggested to change Thread.sleeps to robot delays after testing at some point. I did re-test what I thought were all the tests after moving them to open. @azvegint suggested the same change back to Thread.sleep. I have a current test ongoing for this change instead. Waiting for completion before updating the PR once more ------------- PR Comment: https://git.openjdk.org/jdk/pull/15847#issuecomment-1728454358
