8328247: Remove redundant dir for tests converted from applet to main

[satyenme]

Backporting JDK-8328247: Remove redundant dir for tests converted from applet to main. Renames three javax swing tests, as per change accepted by Oracle.

Notably, this change also adapts JDK-8355333: Some Problem list entries point to non-existent / wrong files, to ensure that the renamed files are correctly excluded. I'm also removing the PrintToFileTest tests from the exclude list (since these only run on windows anyway). The ScrollPaneEventType.java test hasn't been backported yet (I'll follow up with another PR doing so and excluding as needed), so I've left that change out.

Ran GHA Sanity Checks, local Tier 1 and 2, and new tests directly. Patch is clean. Backporting for parity with Oracle.

---

Progress

• Change must be properly reviewed (1 review required, with at least 1 Reviewer)
• JDK-8328247 needs maintainer approval
• Change must not contain extraneous whitespace
• JDK-8355333 needs maintainer approval
• Commit message must refer to an issue
• JDK-8353958 needs maintainer approval

Issues

• JDK-8328247: Remove redundant dir for tests converted from applet to main (Bug - P4 - Approved)
• JDK-8355333: Some Problem list entries point to non-existent / wrong files (Bug - P4 - Requested)
• JDK-8353958: Open source several AWT ScrollPane tests - Batch 2 (Bug - P4 - Approved)

Reviewers

• Paul Hohensee (@phohensee - Reviewer) 🔄 Re-review required (review applies to e9690b0d)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk21u-dev.git pull/2260/head:pull/2260
$ git checkout pull/2260

Update a local copy of the PR:
$ git checkout pull/2260
$ git pull https://git.openjdk.org/jdk21u-dev.git pull/2260/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 2260

View PR using the GUI difftool:
$ git pr show -t 2260

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk21u-dev/pull/2260.diff

Using Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back ssubramaniam! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

[openjdk]

This backport pull request has now been updated with issue from the original commit.

[satyenme]

/issue 8355333

[openjdk]

@satyenme
Adding additional issue to issue list: 8355333: Some Problem list entries point to non-existent / wrong files.

[mlbridge]

Webrevs

• 01: Full (998fd772)
• 00: Full (e9690b0d)

[openjdk]

⚠️ @satyenme This change is now ready for you to apply for maintainer approval. This can be done directly in each associated issue or by using the /approval command.

[satyenme]

/approval request for backport of JDK-8328247: Remove redundant dir for tests converted from applet to main (and most of 8355333: Some Problem list entries point to non-existent / wrong files)

Motivation: With this change, we clean up the test structure and exclude list without changing any functionality (so minimal risk). Change will bring openjdk closer to matching oracle.

Risk: Low. Ran GHA Sanity Checks, local Tier 1 and 2 and renamed manual tests. Patch is clean. Although change has only been present in tip since April, 2025, change has been accepted by Oracle.

[openjdk]

@satyenme
8328247: The approval request has been created successfully.
8355333: The approval request has been created successfully.

[GoeLin]

Hi @satyenme
would it be possible to first backport
8353958: Open source several AWT ScrollPane tests - Batch 2
then
8328247: Remove redundant dir for tests converted from applet to main
and last
DK-8355333: Some Problem list entries point to non-existent / wrong files

I think the fact to have clean backports and a 1:1 matching to what happened in head
is more helpful than avoiding a short time (until 8355333 is pushed) in which the tests
are not problem properly listed.

[satyenme]

Sure, that makes sense - I can take a look at doing that

[satyenme]

/issue 8353958

[openjdk]

@satyenme Please do not rebase or force-push to an active PR as it invalidates existing review comments. Note for future reference, the bots always squash all changes into a single commit automatically as part of the integration. See OpenJDK Developers’ Guide for more information.

[openjdk]

@satyenme
Adding additional issue to issue list: 8353958: Open source several AWT ScrollPane tests - Batch 2.

[GoeLin]

Hi @satyenme, my ask was not to include that change here, but to do three different backports in the right order. Thanks.

[satyenme]

Got it, sorry for misunderstanding. Was initially hesitant to do that since both 8353958 and 8328247 will cause non-excluded (but expected) test failures without 8355333. I suppose if I integrate them at the same time though it shouldn't be an issue.

[satyenme]

New PRs:

• 8353958: 8353958: Open source several AWT ScrollPane tests - Batch 2 #2275
• 8328247: 8328247: Remove redundant dir for tests converted from applet to main #2276
• 8355333: 8355333: Some Problem list entries point to non-existent / wrong files #2277 (not marked as clean, but should be clean after I pull in 8353958 once that's merged)