Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MoneyTest localisation fails #6269

Open
miklcct opened this issue Nov 22, 2024 · 4 comments
Open

MoneyTest localisation fails #6269

miklcct opened this issue Nov 22, 2024 · 4 comments

Comments

@miklcct
Copy link
Contributor

miklcct commented Nov 22, 2024

Expected behavior

The test passes

Observed behavior

The test fails. The package can't be built.

[ERROR] Failures: 
[ERROR]   BikelyUpdaterTest.parseBikeBoxes:46 expected: <Første 12 time(r) er kr 0,00. Deretter kr 10,00 per 1 time(r)> but was: <Første 12 time(r) er 0,00 kr. Deretter 10,00 kr per 1 time(r)>
[ERROR]   MoneyTest.localize:46 expected: <USD 1,00> but was: <1,00 USD>
[ERROR]   MoneyTest.localize:46 expected: <kr 100,00> but was: <100,00 kr>

Version of OTP used (exact commit hash or JAR name)

0202d1efb1770a24c19c9bfd9f5cf86e5543fd27

The machine is Windows 11, Version 22H2 (Build 22621.4460), in English (United Kingdom) locale. Java version is

openjdk version "23.0.1" 2024-10-15
OpenJDK Runtime Environment (build 23.0.1+11-39)
OpenJDK 64-Bit Server VM (build 23.0.1+11-39, mixed mode, sharing)

P.S. The same test passes with the another JDK:

openjdk version "21.0.5" 2024-10-15 LTS
OpenJDK Runtime Environment Temurin-21.0.5+11 (build 21.0.5+11-LTS)
OpenJDK 64-Bit Server VM Temurin-21.0.5+11 (build 21.0.5+11-LTS, mixed mode, sharing)

Data sets in use (links to GTFS and OSM PBF files)

Not relevant

Command line used to start OTP

Not relevant

Router config and graph build config JSON

Not relevant

Steps to reproduce the problem

Build and package OpenTripPlanner

@leonardehrenfried
Copy link
Member

leonardehrenfried commented Nov 25, 2024

This is probably because of the following bug in the JVM: https://bugs.openjdk.org/browse/JDK-8295564

They appeared to have changed the Norwegian formatting in Java 22.

In my view, it's not a very important test and I would be happy to simply disable the parts about Norwegian money formatting, rather than write a JVM-specific test.

@leonardehrenfried
Copy link
Member

We did something similar here: #5260

@miklcct
Copy link
Contributor Author

miklcct commented Nov 25, 2024

Is it possible to make the test not dependent on JVM implementation details?

@leonardehrenfried
Copy link
Member

You're certainly welcome to try, but I personally would not spend the time on it nor would I request someone else do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants