On Thu, 8 Oct 2026 14:44:36 GMT, Kevin Rushforth <[email protected]> wrote:

>> Andy Goryachev has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 14 commits:
>> 
>>  - Merge branch 'master' into 8388276.import.images
>>  - Merge branch 'master' into 8388276.import.images
>>  - spelling
>>  - Merge branch 'master' into 8388276.import.images
>>  - copied from swing fx utils
>>  - Merge branch 'master' into 8388276.import.images
>>  - images
>>  - lossless
>>  - Merge branch 'master' into 8388276.import.images
>>  - whitespace
>>  - ... and 4 more: https://git.openjdk.org/jfx/compare/eab83a4b...b4c734c2
>
> tests/system/src/test/java/test/jfx/incubator/scene/control/richtext/TestImageFormatHandler.java
>  line 61:
> 
>> 59: 
>> 60: // Tests ImageFormatHandler
>> 61: public class TestImageFormatHandler extends RobotTestBase {
> 
> You cannot use Robot for tests in this module. Either move this to system 
> tests, somewhere under test.robot, or rework the test to not need robot.

Oh, I misread where this test lives -- I missed that it is actually _is_ under 
`tests/system`.

In that case, the only thing that is needed if you want to continue extending 
from RobotTestBase is to move the test underneath `test.robot.*`

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2224#discussion_r4220598754

Reply via email to