[XXS] RFR tier1 testbug JDK-8211180: SourceLauncherTest.java fails in JDK12 CI on Win*
Jonathan Gibbons
jonathan.gibbons at oracle.com
Wed Sep 26 21:18:37 UTC 2018
Please review a small urgent fix for a tier1 testbug issue.
It is a perennial problem, comparing golden text on Windows. This time,
with that in mind, I was too clever by half, and proactively converted
platform newlines in the generated output to \n, not realizing that the
method being used for the comparison was doing the opposite, and
converting \n in the golden string to platform newlines!
The fix is just to avoid converting newlines to \n.
$ hg diff
diff -r 76a3e8be46e6
test/langtools/tools/javac/launcher/SourceLauncherTest.java
--- a/test/langtools/tools/javac/launcher/SourceLauncherTest.java Wed
Sep 26 12:47:38 2018 -0700
+++ b/test/langtools/tools/javac/launcher/SourceLauncherTest.java Wed
Sep 26 14:16:45 2018 -0700
@@ -244,7 +244,7 @@
.className(sourceFile.toString())
.run(Task.Expect.SUCCESS)
.getOutput(Task.OutputKind.STDOUT);
- checkEqual("stdout", log.trim().replace(tb.lineSeparator, "\n"),
+ checkEqual("stdout", log.trim(),
"user.dir=" + System.getProperty("user.dir") + "\n" +
"exception: java.security.AccessControlException: " +
"access denied (\"java.util.PropertyPermission\"
\"user.dir\" \"write\")");
JBS: https://bugs.openjdk.java.net/browse/JDK-8211180
Webrev: http://cr.openjdk.java.net/~jjg/8211180/webrev.00/
-- Jon
More information about the compiler-dev
mailing list