Added test for issue NHE-161: https://nhibernate.jira.com/browse/NHE-161

Author: Rico Pfennig

Src/NHibernate.Envers.Tests/NHibernate.Envers.Tests.csproj

@@ -149,6 +149,8 @@
     <Compile Include="NetSpecific\Integration\RevInfo\Time\LongTest.cs" />
     <Compile Include="NetSpecific\Integration\ModifiedFlags\Casee.cs" />
     <Compile Include="NetSpecific\Integration\ModifiedFlags\MergeColModTest.cs" />
+    <Compile Include="NetSpecific\Integration\RevisionOnCollectionChange\Domain.cs" />
+    <Compile Include="NetSpecific\Integration\RevisionOnCollectionChange\Fixture.cs" />
     <Compile Include="NetSpecific\UnitTests\CustomLists\CustomListClasses.cs" />
     <Compile Include="NetSpecific\UnitTests\CustomLists\CustomListsTestModel.cs" />
     <Compile Include="NetSpecific\UnitTests\CustomLists\CustomListTests.cs" />
@@ -802,6 +804,7 @@
     <Compile Include="ValidityTestBase.cs" />
   </ItemGroup>
   <ItemGroup>
+    <EmbeddedResource Include="NetSpecific\Integration\RevisionOnCollectionChange\Mapping.hbm.xml" />
     <EmbeddedResource Include="NetSpecific\Integration\MultiLevelInheritance\Mapping.hbm.xml" />
     <EmbeddedResource Include="NetSpecific\Integration\BidirectionalSameColumn\Mapping.hbm.xml" />
     <EmbeddedResource Include="NetSpecific\Integration\CustomMapping\ImplicitEnversCollectionType\Mapping.hbm.xml" />

Src/NHibernate.Envers.Tests/NetSpecific/Integration/RevisionOnCollectionChange/Domain.cs

@@ -0,0 +1,26 @@
+using System;
+using System.Collections.Generic;
+using NHibernate.Envers.Configuration.Attributes;
+
+namespace NHibernate.Envers.Tests.NetSpecific.Integration.RevisionOnCollectionChange
+{
+	[Audited]
+	public class Person
+	{
+		public Person()
+		{
+			Cars = new List<Car>();
+		}
+		public virtual Guid Id { get; set; }
+
+		public virtual IList<Car> Cars { get; protected set; }
+	}
+
+	[Audited]
+	public class Car
+	{
+		public virtual Guid Id { get; set; }
+
+		public virtual Person Owner { get; set; }
+	}
+}

Src/NHibernate.Envers.Tests/NetSpecific/Integration/RevisionOnCollectionChange/Fixture.cs

@@ -0,0 +1,49 @@
+using NHibernate.Cfg;
+using NHibernate.Envers.Configuration;
+using NUnit.Framework;
+
+namespace NHibernate.Envers.Tests.NetSpecific.Integration.RevisionOnCollectionChange
+{
+	[TestFixture]
+	public class Fixture : TestBase
+	{
+		private object _newOwnerId;
+		private object _carId;
+
+		public Fixture(string strategyType) : base(strategyType)
+		{
+		}
+
+		protected override void AddToConfiguration(Cfg.Configuration configuration)
+		{
+			configuration.SetEnversProperty(ConfigurationKey.GlobalWithModifiedFlag, true);
+			configuration.SetEnversProperty(ConfigurationKey.RevisionOnCollectionChange, true);
+			base.AddToConfiguration(configuration);
+		}
+
+		protected override void Initialize()
+		{
+			using (var tx = Session.BeginTransaction())
+			{
+				var oldOwner = new Person();
+				Session.Save(oldOwner);
+				var newOwner = new Person();
+				_newOwnerId = Session.Save(newOwner);
+				var car = new Car { Owner = oldOwner };
+				_carId = Session.Save(car);
+				tx.Commit();
+			}
+		}
+
+		[Test]
+		public void ChangeParentReferenceShouldNotThwrowException()
+		{
+			using (var tx = Session.BeginTransaction())
+			{
+				var car = Session.Get<Car>(_carId);
+				car.Owner = Session.Get<Person>(_newOwnerId);
+				tx.Commit();
+			}
+		}
+	}
+}

Src/NHibernate.Envers.Tests/NetSpecific/Integration/RevisionOnCollectionChange/Mapping.hbm.xml

@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<hibernate-mapping xmlns="urn:nhibernate-mapping-2.2"
+						 assembly="NHibernate.Envers.Tests"
+						 namespace="NHibernate.Envers.Tests.NetSpecific.Integration.RevisionOnCollectionChange">
+  <class name="Car">
+    <id name="Id">
+      <column name="Id" />
+      <generator class="guid.comb"/>
+    </id>
+    <many-to-one class="Person" name="Owner">
+      <column name="Owner_Id" />
+    </many-to-one>
+  </class>
+
+  <class name="Person">
+    <id name="Id">
+      <column name="Id" />
+      <generator class="guid.comb"/>
+    </id>
+    <bag name="Cars">
+      <key>
+        <column name="Owner_Id" />
+      </key>
+      <one-to-many class="Car" />
+    </bag>
+  </class>
+</hibernate-mapping>