Add tests for legacy logger

For issue #1478

Author: ngbrown

src/NHibernate.Test/Logging/LoggerProviderTest.cs

@@ -1,3 +1,4 @@
+using System;
 using NUnit.Framework;
 
 namespace NHibernate.Test.Logging
@@ -12,17 +13,29 @@ public void LoggerProviderCanCreateLoggers()
 			Assert.That(NHibernateLogger.For(typeof (LoggerProviderTest)), Is.Not.Null);
 		}
 
+		[Test, Obsolete]
+		public void LoggerProviderCanCreateLoggers_Obsolete()
+		{
+			Assert.That(LoggerProvider.LoggerFor("pizza"), Is.Not.Null);
+			Assert.That(LoggerProvider.LoggerFor(typeof (LoggerProviderTest)), Is.Not.Null);
+		}
+
 		[Test]
 		public void WhenNotConfiguredAndLog4NetExistsThenUseLog4NetFactory()
 		{
-#pragma warning disable 618
-			Assert.That(NHibernateLogger.For("pizza"), Is.Not.InstanceOf<NoLoggingInternalLogger>());
-#pragma warning restore 618
-
 			// NoLoggingNHibernateLogger is internal
 			Assert.That(NHibernateLogger.For("pizza").GetType().Name, Is.Not.EqualTo("NoLoggingNHibernateLogger"));
 		}
 
+		[Test, Obsolete]
+		public void WhenNotConfiguredAndLog4NetExistsThenUseLog4NetFactory_Obsolete()
+		{
+			Assert.That(LoggerProvider.LoggerFor("pizza"), Is.Not.InstanceOf<NoLoggingInternalLogger>());
+
+			// works because this is the legacy provider with a legacy logger
+			Assert.That(LoggerProvider.LoggerFor("pizza"), Is.InstanceOf<Log4NetLogger>());
+		}
+
 		[Test, Explicit("Changes global state.")]
 		public void WhenConfiguredAsNullThenNoLoggingFactoryIsUsed()
 		{
@@ -31,5 +44,32 @@ public void WhenConfiguredAsNullThenNoLoggingFactoryIsUsed()
 			// NoLoggingNHibernateLogger is internal
 			Assert.That(NHibernateLogger.For("pizza").GetType().Name, Is.EqualTo("NoLoggingNHibernateLogger"));
 		}
+
+		[Test, Explicit("Changes global state."), Obsolete]
+		public void WhenConfiguredAsNullThenNoLoggingFactoryIsUsed_Obsolete()
+		{
+			NHibernateLogger.SetLoggersFactory(default(INHibernateLoggerFactory));
+
+			// NoLoggingNHibernateLogger is internal
+			Assert.That(LoggerProvider.LoggerFor("pizza"), Is.InstanceOf<NoLoggingInternalLogger>());
+		}
+
+		[Test, Explicit("Changes global state."), Obsolete]
+		public void WhenNoLoggingFactoryIsUsedThenNoLoggingInternalLoggerIsReturned()
+		{
+			LoggerProvider.SetLoggersFactory(new NoLoggingLoggerFactory());
+
+			// NoLoggingNHibernateLogger is internal
+			Assert.That(LoggerProvider.LoggerFor("pizza"), Is.InstanceOf<NoLoggingInternalLogger>());
+		}
+
+		[Test, Explicit("Changes global state."), Obsolete]
+		public void WhenNoLoggingFactoryIsUsedThenNoLoggingNHibernateLoggerIsReturned()
+		{
+			LoggerProvider.SetLoggersFactory(new NoLoggingLoggerFactory());
+
+			// NoLoggingNHibernateLogger is internal
+			Assert.That(NHibernateLogger.For("pizza").GetType().Name, Is.EqualTo("NoLoggingNHibernateLogger"));
+		}
 	}
 }