From fd86225f8b3eb325183d2816bb7d6736c222ccc4 Mon Sep 17 00:00:00 2001 From: Byungju Ju Date: Thu, 18 Dec 2025 23:17:22 +0900 Subject: [PATCH] Add test coverage for persist() path in SimpleJpaRepository.save(). Signed-off-by: Byungju Ju --- .../support/SimpleJpaRepositoryUnitTests.java | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/support/SimpleJpaRepositoryUnitTests.java b/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/support/SimpleJpaRepositoryUnitTests.java index 0d1e8133f5..59532d7f41 100644 --- a/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/support/SimpleJpaRepositoryUnitTests.java +++ b/spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/support/SimpleJpaRepositoryUnitTests.java @@ -60,6 +60,7 @@ * @author Greg Turnquist * @author Yanming Zhou * @author Ariel Morelli Andres + * @author Byungju Ju */ @ExtendWith(MockitoExtension.class) @MockitoSettings(strictness = Strictness.LENIENT) @@ -182,7 +183,19 @@ void mergeGetsCalledWhenAttached() { verify(em).merge(attachedUser); } - @Test // DATAJPA-1535 + @Test // GH-4125 + void persistGetsCalledWhenEntityIsNew() { + + User user = new User(); + + when(information.isNew(user)).thenReturn(true); + + repo.save(user); + + verify(em).persist(user); + } + + @Test // DATAJPA-1535 void doNothingWhenNewInstanceGetsDeleted() { User newUser = new User();