[Helix] Reenable dotnet-watch on helix (#8258)

Author: HaoK

src/Tools/dotnet-watch/test/DotNetWatcherTests.cs

@@ -6,6 +6,7 @@
 using System.IO;
 using System.Globalization;
 using System.Threading.Tasks;
+using Microsoft.AspNetCore.Testing.xunit;
 using Xunit;
 using Xunit.Abstractions;
 
@@ -22,7 +23,8 @@ public DotNetWatcherTests(ITestOutputHelper logger)
             _app = new KitchenSinkApp(logger);
         }
 
-        [Fact]
+        [ConditionalFact]
+        [SkipOnHelix] // https://github.com/aspnet/AspNetCore/issues/8267
         public async Task RunsWithDotnetWatchEnvVariable()
         {
             Assert.True(string.IsNullOrEmpty(Environment.GetEnvironmentVariable("DOTNET_WATCH")), "DOTNET_WATCH cannot be set already when this test is running");

src/Tools/dotnet-watch/test/GlobbingAppTests.cs

@@ -6,6 +6,7 @@
 using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
+using Microsoft.AspNetCore.Testing.xunit;
 using Microsoft.DotNet.Watcher.Tools.Tests;
 using Xunit;
 using Xunit.Abstractions;
@@ -20,9 +21,10 @@ public GlobbingAppTests(ITestOutputHelper logger)
             _app = new GlobbingApp(logger);
         }
 
-        [Theory]
+        [ConditionalTheory]
         [InlineData(true)]
         [InlineData(false)]
+        [SkipOnHelix] // https://github.com/aspnet/AspNetCore/issues/8267
         public async Task ChangeCompiledFile(bool usePollingWatcher)
         {
             _app.UsePollingWatcher = usePollingWatcher;
@@ -40,7 +42,8 @@ public async Task ChangeCompiledFile(bool usePollingWatcher)
             Assert.Equal(2, types);
         }
 
-        [Fact]
+        [ConditionalFact]
+        [SkipOnHelix] // https://github.com/aspnet/AspNetCore/issues/8267
         public async Task DeleteCompiledFile()
         {
             await _app.StartWatcherAsync();
@@ -56,7 +59,8 @@ public async Task DeleteCompiledFile()
             Assert.Equal(1, types);
         }
 
-        [Fact]
+        [ConditionalFact]
+        [SkipOnHelix] // https://github.com/aspnet/AspNetCore/issues/8267
         public async Task DeleteSourceFolder()
         {
             await _app.StartWatcherAsync();
@@ -84,7 +88,8 @@ public async Task RenameCompiledFile()
             await _app.HasRestarted();
         }
 
-        [Fact]
+        [ConditionalFact]
+        [SkipOnHelix] // https://github.com/aspnet/AspNetCore/issues/8267
         public async Task ChangeExcludedFile()
         {
             await _app.StartWatcherAsync();
@@ -97,7 +102,8 @@ public async Task ChangeExcludedFile()
             Assert.NotSame(restart, finished);
         }
 
-        [Fact]
+        [ConditionalFact]
+        [SkipOnHelix] // https://github.com/aspnet/AspNetCore/issues/8267
         public async Task ListsFiles()
         {
             await _app.PrepareAsync();

src/Tools/dotnet-watch/test/NoDepsAppTests.cs

@@ -5,6 +5,7 @@
 using System.Diagnostics;
 using System.IO;
 using System.Threading.Tasks;
+using Microsoft.AspNetCore.Testing.xunit;
 using Xunit;
 using Xunit.Abstractions;
 
@@ -23,7 +24,8 @@ public NoDepsAppTests(ITestOutputHelper logger)
             _output = logger;
         }
 
-        [Fact]
+        [ConditionalFact]
+        [SkipOnHelix] // https://github.com/aspnet/AspNetCore/issues/8267
         public async Task RestartProcessOnFileChange()
         {
             await _app.StartWatcherAsync(new[] { "--no-exit" });
@@ -41,7 +43,8 @@ public async Task RestartProcessOnFileChange()
             Assert.NotEqual(processIdentifier, processIdentifier2);
         }
 
-        [Fact]
+        [ConditionalFact]
+        [SkipOnHelix] // https://github.com/aspnet/AspNetCore/issues/8267
         public async Task RestartProcessThatTerminatesAfterFileChange()
         {
             await _app.StartWatcherAsync();

src/Tools/dotnet-watch/test/dotnet-watch.Tests.csproj

@@ -5,12 +5,11 @@
     <AssemblyName>Microsoft.DotNet.Watcher.Tools.Tests</AssemblyName>
     <DefaultItemExcludes>$(DefaultItemExcludes);TestProjects\**\*</DefaultItemExcludes>
     <TestGroupName>DotNetWatcherToolsTests</TestGroupName>
-    <!-- https://github.com/aspnet/AspNetCore/issues/6549 -->
-    <BuildHelixPayload>false</BuildHelixPayload>
   </PropertyGroup>
 
   <ItemGroup>
     <Compile Include="$(ToolSharedSourceRoot)TestHelpers\**\*.cs" />
+    <Compile Include="$(SharedSourceRoot)test\SkipOnHelixAttribute.cs" />
     <Content Include="TestProjects\**\*" CopyToOutputDirectory="PreserveNewest" />
   </ItemGroup>