RFR: 8303130: Document required Accessibility permissions on macOS

Alexey Ivanov aivanov at openjdk.org
Mon Feb 27 18:52:38 UTC 2023


On Mon, 27 Feb 2023 15:54:28 GMT, Dmitry Markov <dmarkov at openjdk.org> wrote:

> Added the section devoted to client UI test which use Robot API

doc/testing.md line 616:

> 614: system key shortcuts for various platforms are provided below.
> 615: 
> 616: ##### MacOS

Suggestion:

##### macOS

Shall we spell _macOS_ with lower-case ‘m’ like Apple does?

doc/testing.md line 648:

> 646: the default operating system settings need to be adjusted for correct work of
> 647: Robot functionality. The detailed steps how to access and update that settings
> 648: for different platforms are provided below.

Suggestion:

Most automated Client UI tests use `Robot` API to control the UI. Usually,
the default operating system settings need to be adjusted for Robot
to work correctly. The detailed steps how to access and update these settings
for different platforms are provided below.

doc/testing.md line 656:

> 654: Privacy & Security; select Accessibility and ensure the following apps are
> 655: allowed to control your computer: `Java` and `Terminal`. If the tests are run
> 656: from an IDE, the IDE should be granted this permission too.

Suggestion:

`Robot` is not permitted to control your Mac by default since
macOS 10.15. To enable it, choose Apple menu > System Settings, click
Privacy & Security, then click Accessibility and ensure the following apps are
allowed to control your computer: *Java* and *Terminal*. If the tests are run
from an IDE, the IDE should be granted this permission too.

I amended the instructions align with [Allow accessibility apps to access your Mac](https://support.apple.com/en-gb/guide/mac-help/mh43185/mac).

Perhaps, “to allow it” is better than “to enable it”. Also unsure about “`Robot` is not *allowed* to control…” instead of “permitted”.

*Java* and *Terminal* are the names of apps, not part of a command-line, therefore they're shouldn't be rendered in monospace font; using italics suits better to separate the names from regular text (even though it's not required).

doc/testing.md line 664:

> 662: select Registry Editor; navigate to the following key:
> 663: `HKEY_CURRENT_USER\Control Panel\Desktop\ForegroundLockTimeout`; make sure
> 664: its value is set to 0.

Suggestion:

On Windows there is a delay in focus transfer. Usually it causes automated
UI test failure. To disable the delay, type `regedit` in the Search and then
select Registry Editor; navigate to the following key:
`HKEY_CURRENT_USER\Control Panel\Desktop`; make sure
the `ForegroundLockTimeout` value is set to 0.

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

PR: https://git.openjdk.org/jdk/pull/12772



More information about the build-dev mailing list