RFR: 8297525: jdk/jshell/ToolBasicTest.java fails after JDK-8295984
Jan Lahoda
jlahoda at openjdk.org
Wed Nov 23 20:22:19 UTC 2022
On Wed, 23 Nov 2022 19:05:09 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:
> The expected text uses a platform newline, but the actual text seems to always use `'\n'`. Sorry for that.
I was somehow convinced the tests are using native line endings, but they are normalizing them here:
https://github.com/openjdk/jdk/blob/6dc4d891c3ad043405c65e0e0eeef28e9e5a2156/test/langtools/jdk/jshell/ReplToolTesting.java#L203
So, I don't think there's a bug in JShell, it is simply the test expecting wrong values.
(And I didn't fully realize the test is not tier1, and so not run on GitHub.)
Sorry for that and thanks for review!
-------------
PR: https://git.openjdk.org/jdk/pull/11330
More information about the kulla-dev
mailing list