Skip to content

Commit

Permalink
Bump Airbase to 126
Browse files Browse the repository at this point in the history
* Version 125 brings (among other things) updated dependency on Jackson,
  which fixes at least one CVE. Dependency on snakeyaml needs to be
  bumped to avoid conflicts with transitive dependencies.

* Version 126 brings fixes to how locale information is passed to Maven
  Surefire plugin. In relation to that, this commit adds a test to
  verify that the system locale is actually properly configured in the
  JVM.
  • Loading branch information
ksobolew authored and electrum committed Jun 9, 2022
1 parent 99ff1c6 commit dbebd79
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
<parent>
<groupId>io.airlift</groupId>
<artifactId>airbase</artifactId>
<version>123</version>
<version>126</version>
</parent>

<groupId>io.trino</groupId>
Expand Down Expand Up @@ -1692,7 +1692,7 @@
<dependency>
<groupId>org.yaml</groupId>
<artifactId>snakeyaml</artifactId>
<version>1.28</version>
<version>1.30</version>
</dependency>
</dependencies>
</dependencyManagement>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
import org.testng.annotations.Test;

import java.time.ZoneId;
import java.util.Locale;

import static org.testng.Assert.assertEquals;

Expand All @@ -27,4 +28,11 @@ public void testJvmZone()
// Ensure that the zone defined in the POM is correctly set in the test JVM
assertEquals(ZoneId.systemDefault().getId(), "America/Bahia_Banderas");
}

@Test
public void testJvmLocale()
{
// Ensure that locale defined in the POM is correctly set in the test JVM
assertEquals(Locale.getDefault(), Locale.US);
}
}

0 comments on commit dbebd79

Please sign in to comment.