Re-enable telemetry notice tests by removing attribute (#48595)

marcpopMSFT · web-flow · commit 28da9f753656 · 2025-04-22T13:30:47.000-07:00
diff --git a/test/dotnet.Tests/GivenThatTheUserIsRunningDotNetForTheFirstTime.cs b/test/dotnet.Tests/GivenThatTheUserIsRunningDotNetForTheFirstTime.cs
@@ -103,7 +103,7 @@ public void UsingDotnetForTheFirstTimeWithNonVerbsDoesNotPrintEula()
                 .StartWith(firstTimeNonVerbUseMessage);
         }
 
-        [WindowsOnlyFact(Skip="https://github.com/dotnet/sdk/issues/43328")]
+        [WindowsOnlyFact]
         public void ItShowsTheAppropriateMessageToTheUser()
         {
 
@@ -151,7 +151,7 @@ public void ItDoesNotCreateAFirstUseSentinelFileNorAnAspNetCertificateSentinelFi
             homeFolder.Should().NotExist();
         }
 
-        [WindowsOnlyFact(Skip="https://github.com/dotnet/sdk/issues/43328")]
+        [WindowsOnlyFact]
         public void ItShowsTheTelemetryNoticeWhenInvokingACommandAfterInternalReportInstallSuccessHasBeenInvoked()
         {
             var dotnetFirstTime = new DotNetFirstTime();

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ public void UsingDotnetForTheFirstTimeWithNonVerbsDoesNotPrintEula()`
`103`	`103`	`.StartWith(firstTimeNonVerbUseMessage);`
`104`	`104`	`}`
`105`	`105`
`106`		`- [WindowsOnlyFact(Skip="https://github.com/dotnet/sdk/issues/43328")]`
	`106`	`+ [WindowsOnlyFact]`
`107`	`107`	`public void ItShowsTheAppropriateMessageToTheUser()`
`108`	`108`	`{`
`109`	`109`
`@@ -151,7 +151,7 @@ public void ItDoesNotCreateAFirstUseSentinelFileNorAnAspNetCertificateSentinelFi`
`151`	`151`	`homeFolder.Should().NotExist();`
`152`	`152`	`}`
`153`	`153`
`154`		`- [WindowsOnlyFact(Skip="https://github.com/dotnet/sdk/issues/43328")]`
	`154`	`+ [WindowsOnlyFact]`
`155`	`155`	`public void ItShowsTheTelemetryNoticeWhenInvokingACommandAfterInternalReportInstallSuccessHasBeenInvoked()`
`156`	`156`	`{`
`157`	`157`	`var dotnetFirstTime = new DotNetFirstTime();`