RFR: 8348423: [TestBug] stress test Nodes initialization from a background thread [v8]

Andy Goryachev angorya at openjdk.org
Wed Feb 5 21:43:21 UTC 2025


On Wed, 5 Feb 2025 20:31:23 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Andy Goryachev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   more jitter
>
> tests/system/src/test/java/test/robot/javafx/scene/NodeInitializationStressTest.java line 486:
> 
>> 484:             return c;
>> 485:         }, (c) -> {
>> 486:             c.setPannable(nextBoolean());
> 
> Do you think it's worth adding some scrolling here? That might be a good follow-on test enhancement.

follow-up

> tests/system/src/test/java/test/robot/javafx/scene/NodeInitializationStressTest.java line 501:
> 
>> 499:         }, (c) -> {
>> 500:             c.setEditable(nextBoolean());
>> 501:             accessControl(c);
> 
> Is it worth adding code to increment / decrement the spinner?

follow-up

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1690#discussion_r1943697329
PR Review Comment: https://git.openjdk.org/jfx/pull/1690#discussion_r1943697628


More information about the openjfx-dev mailing list