Make some skipped tests flaky (#8795)

ryanbrandenburg · web-flow · commit 7248ecbaf1a5 · 2019-03-27T10:20:16.000-07:00
diff --git a/src/Hosting/test/FunctionalTests/ShutdownTests.cs b/src/Hosting/test/FunctionalTests/ShutdownTests.cs
@@ -28,6 +28,7 @@ public ShutdownTests(ITestOutputHelper output) : base(output) { }
         [ConditionalFact]
         [OSSkipCondition(OperatingSystems.Windows)]
         [OSSkipCondition(OperatingSystems.MacOSX)]
+        [Flaky("https://github.com/aspnet/AspNetCore-Internal/issues/1687", FlakyOn.AzP.Linux)]
         public async Task ShutdownTestRun()
         {
             await ExecuteShutdownTest(nameof(ShutdownTestRun), "Run");
@@ -116,7 +117,6 @@ private async Task ExecuteShutdownTest(string testName, string shutdownMechanic)
             }
         }
 
-
         private static void SendSIGINT(int processId)
         {
             var startInfo = new ProcessStartInfo
diff --git a/src/Servers/Kestrel/test/InMemory.FunctionalTests/ConnectionLimitTests.cs b/src/Servers/Kestrel/test/InMemory.FunctionalTests/ConnectionLimitTests.cs
@@ -147,7 +147,7 @@ public async Task RejectsConnectionsWhenLimitReached()
         }
 
         [Fact]
-        [OSSkipCondition(OperatingSystems.MacOSX, SkipReason = "https://github.com/aspnet/KestrelHttpServer/issues/2282")]
+        [Flaky("https://github.com/aspnet/KestrelHttpServer/issues/2282", FlakyOn.AzP.macOS)]
         public async Task ConnectionCountingReturnsToZero()
         {
             const int count = 100;

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ public ShutdownTests(ITestOutputHelper output) : base(output) { }`
`28`	`28`	`[ConditionalFact]`
`29`	`29`	`[OSSkipCondition(OperatingSystems.Windows)]`
`30`	`30`	`[OSSkipCondition(OperatingSystems.MacOSX)]`
	`31`	`+ [Flaky("https://github.com/aspnet/AspNetCore-Internal/issues/1687", FlakyOn.AzP.Linux)]`
`31`	`32`	`public async Task ShutdownTestRun()`
`32`	`33`	`{`
`33`	`34`	`await ExecuteShutdownTest(nameof(ShutdownTestRun), "Run");`
`@@ -116,7 +117,6 @@ private async Task ExecuteShutdownTest(string testName, string shutdownMechanic)`
`116`	`117`	`}`
`117`	`118`	`}`
`118`	`119`
`119`		`-`
`120`	`120`	`private static void SendSIGINT(int processId)`
`121`	`121`	`{`
`122`	`122`	`var startInfo = new ProcessStartInfo`
Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ public async Task RejectsConnectionsWhenLimitReached()`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`[Fact]`
`150`		`- [OSSkipCondition(OperatingSystems.MacOSX, SkipReason = "https://github.com/aspnet/KestrelHttpServer/issues/2282")]`
	`150`	`+ [Flaky("https://github.com/aspnet/KestrelHttpServer/issues/2282", FlakyOn.AzP.macOS)]`
`151`	`151`	`public async Task ConnectionCountingReturnsToZero()`
`152`	`152`	`{`
`153`	`153`	`const int count = 100;`