Test deleting versioned entity

Author: quaff

spring-data-jpa/src/test/java/org/springframework/data/jpa/domain/sample/VersionedUser.java

@@ -22,6 +22,7 @@
 
 /**
  * @author Oliver Gierke
+ * @author Yanming Zhou
  */
 @Entity
 public class VersionedUser {
@@ -33,6 +34,8 @@ public class VersionedUser {
 	@Version
 	private Long version;
 
+	private String name;
+
 	public Long getId() {
 		return id;
 	}
@@ -48,4 +51,12 @@ public Long getVersion() {
 	public void setVersion(Long version) {
 		this.version = version;
 	}
+
+	public String getName() {
+		return name;
+	}
+
+	public void setName(String name) {
+		this.name = name;
+	}
 }

spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/support/JpaRepositoryTests.java

@@ -18,6 +18,7 @@
 import static org.assertj.core.api.Assertions.*;
 
 import jakarta.persistence.EntityManager;
+import jakarta.persistence.OptimisticLockException;
 import jakarta.persistence.PersistenceContext;
 
 import java.util.Arrays;
@@ -32,6 +33,7 @@
 import org.springframework.data.jpa.domain.sample.PersistableWithIdClassPK;
 import org.springframework.data.jpa.domain.sample.SampleEntity;
 import org.springframework.data.jpa.domain.sample.SampleEntityPK;
+import org.springframework.data.jpa.domain.sample.VersionedUser;
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.data.repository.CrudRepository;
 import org.springframework.test.context.ContextConfiguration;
@@ -46,6 +48,7 @@
  * @author Jens Schauder
  * @author Greg Turnquist
  * @author Krzysztof Krason
+ * @author Yanming Zhou
  */
 @ExtendWith(SpringExtension.class)
 @ContextConfiguration({ "classpath:infrastructure.xml" })
@@ -56,11 +59,13 @@ class JpaRepositoryTests {
 
 	private JpaRepository<SampleEntity, SampleEntityPK> repository;
 	private CrudRepository<PersistableWithIdClass, PersistableWithIdClassPK> idClassRepository;
+	private JpaRepository<VersionedUser, Long> versionedUserRepository;
 
 	@BeforeEach
 	void setUp() {
 		repository = new JpaRepositoryFactory(em).getRepository(SampleEntityRepository.class);
 		idClassRepository = new JpaRepositoryFactory(em).getRepository(SampleWithIdClassRepository.class);
+		versionedUserRepository = new JpaRepositoryFactory(em).getRepository(VersionedUserRepository.class);
 	}
 
 	@Test
@@ -162,6 +167,26 @@ public Iterator<SampleEntityPK> iterator() {
 		assertThat(repository.findAll()).containsExactly(two);
 	}
 
+	@Test
+	void deleteStaleVersionedEntityShouldRaiseOptimisticLockException() {
+
+		VersionedUser entity = new VersionedUser();
+		entity.setName("name");
+		versionedUserRepository.save(entity);
+		versionedUserRepository.flush();
+		em.detach(entity);
+
+		versionedUserRepository.findById(entity.getId()).ifPresent(u -> {
+			u.setName("new name");
+			VersionedUser latest = versionedUserRepository.save(u);
+			versionedUserRepository.flush();
+		});
+
+		assertThatExceptionOfType(OptimisticLockException.class).isThrownBy(() -> {
+			versionedUserRepository.delete(entity);
+		});
+	}
+
 	private interface SampleEntityRepository extends JpaRepository<SampleEntity, SampleEntityPK> {
 
 	}
@@ -170,4 +195,8 @@ private interface SampleWithIdClassRepository
 			extends CrudRepository<PersistableWithIdClass, PersistableWithIdClassPK> {
 
 	}
+
+	private interface VersionedUserRepository extends JpaRepository<VersionedUser, Long> {
+
+	}
 }