Clean-up IObjectsFactory usages

IObjectsFactory is meant for instantiating NHibernate dependencies, but
it was used in some cases for instantiating value types. Some of its
methods are also hard or impossible to implement with many dependency
injection frameworks.

Follow up to #1758

Author: fredericDelaporte

src/NHibernate.Test/Bytecode/ActivatorObjectFactoryFixture.cs

@@ -45,7 +45,7 @@ public void CreateInstanceDefCtor()
 
 
 
-		[Test]
+		[Test, Obsolete]
 		public void CreateInstanceWithNoPublicCtor()
 		{
 			IObjectsFactory of = GetObjectsFactory();
@@ -55,7 +55,7 @@ public void CreateInstanceWithNoPublicCtor()
 			Assert.That(instance, Is.InstanceOf<WithOutPublicParameterLessCtor>());
 		}
 
-		[Test]
+		[Test, Obsolete]
 		public void CreateInstanceOfValueType()
 		{
 			IObjectsFactory of = GetObjectsFactory();
@@ -64,7 +64,7 @@ public void CreateInstanceOfValueType()
 			Assert.That(instance, Is.InstanceOf<ValueType>());
 		}
 
-		[Test]
+		[Test, Obsolete]
 		public void CreateInstanceWithArguments()
 		{
 			IObjectsFactory of = GetObjectsFactory();
@@ -76,4 +76,4 @@ public void CreateInstanceWithArguments()
 			Assert.That(((WithOutPublicParameterLessCtor)instance).Something, Is.EqualTo(value));
 		}
 	}
-}
+}

src/NHibernate.Test/ConnectionTest/MapBasedSessionContextFixture.cs

@@ -2,7 +2,6 @@
 using System.Threading;
 using NHibernate.Cfg;
 using NHibernate.Context;
-using NHibernate.Engine;
 using NUnit.Framework;
 
 namespace NHibernate.Test.ConnectionTest
@@ -59,8 +58,6 @@ public void MapContextThreadSafety()
 
 	public class TestableMapBasedSessionContext : MapBasedSessionContext
 	{
-		public TestableMapBasedSessionContext(ISessionFactoryImplementor factory) : base(factory) { }
-
 		// Context is the app with such implementation. Just for the test case.
 		internal static IDictionary _map;
 
@@ -76,4 +73,4 @@ protected override void SetMap(IDictionary value)
 			_map = value;
 		}
 	}
-}
+}

src/NHibernate.Test/ConnectionTest/ThreadLocalCurrentSessionTest.cs

@@ -1,6 +1,5 @@
 using NHibernate.Cfg;
 using NHibernate.Context;
-using NHibernate.Engine;
 using NUnit.Framework;
 
 namespace NHibernate.Test.ConnectionTest
@@ -71,8 +70,7 @@ public class TestableThreadLocalContext : ThreadLocalSessionContext
 	{
 		private static TestableThreadLocalContext me;
 
-		public TestableThreadLocalContext(ISessionFactoryImplementor factory)
-			: base(factory)
+		public TestableThreadLocalContext()
 		{
 			me = this;
 		}
@@ -88,4 +86,4 @@ public static bool HasBind()
 			return context != null && context.ContainsKey(me.factory);
 		}
 	}
-}
+}

src/NHibernate/Async/Context/ThreadLocalSessionContext.cs

@@ -11,14 +11,13 @@
 using System;
 using System.Collections.Generic;
 
-using NHibernate;
 using NHibernate.Engine;
 
 namespace NHibernate.Context
 {
 	using System.Threading.Tasks;
 	using System.Threading;
-	public partial class ThreadLocalSessionContext : ICurrentSessionContext
+	public partial class ThreadLocalSessionContext : ICurrentSessionContextWithFactory
 	{
 
 		private static async Task CleanupAnyOrphanedSessionAsync(ISessionFactory factory, CancellationToken cancellationToken)

src/NHibernate/Async/Tool/hbm2ddl/SchemaExport.cs

@@ -187,10 +187,7 @@ private async Task ExecuteInitializedAsync(Action<string> scriptAction, bool exe
 			cancellationToken.ThrowIfCancellationRequested();
 			if (dialect.SupportsSqlBatches)
 			{
-				var objFactory = Environment.ObjectsFactory;
-				ScriptSplitter splitter = (ScriptSplitter)objFactory.CreateInstance(typeof(ScriptSplitter), sql);
-
-				foreach (string stmt in splitter)
+				foreach (var stmt in new ScriptSplitter(sql))
 				{
 					log.Debug("SQL Batch: {0}", stmt);
 					cmd.CommandText = stmt;

src/NHibernate/Bytecode/ActivatorObjectsFactory.cs

@@ -9,14 +9,18 @@ public object CreateInstance(System.Type type)
 			return Activator.CreateInstance(type);
 		}
 
+		// Since v5.2
+		[Obsolete("This method has no more usages and will be removed in a future version")]
 		public object CreateInstance(System.Type type, bool nonPublic)
 		{
 			return Activator.CreateInstance(type, nonPublic);
 		}
 
+		// Since v5.2
+		[Obsolete("This method has no more usages and will be removed in a future version")]
 		public object CreateInstance(System.Type type, params object[] ctorArgs)
 		{
 			return Activator.CreateInstance(type, ctorArgs);
 		}
 	}
-}
+}

src/NHibernate/Bytecode/IObjectsFactory.cs

@@ -1,7 +1,9 @@
+using System;
+
 namespace NHibernate.Bytecode
 {
 	/// <summary>
-	/// Interface for instantiate all NHibernate objects.
+	/// Interface for instantiating NHibernate dependencies.
 	/// </summary>
 	public interface IObjectsFactory
 	{
@@ -18,6 +20,8 @@ public interface IObjectsFactory
 		/// <param name="type">The type of object to create.</param>
 		/// <param name="nonPublic">true if a public or nonpublic default constructor can match; false if only a public default constructor can match.</param>
 		/// <returns>A reference to the created object.</returns>
+		// Since v5.2
+		[Obsolete("This method has no more usages and will be removed in a future version")]
 		object CreateInstance(System.Type type, bool nonPublic);
 
 		/// <summary>
@@ -27,6 +31,8 @@ public interface IObjectsFactory
 		/// <param name="type">The type of object to create.</param>
 		/// <param name="ctorArgs">An array of constructor arguments.</param>
 		/// <returns>A reference to the created object.</returns>
+		// Since v5.2
+		[Obsolete("This method has no more usages and will be removed in a future version")]
 		object CreateInstance(System.Type type, params object[] ctorArgs);
 	}
-}
+}

src/NHibernate/Context/AsyncLocalSessionContext.cs

@@ -21,13 +21,16 @@ public class AsyncLocalSessionContext : CurrentSessionContext
 	{
 		private readonly AsyncLocal<ISession> _session = new AsyncLocal<ISession>();
 
-		// Constructor signature required for dynamic invocation code.
+		// Since v5.2
+		[Obsolete("This constructor has no more usages and will be removed in a future version")]
 		public AsyncLocalSessionContext(ISessionFactoryImplementor factory) { }
 
+		public AsyncLocalSessionContext() { }
+
 		protected override ISession Session
 		{
 			get => _session.Value;
 			set => _session.Value = value;
 		}
 	}
-}
+}

src/NHibernate/Context/CallSessionContext.cs

@@ -27,10 +27,14 @@ public class CallSessionContext : MapBasedSessionContext
 		private static readonly AsyncLocal<IDictionary> SessionFactoryMap = new AsyncLocal<IDictionary>();
 		#endif
 
+		// Since v5.2
+		[Obsolete("This constructor has no more usages and will be removed in a future version")]
 		public CallSessionContext(ISessionFactoryImplementor factory) : base(factory)
 		{
 		}
 
+		public CallSessionContext() { }
+
 		/// <summary>
 		/// The key is the session factory and the value is the bound session.
 		/// </summary>

src/NHibernate/Context/CurrentSessionContext.cs

@@ -16,7 +16,7 @@ namespace NHibernate.Context
 	/// through <see cref="ISessionFactory.GetCurrentSession()"/> calls.
 	/// </remarks>
 	[Serializable]
-	public abstract class CurrentSessionContext : ICurrentSessionContext
+	public abstract class CurrentSessionContext : ICurrentSessionContextWithFactory
 	{
 		/// <summary> Gets or sets the currently bound session. </summary>
 		protected abstract ISession Session { get; set; }
@@ -37,6 +37,12 @@ public virtual ISession CurrentSession()
 			return Session;
 		}
 
+		/// <inheritdoc />
+		public virtual void SetFactory(ISessionFactoryImplementor factory)
+		{
+			// No-op by default.
+		}
+
 		/// <summary>
 		/// Binds the specified session to the current context.
 		/// </summary>
@@ -86,4 +92,4 @@ private static CurrentSessionContext GetCurrentSessionContext(ISessionFactory fa
 			return currentSessionContext;
 		}
 	}
-}
+}