fix indent issue from file scoped namespace change (#1370)

Author: SimonCropp

test/Sentry.Extensions.Logging.Tests/MelDiagnosticLoggerTests.cs

@@ -1,5 +1,3 @@
-#if NETCOREAPP2_1 || NET461
-#endif
 using Microsoft.Extensions.Logging;
 
 namespace Sentry.Extensions.Logging.Tests;
@@ -43,25 +41,25 @@ public void LogLevel_HigherLevel_IsEnabled()
     // .NET Core 3 (and hence .NET 5) has turned FormattedLogValues into an internal readonly struct
     // and now we can't match that with NSubstitute
 #if NETCOREAPP2_1 || NET461
-        [Fact]
-        public void Log_PassedThrough()
-        {
-            const SentryLevel expectedLevel = SentryLevel.Debug;
-            const string expectedMessage = "test";
-            var expectedException = new Exception();
+    [Fact]
+    public void Log_PassedThrough()
+    {
+        const SentryLevel expectedLevel = SentryLevel.Debug;
+        const string expectedMessage = "test";
+        var expectedException = new Exception();
 
-            _fixture.Level = SentryLevel.Debug;
-            var sut = _fixture.GetSut();
-            _ = _fixture.MelLogger.IsEnabled(Arg.Any<LogLevel>()).Returns(true);
+        _fixture.Level = SentryLevel.Debug;
+        var sut = _fixture.GetSut();
+        _ = _fixture.MelLogger.IsEnabled(Arg.Any<LogLevel>()).Returns(true);
 
-            sut.Log(expectedLevel, expectedMessage, expectedException);
+        sut.Log(expectedLevel, expectedMessage, expectedException);
 
-            _fixture.MelLogger.Received(1).Log(
-                expectedLevel.ToMicrosoft(),
-                0,
-                Arg.Is<object>(e => e.ToString() == expectedMessage),
-                expectedException,
-                Arg.Any<Func<object, Exception, string>>());
-        }
+        _fixture.MelLogger.Received(1).Log(
+            expectedLevel.ToMicrosoft(),
+            0,
+            Arg.Is<object>(e => e.ToString() == expectedMessage),
+            expectedException,
+            Arg.Any<Func<object, Exception, string>>());
+    }
 #endif
 }

test/Sentry.Tests/Internals/Http/HttpTransportTests.cs

@@ -32,13 +32,13 @@ public async Task SendEnvelopeAsync_CancellationToken_PassedToClient()
 #if NET5_0_OR_GREATER
         await Assert.ThrowsAsync<TaskCanceledException>(() => httpTransport.SendEnvelopeAsync(envelope, token));
 #else
-            // Act
-            await httpTransport.SendEnvelopeAsync(envelope, token);
+        // Act
+        await httpTransport.SendEnvelopeAsync(envelope, token);
 
-            // Assert
-            await httpHandler
-                .Received(1)
-                .VerifiableSendAsync(Arg.Any<HttpRequestMessage>(), Arg.Is<CancellationToken>(c => c.IsCancellationRequested));
+        // Assert
+        await httpHandler
+            .Received(1)
+            .VerifiableSendAsync(Arg.Any<HttpRequestMessage>(), Arg.Is<CancellationToken>(c => c.IsCancellationRequested));
 #endif
     }
 

test/Sentry.Tests/Internals/JsonTests.cs

@@ -202,8 +202,8 @@ public void WriteDynamicValue_ClassWithTimeZone_SerializedClassWithTimeZoneInfo(
 #if NET6_0
             "\"BaseUtcOffset\":\"02:00:00\"",
 #else
-                "\"BaseUtcOffset\":{\"Ticks\":72000000000,\"Days\":0,\"Hours\":2,\"Milliseconds\":0,\"Minutes\":0,\"Seconds\":0",
-                "\"TotalHours\":2,\"TotalMilliseconds\":7200000,\"TotalMinutes\":120,\"TotalSeconds\":7200},",
+            "\"BaseUtcOffset\":{\"Ticks\":72000000000,\"Days\":0,\"Hours\":2,\"Milliseconds\":0,\"Minutes\":0,\"Seconds\":0",
+            "\"TotalHours\":2,\"TotalMilliseconds\":7200000,\"TotalMinutes\":120,\"TotalSeconds\":7200},",
 #endif
         };
         var data = new DataWithSerializableObject<TimeZoneInfo>(timeZone);

test/Sentry.Tests/PlatformAbstractions/RuntimeInfoTests.cs

@@ -41,31 +41,31 @@ public void Parse_TestCases(ParseTestCase parseTestCase)
     }
 
 #if NET461
-        [SkippableFact]
-        public void SetReleaseAndVersionNetFx_OnNetFx_NonNullReleaseAndVersion()
-        {
-            // This test is only relevant when running on CLR.
-            Skip.If(RuntimeInfo.GetRuntime().IsMono());
+    [SkippableFact]
+    public void SetReleaseAndVersionNetFx_OnNetFx_NonNullReleaseAndVersion()
+    {
+        // This test is only relevant when running on CLR.
+        Skip.If(RuntimeInfo.GetRuntime().IsMono());
 
-            var input = new Runtime(".NET Framework");
-            RuntimeInfo.SetNetFxReleaseAndVersion(input);
+        var input = new Runtime(".NET Framework");
+        RuntimeInfo.SetNetFxReleaseAndVersion(input);
 
-            Assert.NotNull(input.Version);
-            Assert.NotNull(input.FrameworkInstallation);
-            Assert.NotNull(input.FrameworkInstallation.Version);
-        }
+        Assert.NotNull(input.Version);
+        Assert.NotNull(input.FrameworkInstallation);
+        Assert.NotNull(input.FrameworkInstallation.Version);
+    }
 #endif
 
 #if NETCOREAPP2_1 || NETCOREAPP3_0
-        [Fact]
-        public void SetNetCoreVersion_NetCoreAsName()
-        {
-            var input = new Runtime(".NET Core");
-            RuntimeInfo.SetNetCoreVersion(input);
+    [Fact]
+    public void SetNetCoreVersion_NetCoreAsName()
+    {
+        var input = new Runtime(".NET Core");
+        RuntimeInfo.SetNetCoreVersion(input);
 
-            Assert.NotNull(input.Version);
-            Assert.Equal(".NET Core", input.Name);
-        }
+        Assert.NotNull(input.Version);
+        Assert.Equal(".NET Core", input.Name);
+    }
 #endif
 
 #if NET5_0_OR_GREATER

test/Sentry.Tests/SentryOptionsExtensionsTests.cs

@@ -17,15 +17,15 @@ public void DisableDuplicateEventDetection_RemovesDisableDuplicateEventDetection
     }
 
 #if NET461
-        [Fact]
-        public void DisableNetFxInstallationsEventProcessor_RemovesDisableNetFxInstallationsEventProcessorEventProcessor()
-        {
-            Sut.DisableNetFxInstallationsIntegration();
-            Assert.DoesNotContain(Sut.EventProcessors,
-                p => p.GetType() == typeof(NetFxInstallationsEventProcessor));
-            Assert.DoesNotContain(Sut.Integrations,
-                p => p.GetType() == typeof(NetFxInstallationsIntegration));
-        }
+    [Fact]
+    public void DisableNetFxInstallationsEventProcessor_RemovesDisableNetFxInstallationsEventProcessorEventProcessor()
+    {
+        Sut.DisableNetFxInstallationsIntegration();
+        Assert.DoesNotContain(Sut.EventProcessors,
+            p => p.GetType() == typeof(NetFxInstallationsEventProcessor));
+        Assert.DoesNotContain(Sut.Integrations,
+            p => p.GetType() == typeof(NetFxInstallationsIntegration));
+    }
 #endif
 
     [Fact]

test/Sentry.Tests/SentryOptionsTests.cs

@@ -24,22 +24,22 @@ public void RequestBodyCompressionLevel_ByDefault_Optimal()
     }
 
 #if NET461
-        [SkippableFact(typeof(IsTypeException))]
-        public void StackTraceFactory_RunningOnMono_HasMonoStackTraceFactory()
-        {
-            Skip.If(!RuntimeInfo.GetRuntime().IsMono());
+    [SkippableFact(typeof(IsTypeException))]
+    public void StackTraceFactory_RunningOnMono_HasMonoStackTraceFactory()
+    {
+        Skip.If(!RuntimeInfo.GetRuntime().IsMono());
 
-            var sut = new SentryOptions();
-            Assert.IsType<MonoSentryStackTraceFactory>(sut.SentryStackTraceFactory);
-        }
+        var sut = new SentryOptions();
+        Assert.IsType<MonoSentryStackTraceFactory>(sut.SentryStackTraceFactory);
+    }
 
-        [SkippableFact(typeof(IsNotTypeException))]
-        public void StackTraceFactory_NotRunningOnMono_NotMonoStackTraceFactory()
-        {
-            Skip.If(RuntimeInfo.GetRuntime().IsMono());
+    [SkippableFact(typeof(IsNotTypeException))]
+    public void StackTraceFactory_NotRunningOnMono_NotMonoStackTraceFactory()
+    {
+        Skip.If(RuntimeInfo.GetRuntime().IsMono());
 
-            var sut = new SentryOptions();
-            Assert.IsNotType<MonoSentryStackTraceFactory>(sut.SentryStackTraceFactory);
-        }
+        var sut = new SentryOptions();
+        Assert.IsNotType<MonoSentryStackTraceFactory>(sut.SentryStackTraceFactory);
+    }
 #endif
 }

test/Sentry.Tests/TaskUnobservedTaskExceptionIntegrationTests.cs

@@ -29,38 +29,38 @@ public void Handle_WithException_CaptureEvent()
 
     // Only triggers in release mode.
 #if RELEASE
-        [Fact] // Integration test.
-        public void Handle_UnobservedTaskException_CaptureEvent()
+    [Fact] // Integration test.
+    public void Handle_UnobservedTaskException_CaptureEvent()
+    {
+        _fixture.AppDomain = AppDomainAdapter.Instance;
+        var captureCalledEvent = new System.Threading.ManualResetEvent(false);
+        _fixture.Hub.When(x => x.CaptureEvent(Arg.Any<SentryEvent>()))
+            .Do(_ => captureCalledEvent.Set());
+
+        var sut = _fixture.GetSut();
+        sut.Register(_fixture.Hub, SentryOptions);
+        try
         {
-            _fixture.AppDomain = AppDomainAdapter.Instance;
-            var captureCalledEvent = new System.Threading.ManualResetEvent(false);
-            _fixture.Hub.When(x => x.CaptureEvent(Arg.Any<SentryEvent>()))
-                .Do(_ => captureCalledEvent.Set());
-
-            var sut = _fixture.GetSut();
-            sut.Register(_fixture.Hub, SentryOptions);
-            try
+            var taskStartedEvent = new System.Threading.ManualResetEvent(false);
+            _ = Task.Run(() =>
             {
-                var taskStartedEvent = new System.Threading.ManualResetEvent(false);
-                _ = Task.Run(() =>
-                {
-                    _ = taskStartedEvent.Set();
-                    throw new Exception("Unhandled on Task");
-                });
-                Assert.True(taskStartedEvent.WaitOne(TimeSpan.FromSeconds(4)));
-                var counter = 0;
-                do
-                {
-                    Assert.True(counter++ < 10);
-                    GC.Collect();
-                    GC.WaitForPendingFinalizers();
-                } while (!captureCalledEvent.WaitOne(TimeSpan.FromMilliseconds(100)));
-            }
-            finally
+                _ = taskStartedEvent.Set();
+                throw new Exception("Unhandled on Task");
+            });
+            Assert.True(taskStartedEvent.WaitOne(TimeSpan.FromSeconds(4)));
+            var counter = 0;
+            do
             {
-                sut.Unregister(_fixture.Hub);
-            }
+                Assert.True(counter++ < 10);
+                GC.Collect();
+                GC.WaitForPendingFinalizers();
+            } while (!captureCalledEvent.WaitOne(TimeSpan.FromMilliseconds(100)));
         }
+        finally
+        {
+            sut.Unregister(_fixture.Hub);
+        }
+    }
 #endif
 
     [Fact]