RFR: 8265439: [TestBug] Enable and fix ignored unit tests in MenuItemTest

Kevin Rushforth kcr at openjdk.java.net
Mon Apr 19 13:58:40 UTC 2021


On Mon, 19 Apr 2021 12:51:06 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:

> This PR enables ignored unit tests from MenuItemTest and fixes them.
> 4 ignored tests are fixed. 
> 2 ignored tests are removed.
> 2 new tests are added.
> 
> **Before fix :**
> total tests = 89
> failures = 0
> ignored tests =  6
> 
> **After fix :**
> total tests = 89
> failures = 0
> ignored tests =  0

It looks like you concluded that the implementation is correct, and that the ignored tests were making bad assumptions. Can you add your evaluation to this PR explaining why the tests are incorrect?

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

PR: https://git.openjdk.java.net/jfx/pull/466


More information about the openjfx-dev mailing list