From 1b3d9ed2c4a697aa9141782f8a0f6309cd592468 Mon Sep 17 00:00:00 2001
From: Oliver Gierke <ogierke@gopivotal.com>
Date: Thu, 10 Oct 2013 13:42:33 +0200
Subject: [PATCH] =?UTF-8?q?DATACMNS-380=20-=20MappingContext.getPersistent?=
 =?UTF-8?q?PropertyPath(=E2=80=A6)=20now=20throws=20MappingException.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../data/mapping/context/AbstractMappingContext.java      | 2 +-
 .../mapping/context/AbstractMappingContextUnitTests.java  | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/springframework/data/mapping/context/AbstractMappingContext.java b/src/main/java/org/springframework/data/mapping/context/AbstractMappingContext.java
index 000ee0845..d940097ed 100644
--- a/src/main/java/org/springframework/data/mapping/context/AbstractMappingContext.java
+++ b/src/main/java/org/springframework/data/mapping/context/AbstractMappingContext.java
@@ -223,7 +223,7 @@ public abstract class AbstractMappingContext<E extends MutablePersistentEntity<?
 			P persistentProperty = current.getPersistentProperty(segment);
 
 			if (persistentProperty == null) {
-				throw new IllegalArgumentException(String.format("No property %s found on %s!", segment, current.getName()));
+				throw new MappingException(String.format("No property %s found on %s!", segment, current.getName()));
 			}
 
 			result.add(persistentProperty);
diff --git a/src/test/java/org/springframework/data/mapping/context/AbstractMappingContextUnitTests.java b/src/test/java/org/springframework/data/mapping/context/AbstractMappingContextUnitTests.java
index 771dd8a75..ef71f895e 100644
--- a/src/test/java/org/springframework/data/mapping/context/AbstractMappingContextUnitTests.java
+++ b/src/test/java/org/springframework/data/mapping/context/AbstractMappingContextUnitTests.java
@@ -185,6 +185,14 @@ public class AbstractMappingContextUnitTests {
 		assertThat(path.getLeafProperty().getName(), is("name"));
 	}
 
+	/**
+	 * @see DATACMNS-380
+	 */
+	@Test(expected = MappingException.class)
+	public void rejectsInvalidPropertyReferenceWithMappingException() {
+		context.getPersistentPropertyPath("foo", Sample.class);
+	}
+
 	class Person {
 		String name;
 	}