From d7f33774e0fd29ce941328fe53862bb93393e475 Mon Sep 17 00:00:00 2001
From: Oliver Gierke <info@olivergierke.de>
Date: Wed, 11 May 2011 18:37:59 +0200
Subject: [PATCH] DATACMNS-35 - Added implementation and test for newly
 introduced Repository.delete(ID id) method.

---
 .../mongodb/repository/SimpleMongoRepository.java     |  8 ++++++++
 .../AbstractPersonRepositoryIntegrationTests.java     | 11 +++++++++++
 2 files changed, 19 insertions(+)

diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/document/mongodb/repository/SimpleMongoRepository.java b/spring-data-mongodb/src/main/java/org/springframework/data/document/mongodb/repository/SimpleMongoRepository.java
index 8b1a88b3c..8a3aef063 100644
--- a/spring-data-mongodb/src/main/java/org/springframework/data/document/mongodb/repository/SimpleMongoRepository.java
+++ b/spring-data-mongodb/src/main/java/org/springframework/data/document/mongodb/repository/SimpleMongoRepository.java
@@ -129,6 +129,14 @@ public class SimpleMongoRepository<T, ID extends Serializable> implements Paging
     return template.getCollection(entityInformation.getCollectionName()).count();
   }
 
+  /*
+   * (non-Javadoc)
+   * @see org.springframework.data.repository.Repository#delete(java.io.Serializable)
+   */
+  public void delete(ID id) {
+  	delete(findOne(id));
+  }
+  
   /*
    * (non-Javadoc)
    * 
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/document/mongodb/repository/AbstractPersonRepositoryIntegrationTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/document/mongodb/repository/AbstractPersonRepositoryIntegrationTests.java
index e434991af..6c79779e9 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/document/mongodb/repository/AbstractPersonRepositoryIntegrationTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/document/mongodb/repository/AbstractPersonRepositoryIntegrationTests.java
@@ -73,6 +73,17 @@ public abstract class AbstractPersonRepositoryIntegrationTests {
     assertThat(result.size(), is(4));
     assertThat(result, not(hasItem(dave)));
   }
+  
+  @Test
+  public void deletesPersonByIdCorrectly() {
+  	
+  	repository.delete(dave.getId());
+
+    List<Person> result = repository.findAll();
+
+    assertThat(result.size(), is(4));
+    assertThat(result, not(hasItem(dave)));
+  }
 
   @Test
   public void findsPersonsByLastname() throws Exception {