<Swing Dev> RFR: 8272232: javax/swing/JTable/4275046/bug4275046.java failed with "Expected value in the cell: 'rededited' but found 'redEDITED'."

[Phil Race]

On Wed, 11 Aug 2021 09:03:57 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

> The test fails in CI testing citing expected "rededited" but found "redEDITED", which seems to point to fact that either there is a CAPS_LOCK key switched on or some other test did not use SHIFT key corectly  ie, pressed but not released.
> 
> Considering many more tests would have failed if CAPS_LOCK is turned on, I tried to find if some tests uses SHIFT key mistakenly and it seems JRadioButton test has press/release order wrong and it uses SHIFT. Rectified that.
> 
> In addition, corrected some known CI issues ie waiting after frame is made visible, made frame to centre. Along with that, also modified testcase to check lowercase string. Since the original JDK-4275046 issue is about newly entered text is made part of cell or not, it does not matter if it's in lowercase/uppercase and if such CAPS_LOCK or Shift key problem happens again, this test will not be affected.
> 
> CI run for several iterations running these 2 tests one after another in all platforms is green.

Someone presses the buttons while the test is executed
    A few tests are executed in parallel(bug in jtreg or mach5?)
    Bug in KVM which generates this button press.
    All of them are more important than the current cleanup, and I suggest investigating those possibilities on the lab_support+client_team chats

I don't believe any of the scenarios.


As you well know these headful tests will be re-run 2 more times if they fail.

It seems improbable some one was sitting there waiting to jump on the keyboard and interfere

with the test at the exact moment 3 different times.

And the same for a KVM error.

As for in parallel we'd have hundreds of failures in such a case and we do not see that

And looking at the test log for when this failed 4 tests in total failed and they all have similar problems

===

java\awt\SplashScreen\MultiResolutionSplash\MultiResolutionSplashTest.java

Execution failed: `main' threw exception: java.lang.RuntimeException: Focus is lost! Expected 'ab' got 'AB'.

---

java/awt/Toolkit/RealSync/Test.java

Test failed: testType
[2021-08-10T04:06:59,103Z] Cause: Expected 'a' got 'A'.

----

javax/swing/JFileChooser/8041694/bug8041694.java

java.lang.RuntimeException: The selected directory name is not the expected 'd ' but 'D '.

======

Moreover this test isn't even the first that failed - that was MultiResolutionSplashTest - so the problem was
already there

So some test left SHIFT pressed. I can imagine a timing problem where a test was exited before it managed to release.

I am curious how Prasanta decided test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java might be the problem

It was executed long after these tests were already failing.

So the root cause test is not yet identified.

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

PR: https://git.openjdk.java.net/jdk/pull/5079