Fix Invalid Console Handle

.github/workflows/dev-build-windows.yml

@@ -0,0 +1,37 @@
+name: Dev build - Windows
+
+on:
+  push:
+    branches:
+    - dev
+  pull_request:
+    branches: 
+    - dev
+
+jobs:
+  build:
+    runs-on: windows-latest
+    strategy:
+      matrix:
+        dotnet-version: ["7.0.x"]
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Setup .NET Code SDK ${{ matrix.dotnet-version }}
+      uses: actions/setup-dotnet@v3
+      with:
+        dotnet-version: ${{ matrix.dotnet-version }}
+    - name: Install dependencies
+      run: dotnet restore
+    - name: Build package sources
+      run: dotnet build -c Release --no-restore
+    - name: Build samples
+      run: dotnet build ./examples/solution.sln
+    - name: Test
+      run: dotnet test --no-restore --collect:"XPlat Code Coverage"
+    - name: Publish code coverage
+      uses: codecov/codecov-action@v1
+      with:
+        files: "**/coverage.cobertura.xml"
+        flags: unittests
+        name: vertical-spectreconsolelogger-codecov

.github/workflows/dev-build.yml

@@ -4,6 +4,9 @@ on:
   push:
     branches:
     - dev
+  pull_request:
+    branches: 
+    - dev
 
 jobs:
   build:

src/Output/ConsoleWriter.cs

@@ -41,7 +41,10 @@
         /// </summary>
         protected void ResetLineCore()
         {
-            AnsiConsole.Cursor.Move(CursorDirection.Left, Console.CursorLeft);
+            if (!Console.IsOutputRedirected)
+            {
+                AnsiConsole.Cursor.Move(CursorDirection.Left, Console.CursorLeft);
+            }
         }
     }
 }

test/Output/ActualConsoleWriterTests.cs

@@ -0,0 +1,83 @@
+using System;
+using System.Threading.Tasks;
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Logging;
+using Xunit;
+
+namespace Vertical.SpectreLogger.Tests.Output
+{
+    public class ActualConsoleWriterTests
+    {
+        [Fact]
+        public async Task WriteInBackground()
+        {
+            var services = new ServiceCollection()
+                .AddLogging(builder => builder.AddSpectreConsole(
+                    config =>
+                    {
+                        config.WriteInBackground();
+                    }))
+                .BuildServiceProvider();
+
+            var logger = services.GetRequiredService<ILogger<ActualConsoleWriterTests>>();
+
+            logger.LogInformation("Test event successful");
+
+            await services.DisposeAsync();
+        }
+
+        [Fact]
+        public async Task WriteInForeground()
+        {
+            var services = new ServiceCollection()
+                .AddLogging(builder => builder.AddSpectreConsole(
+                    config =>
+                    {
+                        config.WriteInForeground();
+                    }))
+                .BuildServiceProvider();
+
+            var logger = services.GetRequiredService<ILogger<ActualConsoleWriterTests>>();
+
+            logger.LogInformation("Test event successful");
+
+            await services.DisposeAsync();
+        }
+
+        [Fact]
+        public async Task WriteErrorInBackground()
+        {
+            var services = new ServiceCollection()
+                .AddLogging(builder => builder.AddSpectreConsole(
+                    config =>
+                    {
+                        config.WriteInBackground();
+                    }))
+                .BuildServiceProvider();
+
+            var logger = services.GetRequiredService<ILogger<ActualConsoleWriterTests>>();
+
+            logger.LogError(new ArgumentException(), "Oops");
+
+            await services.DisposeAsync();
+        }
+
+        [Fact]
+        public async Task WriteErrorInForeground()
+        {
+            var services = new ServiceCollection()
+                .AddLogging(builder => builder.AddSpectreConsole(
+                    config =>
+                    {
+                        config.WriteInForeground();
+                    }))
+                .BuildServiceProvider();
+
+            var logger = services.GetRequiredService<ILogger<ActualConsoleWriterTests>>();
+
+            logger.LogError(new ArgumentException(), "Oops");
+
+            await services.DisposeAsync();
+        }
+    }
+}