Skip to content

Commit

Permalink
8333698: [17u] TestJstatdRmiPort fails after JDK-8333667
Browse files Browse the repository at this point in the history
Reviewed-by: mdoerr
  • Loading branch information
shipilev committed Jun 7, 2024
1 parent 2d9b1a6 commit bcd19a5
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions test/jdk/sun/tools/jstatd/JstatdTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -358,8 +358,11 @@ private void runTest(boolean useShortSyntax) throws Throwable {
OutputAnalyzer output = jstatdThread.getOutput();
List<String> stdout = output.asLinesWithoutVMWarnings();
output.reportDiagnosticSummary();
assertEquals(stdout.size(), 1, "Output should contain one line");
assertTrue(stdout.get(0).startsWith("jstatd started"), "List should start with 'jstatd started'");
// These asserts are disabled until JDK-8272317 is backported:
// otherwise there are SM deprecation notices that fail them.
// assertEquals(stdout.size(), 1, "Output should contain one line");
// assertTrue(stdout.get(0).startsWith("jstatd started"), "List should start with 'jstatd started'");
output.shouldContain("jstatd started");
assertNotEquals(output.getExitValue(), 0,
"jstatd process exited with unexpected exit code");
}
Expand Down

1 comment on commit bcd19a5

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.