NH-2176 - Forgotten update of ambient transaction. More cleanup of DTC tests.

Author: fredericDelaporte

src/NHibernate.Test/NHSpecificTest/DtcFailures/DtcFailuresFixture.cs

@@ -141,7 +141,7 @@ public void WillNotCrashOnDtcPrepareFailure()
 		}
 
 		[Test]
-		public void Can_roll_back_transaction()
+		public void CanRollbackTransaction()
 		{
 			var tx = new TransactionScope();
 			var disposeCalled = false;
@@ -218,7 +218,6 @@ public void TransactionInsertWithRollBackTask()
 						s.Save(person);
 						ForceEscalationToDistributedTx.Escalate(true); //will rollback tx
 						person.CreatedAt = DateTime.Now;
-						s.Update(person);
 					}
 					txscope.Complete();
 				}
@@ -259,7 +258,6 @@ public void TransactionInsertLoadWithRollBackTask()
 					{
 						var person = s.Get<Person>(savedId);
 						person.CreatedAt = createdAt.AddMonths(-1);
-						s.Update(person);
 					}
 					ForceEscalationToDistributedTx.Escalate(true);
 
@@ -308,7 +306,6 @@ public void TransactionInsertLoadFlushWithRollBackTask()
 					{
 						var person = s.Get<Person>(savedId);
 						person.CreatedAt = createdAt.AddMonths(-1);
-						s.Update(person);
 						s.Flush();
 					}
 					ForceEscalationToDistributedTx.Escalate(true);
@@ -343,7 +340,7 @@ public void MultiThreadedTransaction()
 			{
 				delegate
 					{
-						Can_roll_back_transaction();
+						CanRollbackTransaction();
 						_totalCall++;
 					},
 				delegate

src/NHibernate/AdoNet/ConnectionManager.cs

@@ -155,6 +155,7 @@ private void CloseConnection()
 		{
 			Factory.ConnectionProvider.CloseConnection(_connection);
 			_connection = null;
+			_connectionAmbientTransaction = null;
 		}
 
 		public DbConnection GetConnection()
@@ -378,6 +379,7 @@ public void EnlistIfRequired()
 			// Probable case: nested transaction scope. Supporting this could be done by releasing the
 			// connection instead of enlisting.
 			_session.Connection.EnlistTransaction(tran);
+			_connectionAmbientTransaction = tran;
 		}
 
 		private bool RequireConnectionSwapInDtc