RFR: 8260468: Wrong behavior of LocalDateTimeStringConverter

[Kevin Rushforth]

On Thu, 4 Feb 2021 20:04:39 GMT, Nir Lisker <nlisker at openjdk.org> wrote:

> The added test will look meaningless after the patch, since it's trying to force a change that can't happen anymore, and this can be confusing in the future. Does it still make sense to commit it? It's more of a demonstration of the bug and the fix.

This seems fine to me. We've done this sort of thing for other tests as well.

> I also noticed that other tests in `LocalDateTimeStringConverterTest` are not named with `test` in the beginning. Is this wrong?

This is a loose naming convention that many classes follow, but many others do not, so it is not a problem.

-------------

PR: https://git.openjdk.java.net/jfx/pull/393