Set test log output via environment variable

src/NzbDrone.Test.Common/LoggingTest.cs

@@ -1,6 +1,8 @@
 using NLog;
 using NLog.Config;
 using NLog.Targets;
+using System;
+using System.IO;
 using NUnit.Framework;
 using NUnit.Framework.Interfaces;
 using NzbDrone.Common.EnvironmentInfo;
@@ -20,9 +22,19 @@ protected static void InitLogging()
             if (LogManager.Configuration == null || LogManager.Configuration.AllTargets.None(c => c is ExceptionVerification))
             {
                 LogManager.Configuration = new LoggingConfiguration();
-                var consoleTarget = new ConsoleTarget { Layout = "${level}: ${message} ${exception}" };
-                LogManager.Configuration.AddTarget(consoleTarget.GetType().Name, consoleTarget);
-                LogManager.Configuration.LoggingRules.Add(new LoggingRule("*", LogLevel.Trace, consoleTarget));
+
+                var logOutput = TestLogOutput.Console;
+                Enum.TryParse<TestLogOutput>(Environment.GetEnvironmentVariable("LIDARR_TESTS_LOG_OUTPUT"), out logOutput);
+
+                switch (logOutput)
+                {
+                    case TestLogOutput.Console:
+                        RegisterConsoleLogger();
+                        break;
+                    case TestLogOutput.File:
+                        RegisterFileLogger();
+                        break;
+                }
 
                 RegisterExceptionVerification();
 
@@ -30,6 +42,32 @@ protected static void InitLogging()
             }
         }
 
+        private static void RegisterConsoleLogger()
+        {
+            var consoleTarget = new ConsoleTarget { Layout = "${level}: ${message} ${exception}" };
+            LogManager.Configuration.AddTarget(consoleTarget.GetType().Name, consoleTarget);
+            LogManager.Configuration.LoggingRules.Add(new LoggingRule("*", LogLevel.Trace, consoleTarget));
+        }
+
+        private static void RegisterFileLogger()
+        {
+            const string layout = @"${level}|${message}${onexception:inner=${newline}${newline}${exception:format=ToString}${newline}}";
+
+            var fileTarget = new FileTarget();
+
+            fileTarget.Name = "Test File Logger";
+            fileTarget.FileName = Path.Combine(TestContext.CurrentContext.WorkDirectory, "TestLog.txt");
+            fileTarget.AutoFlush = false;
+            fileTarget.KeepFileOpen = true;
+            fileTarget.ConcurrentWrites = true;
+            fileTarget.ConcurrentWriteAttemptDelay = 50;
+            fileTarget.ConcurrentWriteAttempts = 10;
+            fileTarget.Layout = layout;
+
+            LogManager.Configuration.AddTarget(fileTarget.GetType().Name, fileTarget);
+            LogManager.Configuration.LoggingRules.Add(new LoggingRule("*", LogLevel.Trace, fileTarget));
+        }
+
         private static void RegisterExceptionVerification()
         {
             var exceptionVerification = new ExceptionVerification();
@@ -42,6 +80,7 @@ public void LoggingTestSetup()
         {
             InitLogging();
             ExceptionVerification.Reset();
+            TestLogger.Info("--- Start: {0} ---", TestContext.CurrentContext.Test.FullName);
         }
 
         [TearDown]
@@ -53,6 +92,8 @@ public void LoggingDownBase()
             {
                 ExceptionVerification.AssertNoUnexpectedLogs();
             }
+
+            TestLogger.Info("--- End: {0} ---", TestContext.CurrentContext.Test.FullName);
         }
     }
 }

src/NzbDrone.Test.Common/NzbDrone.Test.Common.csproj

@@ -107,6 +107,7 @@
     <Compile Include="StringExtensions.cs" />
     <Compile Include="TestBase.cs" />
     <Compile Include="TestException.cs" />
+    <Compile Include="TestLogOutput.cs" />
     <Compile Include="TestValidator.cs" />
   </ItemGroup>
   <ItemGroup>

src/NzbDrone.Test.Common/TestLogOutput.cs

@@ -0,0 +1,9 @@
+namespace NzbDrone.Test.Common
+{
+    public enum TestLogOutput
+    {
+        Console = 0,
+        File = 1,
+        None = 2
+    }
+}

test.sh

@@ -4,14 +4,20 @@ WHERE="cat != ManualTest"
 TEST_DIR="."
 TEST_PATTERN="*Test.dll"
 ASSEMBLIES=""
+TEST_LOG_FILE="TestLog.txt"
 
 if [ -d "$TEST_DIR/_tests" ]; then
   TEST_DIR="$TEST_DIR/_tests"
 fi
 
+rm -f "$TEST_LOG_FILE"
+
+# Uncomment to log test output to a file instead of the console
+# export LIDARR_TESTS_LOG_OUTPUT="File"
+
 NUNIT="$TEST_DIR/NUnit.ConsoleRunner.3.2.0/tools/nunit3-console.exe"
 NUNIT_COMMAND="$NUNIT"
-NUNIT_PARAMS="--teamcity"
+NUNIT_PARAMS="--teamcity --workers=1"
 
 if [ "$PLATFORM" = "Windows" ]; then
   WHERE="$WHERE && cat != LINUX"