jbonofre commented on code in PR #886:
URL: https://github.com/apache/polaris/pull/886#discussion_r1930713912
##########
quarkus/service/build.gradle.kts:
##########
@@ -165,6 +165,8 @@ tasks.named<Test>("intTest").configure {
// Same issue as above: allow a java security manager after Java 21
// (this setting is for the application under test, while the setting above
is for test code).
systemProperty("quarkus.test.arg-line", "-Djava.security.manager=allow")
+ // force the locale, just in case the user is having another default locale
+ systemProperty("quarkus.default-locale", "en_US")
Review Comment:
For instance
https://github.com/apache/polaris/blob/main/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisManagementServiceIntegrationTest.java#L366
and
https://github.com/apache/polaris/blob/main/integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisManagementServiceIntegrationTest.java#L383
assumed the locale is `en_US`: it expects `must not be null`.
If the locale is not `en_US`, let say `fr_FR`, then the tests fail as we
have `ne doit pas être nul` instead of `must not be null`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
