8326564: Problem list HttpURLConnectionExpectContinueTest.java due to 8326503

[sendaoYan]

The testcase java/net/HttpURLConnection/HttpURLConnectionExpectContinueTest.java was newly added, it's backport from jdk23.

This testcase rely on jtreg-7, and it will fail run by jtreg-6: error: package org.junit.jupiter.api does not exist

If I use jtreg7 instead of jtreg6 to run all the testcases of jdk11u-dev, there are 424 testcases fail.
There are a total of 2 types failure:
1.javax/xml/jaxp testcases fail because of java.security.AccessControlException: access denied (“java.util.PropertyPermission” “testng.thread.affinity” “read”)
2. 5 tools/javac testcases fails, such as tools/javac/6257443/T6257443.java, the fail message is: Error. Parse Exception: Bad classname provided for clean’: foo.package-info`

So, the solution that use jtreg-7 instead of jtreg-6 is unacctable. Util we found a good solution to run this testcase success, this testcase should be excluded.

---

Progress

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

Issue

• JDK-8326564: Problem list HttpURLConnectionExpectContinueTest.java due to 8326503 (Sub-task - P4 - Requested) ⚠️ Issue is not open.

Reviewers

• Goetz Lindenmaier (@GoeLin - Reviewer)
• Andrew John Hughes (@gnu-andrew - Reviewer)

Reviewing

Using git

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

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

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 2545

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

Using diff file

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

Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back syan! 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.

[sendaoYan]

Only change the ProblemList.txt, the risk is low.

[mlbridge]

Webrevs

• 02: Full - Incremental (89704c3f)
• 01: Full - Incremental (908aff3e)
• 00: Full (ca95f323)

[GoeLin]

It is better to make a subtask of 8326503 for this called "Problem list HttpURLConnectionExpectContinueTest.java due to 8326503". The bugid used in the ProblemList should point to an open bug.

[sendaoYan]

It is better to make a subtask of 8326503 for this called "Problem list HttpURLConnectionExpectContinueTest.java due to 8326503". The bugid used in the ProblemList should point to an open bug.

Thank you for your correction, the subtask has been created.

[openjdk]

@sendaoYan usage: /approval [<id>] (request|cancel) [<text>]

[sendaoYan]

/approval JDK-8326564 request The testcase java/net/HttpURLConnection/HttpURLConnectionExpectContinueTest.java should be excluded util JDK-8326503 is fixed

[openjdk]

@sendaoYan
JDK-8326564: The approval request has been created successfully.

[GoeLin]

LGTM

[openjdk]

⚠️ @sendaoYan 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.

[openjdk]

@sendaoYan This change is no longer ready for integration - check the PR body for details.

[sendaoYan]

/integrate

[openjdk]

@sendaoYan
Your change (at version 89704c3) is now ready to be sponsored by a Committer.

[jerboaa]

Perhaps we should push the test fix instead? See #2553 (avoids the unexclude hand-shake).

[GoeLin]

Can this be pushed today or tomorrow? If so that's the best I think.
I'll revoke my fix-yes from the ProblemList change.

[jerboaa]

Can this be pushed today or tomorrow?

That should be doable, yes.

[gnu-andrew]

Looks fine.

[jerboaa]

We've pushed a fix with #2553 so this is no longer needed.

[sendaoYan]

We've pushed a fix with #2553 so this is no longer needed.

OK