From ad40b37f4095b48c14363ad2afa3a4ad3eddd496 Mon Sep 17 00:00:00 2001 From: Oliver Gierke Date: Wed, 11 May 2011 18:39:27 +0200 Subject: [PATCH] DATACMNS-35 - Added implementation and test for newly introduced Repository.delete(ID id) method. --- .../data/jpa/repository/JpaRepository.java | 11 +++++++++++ .../jpa/repository/support/SimpleJpaRepository.java | 13 +++++++++++++ .../data/jpa/repository/UserRepositoryTests.java | 10 ++++++++++ 3 files changed, 34 insertions(+) diff --git a/src/main/java/org/springframework/data/jpa/repository/JpaRepository.java b/src/main/java/org/springframework/data/jpa/repository/JpaRepository.java index 82cd65284..7ad71cdc9 100644 --- a/src/main/java/org/springframework/data/jpa/repository/JpaRepository.java +++ b/src/main/java/org/springframework/data/jpa/repository/JpaRepository.java @@ -98,6 +98,17 @@ public interface JpaRepository extends long count(); + /* + * (non-Javadoc) + * + * @see + * org.springframework.data.repository.Repository#delete(java.io.Serializable + * ) + */ + @Transactional + public void delete(ID id); + + /* * (non-Javadoc) * diff --git a/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java b/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java index f00a8292e..8f91bd009 100644 --- a/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java +++ b/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java @@ -112,6 +112,19 @@ public class SimpleJpaRepository implements } + /* + * (non-Javadoc) + * + * @see + * org.springframework.data.jpa.repository.JpaRepository#delete(java.io. + * Serializable) + */ + public void delete(ID id) { + + delete(findOne(id)); + } + + /* * (non-Javadoc) * diff --git a/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java b/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java index b72083d58..a6df73ed0 100644 --- a/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java +++ b/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java @@ -187,6 +187,16 @@ public class UserRepositoryTests { } + @Test + public void deletesAUserById() { + + flushTestUsers(); + + repository.delete(firstUser.getId()); + assertNull(repository.findOne(firstUser.getId())); + } + + /** * Tests deleting a user. */