fix(cd): Add WSL setup and fix LOLWUT assertions for CD pipeline (#334)

Signed-off-by: currantw <taylor.curran@improving.com>

Author: currantw

.github/workflows/cd.yml

@@ -263,8 +263,16 @@ jobs:
                   dotnet add tests/Valkey.Glide.IntegrationTests package Valkey.Glide --version $RELEASE_VERSION
                   git diff
 
+            - name: Setup WSL (Windows only)
+              # WSL2 is not available on windows-11-arm runners (no Hyper-V).
+              if: ${{ matrix.host.OS == 'windows' && matrix.host.ARCH != 'arm64' }}
+              uses: Vampire/setup-wsl@v7
+              with:
+                  distribution: Ubuntu-22.04
+                  additional-packages: build-essential git make pkg-config libssl-dev
+
             - name: Install server
-              # WSL2 is not available on windows-11-arm runners (no Hyper-V), so we cannot run Valkey server there.
+              # WSL2 is not available on windows-11-arm runners (no Hyper-V).
               if: ${{ !(matrix.host.OS == 'windows' && matrix.host.ARCH == 'arm64') }}
               uses: ./.github/workflows/install-server
               with:

tests/Valkey.Glide.IntegrationTests/ServerManagementCommandTests.cs

@@ -10,7 +10,7 @@ namespace Valkey.Glide.IntegrationTests;
 /// </summary>
 [Collection(typeof(ServerManagementCommandTests))]
 [CollectionDefinition(DisableParallelization = true)]
-public class ServerManagementCommandTests(ServerManagementFixture fixture) : IClassFixture<ServerManagementFixture>
+public class ServerManagementCommandTests(ServerManagementCommandFixture fixture) : IClassFixture<ServerManagementCommandFixture>
 {
     private GlideClient StandaloneClient => fixture.StandaloneClient!;
     private GlideClusterClient ClusterClient => fixture.ClusterClient!;
@@ -132,51 +132,27 @@ public async Task FlushDatabaseAsync_Cluster_WithSyncMode_AndRoute()
 
     [Fact]
     public async Task LolwutAsync_Standalone_WithVersion()
-    {
-        string result = await StandaloneClient.LolwutAsync(new LolwutOptions { Version = 5 });
-        Assert.NotEmpty(result);
-        Assert.Contains("Valkey", result, StringComparison.OrdinalIgnoreCase);
-    }
+        => AssertContainsServerName(await StandaloneClient.LolwutAsync(new LolwutOptions { Version = 5 }));
 
     [Fact]
     public async Task LolwutAsync_Standalone_WithVersionAndParameters()
-    {
-        string result = await StandaloneClient.LolwutAsync(new LolwutOptions { Version = 5, Parameters = [40, 20] });
-        Assert.NotEmpty(result);
-        Assert.Contains("Valkey", result, StringComparison.OrdinalIgnoreCase);
-    }
+        => AssertContainsServerName(await StandaloneClient.LolwutAsync(new LolwutOptions { Version = 5, Parameters = [40, 20] }));
 
     [Fact]
     public async Task LolwutAsync_Cluster_WithVersion()
-    {
-        string result = await ClusterClient.LolwutAsync(new LolwutOptions { Version = 5 });
-        Assert.NotEmpty(result);
-        Assert.Contains("Valkey", result, StringComparison.OrdinalIgnoreCase);
-    }
+        => AssertContainsServerName(await ClusterClient.LolwutAsync(new LolwutOptions { Version = 5 }));
 
     [Fact]
     public async Task LolwutAsync_Cluster_WithVersionAndParameters()
-    {
-        string result = await ClusterClient.LolwutAsync(new LolwutOptions { Version = 5, Parameters = [40, 20] });
-        Assert.NotEmpty(result);
-        Assert.Contains("Valkey", result, StringComparison.OrdinalIgnoreCase);
-    }
+        => AssertContainsServerName(await ClusterClient.LolwutAsync(new LolwutOptions { Version = 5, Parameters = [40, 20] }));
 
     [Fact]
     public async Task LolwutAsync_Standalone_WithParametersOnly()
-    {
-        string result = await StandaloneClient.LolwutAsync(new LolwutOptions { Parameters = [40, 20] });
-        Assert.NotEmpty(result);
-        Assert.Contains("Valkey", result, StringComparison.OrdinalIgnoreCase);
-    }
+        => AssertContainsServerName(await StandaloneClient.LolwutAsync(new LolwutOptions { Parameters = [40, 20] }));
 
     [Fact]
     public async Task LolwutAsync_Cluster_WithParametersOnly()
-    {
-        string result = await ClusterClient.LolwutAsync(new LolwutOptions { Parameters = [40, 20] });
-        Assert.NotEmpty(result);
-        Assert.Contains("Valkey", result, StringComparison.OrdinalIgnoreCase);
-    }
+        => AssertContainsServerName(await ClusterClient.LolwutAsync(new LolwutOptions { Parameters = [40, 20] }));
 
     #endregion
 
@@ -366,13 +342,19 @@ public async Task WaitAofAsync_Cluster_ReturnsResults()
     }
 
     #endregion
+
+    /// <summary>
+    /// Asserts that the result contains the expected server name.
+    /// </summary>
+    private static void AssertContainsServerName(string result)
+        => Assert.Contains(["VALKEY", "REDIS"], name => result.Contains(name, StringComparison.OrdinalIgnoreCase));
 }
 
 /// <summary>
 /// Fixture that provides isolated Valkey server instances for server management tests.
 /// Tests that call FlushAll/FlushDB need their own servers to avoid interfering with other tests.
 /// </summary>
-public class ServerManagementFixture : IAsyncLifetime
+public class ServerManagementCommandFixture : IAsyncLifetime
 {
     private StandaloneServer? _standaloneServer;
     private ClusterServer? _clusterServer;