[tests] Update EventLogsPipelineUnitTests referenced issue (#4905)
authorMitchell Hwang <16830051+mdh1418@users.noreply.github.com>
Thu, 5 Sep 2024 00:44:12 +0000 (20:44 -0400)
committerGitHub <noreply@github.com>
Thu, 5 Sep 2024 00:44:12 +0000 (17:44 -0700)
Some EventLogsPipelineUnitTests originally disabled for Windows began
failing on other platforms. They were additionally disabled for all
platforms, referencing issue
https://github.com/dotnet/diagnostics/issues/4718.

https://github.com/dotnet/diagnostics/issues/4718 was closed, opting to
use https://github.com/dotnet/diagnostics/issues/2541 as the main
tracking issue.

Update the issue reference to reflect this.

src/tests/Microsoft.Diagnostics.Monitoring.EventPipe/EventLogsPipelineUnitTests.cs

index 7cc92be71c7047456bd74e7f385a442e01b00388..c8ed1e4f249527c8b9ff788254547f0197a84780 100644 (file)
@@ -68,14 +68,9 @@ namespace Microsoft.Diagnostics.Monitoring.EventPipe.UnitTests
         /// <summary>
         /// Test that log events at or above the default level are collected.
         /// </summary>
-        [SkippableTheory(Skip = "https://github.com/dotnet/diagnostics/issues/4718"), MemberData(nameof(Configurations))]
+        [SkippableTheory(Skip = "https://github.com/dotnet/diagnostics/issues/2541"), MemberData(nameof(Configurations))]
         public async Task TestLogsAllCategoriesDefaultLevel(TestConfiguration config)
         {
-            if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
-            {
-                throw new SkipTestException("https://github.com/dotnet/diagnostics/issues/2541");
-            }
-
             using Stream outputStream = await GetLogsAsync(config, settings => {
                 settings.UseAppFilters = false;
                 settings.LogLevel = LogLevel.Warning;
@@ -169,14 +164,9 @@ namespace Microsoft.Diagnostics.Monitoring.EventPipe.UnitTests
         /// Test that log events are collected for the categories and levels specified by the application
         /// and for the categories and levels specified in the filter specs.
         /// </summary>
-        [SkippableTheory(Skip = "https://github.com/dotnet/diagnostics/issues/4718"), MemberData(nameof(Configurations))]
+        [SkippableTheory(Skip = "https://github.com/dotnet/diagnostics/issues/2541"), MemberData(nameof(Configurations))]
         public async Task TestLogsUseAppFiltersAndFilterSpecs(TestConfiguration config)
         {
-            if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
-            {
-                throw new SkipTestException("https://github.com/dotnet/diagnostics/issues/2541");
-            }
-
             using Stream outputStream = await GetLogsAsync(config, settings => {
                 settings.FilterSpecs = new Dictionary<string, LogLevel?>()
                 {