Switched the "DebugTarget" to be a "TraceTarget", because the "DebugTarget" doesn't write if compiled in a Release build - tests were failing, and, a DebugTarget isn't that useful in "live" code, so it wasn't helpful, anyways.

Author: ivanpointer

Take2/NuLog.Tests/Integration/Factories/StandardLoggerFactoryIntegrationTests.cs

@@ -135,7 +135,7 @@ public void Should_UseTargetsAndBuildTagRouter()
             var targetConfig = new TargetConfig
             {
                 Name = "debug",
-                Type = "NuLog.Targets.DebugTarget",
+                Type = "NuLog.Targets.TraceTarget",
                 Properties = new Dictionary<string, object>
                 {
                     { "layout", "${Message}" }
@@ -233,7 +233,7 @@ public void Should_SetDispatcherToNewLogger()
                     new TargetConfig
                     {
                         Name = "debug",
-                        Type = "NuLog.Targets.DebugTarget",
+                        Type = "NuLog.Targets.TraceTarget",
                         Properties = new Dictionary<string, object>
                         {
                             { "layout", "${Message}" }
@@ -273,7 +273,7 @@ public void Should_SetMetaDataProviderToNewLogger()
                     new TargetConfig
                     {
                         Name = "debug",
-                        Type = "NuLog.Targets.DebugTarget",
+                        Type = "NuLog.Targets.TraceTarget",
                         Properties = new Dictionary<string, object>
                         {
                             { "layout", "${MyMetaData}" }
@@ -317,7 +317,7 @@ public void Should_SetDefaultTagsToNewLogger()
                     new TargetConfig
                     {
                         Name = "debug",
-                        Type = "NuLog.Targets.DebugTarget",
+                        Type = "NuLog.Targets.TraceTarget",
                         Properties = new Dictionary<string, object>
                         {
                             { "layout", "${Tags}" }
@@ -357,7 +357,7 @@ public void Should_SetDefaultMetaDataToNewLogger()
                     new TargetConfig
                     {
                         Name = "debug",
-                        Type = "NuLog.Targets.DebugTarget",
+                        Type = "NuLog.Targets.TraceTarget",
                         Properties = new Dictionary<string, object>
                         {
                             { "layout", "${MyMetaData}" }

Take2/NuLog.Tests/NuLog.Tests.csproj

@@ -110,13 +110,11 @@
     <Compile Include="Unit\TagRouters\RuleProcessors\RuleProcessorIncludeRulesTests.cs" />
     <Compile Include="Unit\TagRouters\RuleProcessors\RuleProcessorTestsBase.cs" />
     <Compile Include="Unit\TagRouters\TagGroupProcessors\TagGroupProcessorTests.cs" />
-    <Compile Include="Unit\Targets\DebugTargetTests.cs" />
+    <Compile Include="Unit\Targets\TraceTargetTests.cs" />
     <Compile Include="Unit\Targets\LayoutTargetBaseTests.cs" />
     <Compile Include="Unit\Targets\TextFileTargetTests.cs" />
   </ItemGroup>
-  <ItemGroup>
-    <Folder Include="Integration\Targets\" />
-  </ItemGroup>
+  <ItemGroup />
   <ItemGroup>
     <None Include="App.config">
       <SubType>Designer</SubType>

Take2/NuLog.Tests/Unit/Factories/StandardLoggerFactoryUnitTests.cs

@@ -196,7 +196,7 @@ public void Should_CreateMultipleTargets()
                 },new TargetConfig
                 {
                     Name = "dummy",
-                    Type = "NuLog.Targets.DebugTarget, NuLog"
+                    Type = "NuLog.Targets.TraceTarget, NuLog"
                 }
             };
             var config = new Config
@@ -211,7 +211,7 @@ public void Should_CreateMultipleTargets()
             // Validate
             Assert.Equal(2, targets.Count);
             Assert.True(targets.Any(t => typeof(DummyTarget).IsAssignableFrom(t.GetType())));
-            Assert.True(targets.Any(t => typeof(DebugTarget).IsAssignableFrom(t.GetType())));
+            Assert.True(targets.Any(t => typeof(TraceTarget).IsAssignableFrom(t.GetType())));
         }
 
         /// <summary>

…g.Tests/Unit/Targets/DebugTargetTests.cs …g.Tests/Unit/Targets/TraceTargetTests.csTake2/NuLog.Tests/Unit/Targets/DebugTargetTests.cs renamed to Take2/NuLog.Tests/Unit/Targets/TraceTargetTests.cs Take2/NuLog.Tests/Unit/Targets/DebugTargetTests.cs renamed to Take2/NuLog.Tests/Unit/Targets/TraceTargetTests.cs

@@ -12,59 +12,59 @@
 namespace NuLog.Tests.Unit.Targets
 {
     /// <summary>
-    /// Documents (and verifies) the expected behavior of the debug target.
+    /// Documents (and verifies) the expected behavior of the trace target.
     /// </summary>
     [Trait("Category", "Unit")]
-    public class DebugTargetTests : IDisposable
+    public class TraceTargetTests : IDisposable
     {
         private HashSetTraceListener traceListener;
 
-        public DebugTargetTests()
+        public TraceTargetTests()
         {
             this.traceListener = new HashSetTraceListener();
-            Debug.Listeners.Add(this.traceListener);
+            Trace.Listeners.Add(this.traceListener);
         }
 
         public void Dispose()
         {
-            Debug.Listeners.Remove(this.traceListener);
+            Trace.Listeners.Remove(this.traceListener);
             this.traceListener = null;
         }
 
         /// <summary>
-        /// The debug target should write to debug.
+        /// The trace target should write to trace.
         /// </summary>
-        [Fact(DisplayName = "Should_WriteToDebug")]
-        public void Should_WriteToDebug()
+        [Fact(DisplayName = "Should_WriteToTrace")]
+        public void Should_WriteToTrace()
         {
             // Setup
             var logEvent = new LogEvent
             {
-                Message = "Should_WriteToDebug - hello, world!"
+                Message = "Should_WriteToTrace - hello, world!"
             };
             var layout = A.Fake<ILayout>();
             A.CallTo(() => layout.Format(A<LogEvent>.Ignored)).Returns(logEvent.Message);
 
-            var target = new DebugTarget();
+            var target = new TraceTarget();
             target.SetLayout(layout);
 
             // Execute
             target.Write(logEvent);
 
             // Verify
-            Assert.Contains("Should_WriteToDebug - hello, world!", this.traceListener.Messages);
+            Assert.Contains("Should_WriteToTrace - hello, world!", this.traceListener.Messages);
         }
 
         /// <summary>
-        /// The debug target should use the configured layout to format its messages.
+        /// The trace target should use the configured layout to format its messages.
         /// </summary>
         [Fact(DisplayName = "Should_UseLayout")]
         public void Should_UseLayout()
         {
             // Setup
             var layout = A.Fake<ILayout>();
             A.CallTo(() => layout.Format(A<LogEvent>.Ignored)).Returns("Should_UseLayout - formatted");
-            var target = new DebugTarget();
+            var target = new TraceTarget();
             target.SetLayout(layout);
 
             // Execute
@@ -79,14 +79,14 @@ public void Should_UseLayout()
         }
 
         /// <summary>
-        /// The debug target should throw an invalid operation exception when no layout is given, and
+        /// The trace target should throw an invalid operation exception when no layout is given, and
         /// the target is asked to write a log message.
         /// </summary>
         [Fact(DisplayName = "Should_UseLayout")]
         public void Should_ThrowInvalidOperationWithoutLayout()
         {
             // Setup
-            var target = new DebugTarget();
+            var target = new TraceTarget();
 
             // Execute / Verify
             Assert.Throws(typeof(InvalidOperationException), () =>

Take2/NuLog/NuLog.csproj

@@ -94,7 +94,7 @@
     <Compile Include="TagRouters\StandardTagRouter.cs" />
     <Compile Include="TagRouters\TagGroupProcessors\StandardTagGroupProcessor.cs" />
     <Compile Include="Targets\ConsoleTarget.cs" />
-    <Compile Include="Targets\DebugTarget.cs" />
+    <Compile Include="Targets\TraceTarget.cs" />
     <Compile Include="Targets\EventLogShim.cs" />
     <Compile Include="Targets\EventLogTarget.cs" />
     <Compile Include="Targets\IEventLog.cs" />

Take2/NuLog/Targets/DebugTarget.cs Take2/NuLog/Targets/TraceTarget.csTake2/NuLog/Targets/DebugTarget.cs renamed to Take2/NuLog/Targets/TraceTarget.cs

@@ -9,9 +9,9 @@
 namespace NuLog.Targets
 {
     /// <summary>
-    /// A logger target that writes to debug (System.Diagnostics).
+    /// A logger target that writes to trace (System.Diagnostics).
     /// </summary>
-    public class DebugTarget : LayoutTargetBase
+    public class TraceTarget : LayoutTargetBase
     {
         public override void Write(LogEvent logEvent)
         {
@@ -20,7 +20,7 @@ public override void Write(LogEvent logEvent)
             {
                 // Use the layout to format the log event, before writing it to debug.
                 var formatted = this.Layout.Format(logEvent);
-                Debug.Write(formatted);
+                Trace.Write(formatted);
             }
             else
             {