Skip to content

8362658: sun/security/ssl/SSLEngineImpl/* tests duplicate jvm flags #28428

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
+7 −36
Open

8362658: sun/security/ssl/SSLEngineImpl/* tests duplicate jvm flags #28428

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
379d382
JDK-8362658 : Updated code to remove duplicate jvm flags.
nehajoshinj Nov 18, 2025
d42d36a
JDK-8362658 : Updated copyright and removed unused import.
nehajoshinj Nov 19, 2025
e99cd51
Merge branch 'openjdk:master' into JDK-8362658/Open/Duplicate_JVM_flags
nehajoshinj Nov 20, 2025
3b6c8d4
JDK-8362658 : Added back the space.
nehajoshinj Nov 20, 2025
60bc1d7
JDK-8362658 : Reformat code to comply with 80-character line length l…
nehajoshinj Nov 20, 2025
a0cc39e
JDK-8362658 : Reformat code to comply with 80-character line length l…
nehajoshinj Nov 20, 2025
c641c6e
JDK-8368524 : Removed the list of test case from problemList file.
nehajoshinj Nov 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 0 additions & 16 deletions test/jdk/ProblemList-jvmti-stress-agent.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,22 +21,6 @@
# questions.
#

#############################################################################
#
# List of quarantined tests failing with jvmti stress agent in any mode.
#
#############################################################################


sun/security/ssl/SSLEngineImpl/SSLEngineKeyLimit.java 8362658 generic-all
sun/security/ssl/SSLSessionImpl/MultiNSTClient.java 8362658 generic-all
sun/security/ssl/SSLSessionImpl/MultiNSTNoSessionCreation.java 8362658 generic-all
sun/security/ssl/SSLSessionImpl/MultiNSTParallel.java 8362658 generic-all
sun/security/ssl/SSLSessionImpl/MultiNSTSequence.java 8362658 generic-all
sun/security/ssl/SSLSessionImpl/ResumptionUpdateBoundValues.java 8362658 generic-all
sun/security/ssl/SSLSocketImpl/SSLSocketKeyLimit.java 8362658 generic-all


# List of tests incompatible with jvmti stress agent or requiring more investigation

com/sun/jdi/EATests.java#id0 0000000 generic-all
Expand Down
5 changes: 1 addition & 4 deletions test/jdk/sun/security/ssl/SSLEngineImpl/SSLEngineKeyLimit.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@

import jdk.test.lib.process.ProcessTools;
import jdk.test.lib.process.OutputAnalyzer;
import jdk.test.lib.Utils;

public class SSLEngineKeyLimit extends SSLContextTemplate {

Expand Down Expand Up @@ -119,9 +118,7 @@ public static void main(String args[]) throws Exception {
System.getProperty("test.java.opts"));

ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
Utils.addTestJavaOpts("SSLEngineKeyLimit", "p", args[1],
args[2]));

"SSLEngineKeyLimit", "p", args[1], args[2]);
OutputAnalyzer output = ProcessTools.executeProcess(pb);
try {
output.shouldContain(String.format(
Expand Down
4 changes: 1 addition & 3 deletions test/jdk/sun/security/ssl/SSLSessionImpl/MultiNSTClient.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@
* @run main/othervm MultiNSTClient -Djdk.tls.client.protocols=TLSv1.2 -Djdk.tls.server.enableSessionTicketExtension=true -Djdk.tls.client.enableSessionTicketExtension=true
*/

import jdk.test.lib.Utils;
import jdk.test.lib.process.OutputAnalyzer;
import jdk.test.lib.process.ProcessTools;

Expand Down Expand Up @@ -77,8 +76,7 @@ public static void main(String[] args) throws Exception {
System.out.println("test.java.opts: " +
System.getProperty("test.java.opts"));

ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
Utils.addTestJavaOpts("MultiNSTClient", "p"));
ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder("MultiNSTClient", "p");

OutputAnalyzer output = ProcessTools.executeProcess(pb);
boolean pass = true;
Expand Down
3 changes: 1 addition & 2 deletions test/jdk/sun/security/ssl/SSLSessionImpl/MultiNSTNoSessionCreation.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@
* @run main/othervm MultiNSTNoSessionCreation -Djdk.tls.client.protocols=TLSv1.2 -Djdk.tls.server.newSessionTicketCount=0
*/

import jdk.test.lib.Utils;
import jdk.test.lib.process.OutputAnalyzer;
import jdk.test.lib.process.ProcessTools;

Expand Down Expand Up @@ -61,7 +60,7 @@ public static void main(String[] args) throws Exception {
System.getProperty("test.java.opts"));

ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
Utils.addTestJavaOpts("MultiNSTNoSessionCreation", "p"));
"MultiNSTNoSessionCreation", "p");

OutputAnalyzer output = ProcessTools.executeProcess(pb);
try {
Expand Down
4 changes: 1 addition & 3 deletions test/jdk/sun/security/ssl/SSLSessionImpl/MultiNSTParallel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@
* @run main/othervm MultiNSTParallel 10 -Djdk.tls.client.protocols=TLSv1.3
*/

import jdk.test.lib.Utils;
import jdk.test.lib.process.OutputAnalyzer;
import jdk.test.lib.process.ProcessTools;

Expand Down Expand Up @@ -116,8 +115,7 @@ public static void main(String[] args) throws Exception {
System.out.println("test.java.opts: " +
System.getProperty("test.java.opts"));

ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
Utils.addTestJavaOpts("MultiNSTParallel", "p"));
ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder("MultiNSTParallel", "p");

OutputAnalyzer output = ProcessTools.executeProcess(pb);
try {
Expand Down
4 changes: 1 addition & 3 deletions test/jdk/sun/security/ssl/SSLSessionImpl/MultiNSTSequence.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@
* @run main/othervm MultiNSTSequence -Djdk.tls.server.newSessionTicketCount=2
*/

import jdk.test.lib.Utils;
import jdk.test.lib.process.OutputAnalyzer;
import jdk.test.lib.process.ProcessTools;

Expand Down Expand Up @@ -69,8 +68,7 @@ public static void main(String[] args) throws Exception {
System.out.println("test.java.opts: " +
System.getProperty("test.java.opts"));

ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
Utils.addTestJavaOpts("MultiNSTSequence", "p"));
ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder("MultiNSTSequence", "p");

OutputAnalyzer output = ProcessTools.executeProcess(pb);
boolean pass = true;
Expand Down
3 changes: 1 addition & 2 deletions test/jdk/sun/security/ssl/SSLSessionImpl/ResumptionUpdateBoundValues.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@

import jdk.test.lib.process.OutputAnalyzer;
import jdk.test.lib.process.ProcessTools;
import jdk.test.lib.Utils;

public class ResumptionUpdateBoundValues extends SSLContextTemplate {

Expand Down Expand Up @@ -188,7 +187,7 @@ public static void main(String[] args) throws Exception {
System.getProperty("test.java.opts"));

ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
Utils.addTestJavaOpts("ResumptionUpdateBoundValues", "p"));
"ResumptionUpdateBoundValues", "p");

OutputAnalyzer output = ProcessTools.executeProcess(pb);
try {
Expand Down
4 changes: 1 addition & 3 deletions test/jdk/sun/security/ssl/SSLSocketImpl/SSLSocketKeyLimit.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@

import jdk.test.lib.process.OutputAnalyzer;
import jdk.test.lib.process.ProcessTools;
import jdk.test.lib.Utils;
import jdk.test.lib.hexdump.HexPrinter;

public class SSLSocketKeyLimit {
Expand Down Expand Up @@ -135,8 +134,7 @@ public static void main(String args[]) throws Exception {
System.getProperty("test.java.opts"));

ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
Utils.addTestJavaOpts("SSLSocketKeyLimit", "p", args[1],
args[2]));
"SSLSocketKeyLimit", "p", args[1], args[2]);

OutputAnalyzer output = ProcessTools.executeProcess(pb);
try {
Expand Down