Adding appropriate alert for non auto prop ghost property

SVN: trunk@4933

Author: ayende

src/NHibernate.Test/GhostProperty/GhostPropertyFixture.cs

@@ -1,13 +1,16 @@
 using System.Collections;
 using NHibernate.ByteCode.Castle;
 using NHibernate.Cfg;
+using NHibernate.Tuple.Entity;
 using NUnit.Framework;
 
 namespace NHibernate.Test.GhostProperty
 {
 	[TestFixture]
 	public class GhostPropertyFixture : TestCase
 	{
+		private string log;
+
 		protected override string MappingsAssembly
 		{
 			get { return "NHibernate.Test"; }
@@ -55,6 +58,21 @@ protected override void OnTearDown()
 			}
 		}
 
+		protected override void BuildSessionFactory()
+		{
+			using (var logSpy = new LogSpy(typeof(EntityMetamodel)))
+			{
+				base.BuildSessionFactory();
+				log = logSpy.GetWholeLog();
+			}
+		}
+
+		[Test]
+		public void ShouldGenerateErrorForNonAutoPropGhostProp()
+		{
+			Assert.IsTrue(log.Contains("Lazy or ghost property NHibernate.Test.GhostProperty.Order.Payment is not an auto property, which may result in uninitialized property access"));
+		}
+
 		[Test]
 		public void CanGetActualValueFromLazyManyToOne()
 		{

src/NHibernate.Test/GhostProperty/Order.cs

@@ -3,7 +3,13 @@
 	public class Order
 	{
 		public virtual int Id { get; set; }
-		public virtual Payment Payment { get; set; }
+		private Payment payment;
+
+		public virtual Payment Payment
+		{
+			get { return payment; }
+			set { payment = value; }
+		}
 	}
 
 	public abstract class Payment

src/NHibernate/Tuple/Entity/EntityMetamodel.cs

@@ -303,7 +303,8 @@ private static void VerifyCanInterceptPropertiesForLazyOrGhostProperties(Persist
 		{
 			foreach (var prop in persistentClass.PropertyClosureIterator)
 			{
-				if (prop.IsLazy == false && prop.UnwrapProxy)
+				if (prop.IsLazy == false &&
+					prop.UnwrapProxy == false)
 					continue;
 
 				var getter = prop.GetGetter(persistentClass.MappedClass);