Update project to JDK 23.

Author: ammbra

Dockerfile

@@ -1,5 +1,5 @@
 # Define your base image
-FROM container-registry.oracle.com/java/openjdk:22-oraclelinux8 as jre-build
+FROM container-registry.oracle.com/java/openjdk:23-oraclelinux9 as jre-build
 
 RUN $JAVA_HOME/bin/jlink \
        --add-modules java.base,java.compiler,java.desktop,java.instrument,java.management,java.net.http,java.prefs,java.rmi,java.scripting,java.security.jgss,java.sql.rowset,jdk.jfr,jdk.net,jdk.unsupported,jdk.management.agent,jdk.crypto.ec,jdk.management.jfr \
@@ -11,7 +11,7 @@ RUN $JAVA_HOME/bin/jlink \
 # Define your base image
 FROM oraclelinux:9-slim
 
-ENV JAVA_HOME /usr/java/openjdk-22
+ENV JAVA_HOME /usr/java/openjdk-23
 ENV PATH $JAVA_HOME/bin:$PATH
 COPY --from=jre-build /javaruntime $JAVA_HOME
 

README.md

@@ -81,7 +81,8 @@ spring.ssl.bundle.jks.mybundle.keystore.location=stores/keystore.p12
 spring.ssl.bundle.jks.mybundle.truststore.location=stores/truststore.p12
 ```
 
-**IMPORTANT** For demo purposes, `application.properties` file contains the passwords in clear. For production environments you must consider encrypting your passwords.
+**IMPORTANT** For demo purposes, `application.properties` contains the passwords in clear. 
+For production environments you must consider encrypting your passwords.
 
 ## How to play
 

pom.xml

@@ -5,7 +5,7 @@
     <parent>
         <groupId>org.springframework.boot</groupId>
         <artifactId>spring-boot-starter-parent</artifactId>
-        <version>3.2.4</version>
+        <version>3.3.2</version>
         <relativePath/>
     </parent>
 
@@ -15,7 +15,7 @@
     <name>tictactoe</name>
     <description>TicTacToe with Spring Boot</description>
     <properties>
-        <java.version>22</java.version>
+        <java.version>23</java.version>
         <maven.dependency.plugin.version>3.5.0</maven.dependency.plugin.version>
         <jackson.databind.version>2.15.2</jackson.databind.version>
     </properties>

src/main/java/com/example/game/ApplicationStartup.java

@@ -31,11 +31,9 @@ public void onApplicationEvent(ApplicationReadyEvent event) {
 			countTLSHandshake(metricsRegistry, es);
 			countX509Parser(metricsRegistry, es);
 
-			es.onEvent(JDK_X509_VALIDATION, recordedEvent -> {
-				Gauge.builder(JDK_X509_VALIDATION + VALIDATION_COUNTER,
-								recordedEvent, e -> e.getLong(VALIDATION_COUNTER))
-						.description("X509 Certificate Validation Gauge").register(metricsRegistry);
-			});
+			es.onEvent(JDK_X509_VALIDATION, recordedEvent -> Gauge.builder(JDK_X509_VALIDATION + VALIDATION_COUNTER,
+							recordedEvent, e -> e.getLong(VALIDATION_COUNTER))
+					.description("X509 Certificate Validation Gauge").register(metricsRegistry));
 
 			es.start();
 		} catch (IOException e) {
@@ -44,7 +42,7 @@ public void onApplicationEvent(ApplicationReadyEvent event) {
 	}
 
 	private static void countTLSHandshake(CompositeMeterRegistry metricsRegistry, EventStream es) {
-		es.onEvent(JDK_TLS_HANDSHAKE, recordedEvent -> {
+		es.onEvent(JDK_TLS_HANDSHAKE, _ -> {
 
 			Counter counter = metricsRegistry.find(JDK_TLS_HANDSHAKE).counter();
 			if (Objects.nonNull(counter)) {

src/main/java/com/example/game/board/BoardService.java

@@ -8,6 +8,8 @@
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.List;
+import java.util.Random;
+import java.util.stream.IntStream;
 
 @Service
 public class BoardService {
@@ -42,8 +44,17 @@ public Board build(Player player, Cell option) {
 
 
     public void randomMove(Board board) {
-        Coordinate location = cellManager.findRandomFreeCell(board.getLines());
-        move(board, location);
+        List<List<String>> lines = board.getLines();
+        List<Coordinate> freeCells = IntStream.range(0, lines.size())
+                .boxed()
+                .flatMap(rowIndex -> IntStream.range(0, lines.getFirst().size())
+                        .mapToObj(columnIndex -> new Coordinate(rowIndex, columnIndex)))
+                .filter(cell -> lines.get(cell.horizontal()).get(cell.vertical()).isEmpty())
+                .toList();
+        if (!freeCells.isEmpty()) {
+            int random = new Random().nextInt(freeCells.size());
+            move(board, freeCells.get(random));
+        }
     }
 
     public void move(Board board, Coordinate location) {

src/main/java/com/example/game/board/cell/CellManager.java

@@ -47,17 +47,4 @@ public static List<List<String>> collectAll(List<List<String>> lines) {
         return borders;
     }
 
-    public Coordinate findRandomFreeCell(List<List<String>> lines) {
-        List<Coordinate> freeCells = IntStream.range(0, lines.size())
-                .boxed()
-                .flatMap(rowIndex -> IntStream.range(0, lines.getFirst().size())
-                        .mapToObj(columnIndex -> new Coordinate(rowIndex, columnIndex)))
-                .filter(cell -> lines.get(cell.horizontal()).get(cell.vertical()).isEmpty())
-                .toList();
-
-        int random = new Random().nextInt(freeCells.size());
-
-        return freeCells.get(random);
-
-    }
 }

src/test/java/com/example/game/board/BoardServiceTest.java

@@ -63,7 +63,6 @@ void randomMove() {
 		when(cellManager.init()).thenCallRealMethod();
 
 		Board board = service.build(new Player(), Cell.X);
-		when(cellManager.findRandomFreeCell(board.getLines())).thenCallRealMethod();
 
 		service.randomMove(board);
 

src/test/java/com/example/game/board/cell/CellManagerTest.java

@@ -19,18 +19,6 @@ void init() {
 		assertThat(board.size()).isEqualTo(3);
 	}
 
-	@Test
-	void findRandomFreeCell() {
-		CellManager cellManager =  new CellManager();
-		List<List<String>> lines =  new ArrayList<>();
-		lines.addAll(Collections.singleton(Arrays.asList(Cell.EMPTY.toString(), Cell.EMPTY.toString(), Cell.EMPTY.toString())));
-
-		Coordinate freeCell = cellManager.findRandomFreeCell(lines);
-		assertThat(freeCell.horizontal()).isNotNegative();
-		assertThat(freeCell.vertical()).isNotNegative();
-
-	}
-
 	@Test
 	void collectAll() {
 		List<List<String>> lines = new ArrayList<>();