Hibernate returns null instead of throwing an exception when using @N…

orm/hibernate-orm-6/pom.xml

@@ -36,11 +36,17 @@
 			<artifactId>junit</artifactId>
 			<version>${version.junit}</version>
 		</dependency>
-		<dependency>
-			<groupId>org.slf4j</groupId>
-			<artifactId>slf4j-log4j12</artifactId>
-			<version>${version.org.slf4j}</version>
-		</dependency>
+        <dependency>
+            <groupId>org.apache.logging.log4j</groupId>
+            <artifactId>log4j-api</artifactId>
+            <version>2.17.1</version>
+        </dependency>
+        <dependency>
+            <groupId>org.apache.logging.log4j</groupId>
+            <artifactId>log4j-core</artifactId>
+            <version>2.17.1</version>
+            <scope>test</scope>
+        </dependency>
 	</dependencies>
 
 	<build>

orm/hibernate-orm-6/src/test/java/org/hibernate/bugs/ChessGame.java

@@ -0,0 +1,57 @@
+package org.hibernate.bugs;
+
+import java.time.LocalDate;
+
+import org.hibernate.annotations.NotFound;
+import org.hibernate.annotations.NotFoundAction;
+
+import jakarta.persistence.Entity;
+import jakarta.persistence.FetchType;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.GenerationType;
+import jakarta.persistence.Id;
+import jakarta.persistence.ManyToOne;
+import jakarta.persistence.Version;
+
+@Entity
+public class ChessGame {
+
+	@Id
+    @GeneratedValue(strategy = GenerationType.SEQUENCE)
+    private Long id;
+
+    @ManyToOne(fetch = FetchType.LAZY)
+    @NotFound(action = NotFoundAction.EXCEPTION)
+    private ChessPlayer playerWhite;
+
+    @ManyToOne(fetch = FetchType.LAZY)
+    @NotFound(action = NotFoundAction.IGNORE)
+    private ChessPlayer playerBlack;
+
+    @Version
+    private int version;
+
+    public Long getId() {
+        return id;
+    }
+
+    public int getVersion() {
+        return version;
+    }
+
+    public ChessPlayer getPlayerWhite() {
+        return playerWhite;
+    }
+
+    public void setPlayerWhite(ChessPlayer playerWhite) {
+        this.playerWhite = playerWhite;
+    }
+
+    public ChessPlayer getPlayerBlack() {
+        return playerBlack;
+    }
+
+    public void setPlayerBlack(ChessPlayer playerBlack) {
+        this.playerBlack = playerBlack;
+    }
+}

orm/hibernate-orm-6/src/test/java/org/hibernate/bugs/ChessPlayer.java

@@ -0,0 +1,53 @@
+package org.hibernate.bugs;
+
+import java.time.LocalDate;
+import java.util.Set;
+
+import jakarta.persistence.Entity;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.GenerationType;
+import jakarta.persistence.Id;
+import jakarta.persistence.OneToMany;
+import jakarta.persistence.Version;
+
+@Entity
+public class ChessPlayer {
+
+	@Id
+    @GeneratedValue(strategy = GenerationType.SEQUENCE)
+	private Long id;
+
+    @OneToMany(mappedBy = "playerWhite")
+    private Set<ChessGame> gamesWhite;
+
+    @OneToMany(mappedBy = "playerBlack")
+    private Set<ChessGame> gamesBlack;
+
+    @Version
+    private int version;
+
+    public Long getId() {
+        return id;
+    }
+
+    public Set<ChessGame> getGamesWhite() {
+        return gamesWhite;
+    }
+
+    public void setGamesWhite(Set<ChessGame> gamesWhite) {
+        this.gamesWhite = gamesWhite;
+    }
+
+    public Set<ChessGame> getGamesBlack() {
+        return gamesBlack;
+    }
+
+    public void setGamesBlack(Set<ChessGame> gamesBlack) {
+        this.gamesBlack = gamesBlack;
+    }
+
+    public int getVersion() {
+        return version;
+    }
+
+}

orm/hibernate-orm-6/src/test/java/org/hibernate/bugs/JPAUnitTestCase.java

@@ -1,18 +1,26 @@
 package org.hibernate.bugs;
 
-import jakarta.persistence.EntityManager;
-import jakarta.persistence.EntityManagerFactory;
-import jakarta.persistence.Persistence;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.fail;
 
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 
+import jakarta.persistence.EntityManager;
+import jakarta.persistence.EntityManagerFactory;
+import jakarta.persistence.Persistence;
+
 /**
  * This template demonstrates how to develop a test case for Hibernate ORM, using the Java Persistence API.
  */
 public class JPAUnitTestCase {
 
+    Logger log = LogManager.getLogger(this.getClass().getName());
+
 	private EntityManagerFactory entityManagerFactory;
 
 	@Before
@@ -28,10 +36,20 @@ public void destroy() {
 	// Entities are auto-discovered, so just add them anywhere on class-path
 	// Add your tests, using standard JUnit.
 	@Test
-	public void hhh123Test() throws Exception {
+	public void notFoundIssue() throws Exception {
 		EntityManager entityManager = entityManagerFactory.createEntityManager();
 		entityManager.getTransaction().begin();
-		// Do stuff...
+
+        ChessGame game = entityManager.find(ChessGame.class, 1L);
+        assertNotNull("Returned entity shouldn't be null", game);
+        assertNull(game.getPlayerBlack(), "Broken foreign key reference with NotFoundAction.IGNORE should return null");
+        try {
+            game.getPlayerWhite();
+            fail("Accessing a broken foreign key reference should throw an exception.");
+        } catch (Exception e) {
+            // Exception expected
+        }
+
 		entityManager.getTransaction().commit();
 		entityManager.close();
 	}

orm/hibernate-orm-6/src/test/resources/META-INF/persistence.xml

@@ -23,6 +23,7 @@
             <property name="hibernate.show_sql" value="true"/>
             <property name="hibernate.format_sql" value="true"/>
             <property name="hibernate.hbm2ddl.auto" value="create-drop"/>
+            <property name="javax.persistence.sql-load-script-source" value="data.sql"/>
 
             <property name="hibernate.max_fetch_depth" value="5"/>
 

orm/hibernate-orm-6/src/test/resources/data.sql

@@ -0,0 +1 @@
+INSERT INTO ChessGame(id, version, playerwhite_id, playerblack_id) VALUES (1, 0, 1, 2);

orm/hibernate-orm-6/src/test/resources/log4j2.xml

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Configuration>
+  <Appenders>
+    <Console name="STDOUT" target="SYSTEM_OUT">
+      <PatternLayout pattern="%d{HH:mm:ss,SSS} %-5p [%c] - %m%n"/>
+    </Console>
+  </Appenders>
+  <Loggers>
+    <Logger name="org.hibernate.stat" level="debug"/>
+    <Logger name="org.hibernate.SQL" level="debug"/>
+
+    <Logger name="org.hibernate" level="info" />
+    <Root level="info">
+      <AppenderRef ref="STDOUT"/>
+    </Root>
+  </Loggers>
+</Configuration>