Skip to content

Commit

Permalink
Fix MacOS tests failures in java.hints
Browse files Browse the repository at this point in the history
The javac outputs of the following tests in java/java.hints
ErrorHintsProviderTest were updated for non-mac OSes due to changes in
JDK 7 updates:
- testShortErrors5: [jdk-6968793](openjdk/jdk@b77effad6c6)
- testTestShortErrorsSVUIDWarning: [jdk-6957438](openjdk/jdk@1c75e97108)

Since Apple JDK was present only up till JDK 6, we no longer require the
special case for MacOS since JDK 8.

Thus, making the test argument value `specialMacTreatment = false` for
these two tests, and renaming the corresponding goldenfiles from
"-nonmac.pass" suffix to ".pass" suffix.

Signed-off-by: Siddharth Srinivasan <[email protected]>
  • Loading branch information
sid-srini committed Oct 30, 2024
1 parent 635860e commit daf79f9
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 8 deletions.

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
5:8-5:14:error:compiler message file broken:key=compiler.err.cant.resolve.location.args arguments=method, create, , , class, javahints.TestShortErrors5, {6}, {7}
5:8-5:14:error:cannot find symbol
symbol:method create()
location:class TestShortErrors5

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1 +1 @@
2:13-2:40:warning:serializable class javahints.TestShortErrorsSVUIDWarning has no definition of serialVersionUID
2:13-2:40:warning:[serial] serializable class TestShortErrorsSVUIDWarning has no definition of serialVersionUID
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ public void testShortErrors4() throws Exception {
}

public void testShortErrors5() throws Exception {
performTest("TestShortErrors5", true);
performTest("TestShortErrors5", false);
}

public void testShortErrors6() throws Exception {
Expand Down Expand Up @@ -228,7 +228,7 @@ public void testTestShortErrorsSVUIDWarning() throws Exception {
TestCompilerSettings.commandLine = "-Xlint:serial";

try {
performTest("TestShortErrorsSVUIDWarning", true);
performTest("TestShortErrorsSVUIDWarning", false);
} finally {
TestCompilerSettings.commandLine = null;
}
Expand Down

0 comments on commit daf79f9

Please sign in to comment.