RFR: 8328667: [Testbug] Enable ignored Spinner unit tests [v2]
Andy Goryachev
angorya at openjdk.org
Thu Mar 21 15:00:27 UTC 2024
On Thu, 21 Mar 2024 13:40:37 GMT, Karthik P K <kpk at openjdk.org> wrote:
>> Enabled a test in `SpinnerTest` class.
>> Added comment to the editing test, why it can not be enabled. When Spinner value is set using `getEditor` `setText` method, the listener added to the spinner is not invoked. There is no bug for this issue. I'm not sure about the expectation in this case.
>> Other set of tests which are ignored are `LocalTimeSpinnerValueFactory` tests. Since these tests are unstable and fail only at certain time of the day, not enabling this test as well.
>
> Karthik P K has updated the pull request incrementally with one additional commit since the last revision:
>
> Add bug id to ignore comment
modules/javafx.controls/src/test/java/test/javafx/scene/control/SpinnerTest.java line 341:
> 339: **************************************************************************/
> 340:
> 341: @Ignore("JDK-8328701")
the spinner is not editable and no commitValue() is issued (please see my comment in the ticket)
https://bugs.openjdk.org/browse/JDK-8328701?focusedId=14659352&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-14659352
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1414#discussion_r1534088412
More information about the openjfx-dev
mailing list