Disable system watchdog compat tests on Linux

TODO: #1592
tgstation · Jul 13, 2023 · 64564fa · 64564fa
1 parent f707d78
commit 64564fa
Showing 1 changed file with 13 additions and 12 deletions.
diff --git a/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs b/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs
@@ -1117,18 +1117,19 @@ async Task FailFast(Task task)
 
 					async Task RunInstanceTests()
 					{
-						var compatTests = FailFast(
-							instanceTest
-								.RunCompatTests(
-									// linux 510 has a critical bug where replacing the directory in non-basic watchdogs causes the DreamDaemon cwd to change
-									new PlatformIdentifier().IsWindows || Environment.GetEnvironmentVariable("General__UseBasicWatchdog") == "true"
-										? new Version(510, 1346)
-										: new Version(511, 1385),
-									adminClient.Instances.CreateClient(compatInstance),
-									compatDMPort,
-									compatDDPort,
-									server.HighPriorityDreamDaemon,
-									cancellationToken));
+						// Some earlier linux BYOND versions have a critical bug where replacing the directory in non-basic watchdogs causes the DreamDaemon cwd to change
+						var canRunCompatTests = new PlatformIdentifier().IsWindows || Environment.GetEnvironmentVariable("General__UseBasicWatchdog") == "true";
+						var compatTests = canRunCompatTests
+							? FailFast(
+								instanceTest
+									.RunCompatTests(
+										new Version(510, 1346),
+										adminClient.Instances.CreateClient(compatInstance),
+										compatDMPort,
+										compatDDPort,
+										server.HighPriorityDreamDaemon,
+										cancellationToken))
+							: Task.CompletedTask;
 
 						if (TestingUtils.RunningInGitHubActions) // they only have 2 cores, can't handle intense parallelization
 							await compatTests;