Fix flaky CleanBackup test: use version-based sorting instead of CreationTime (#396)

* fix: use version-based sorting in CleanBackup instead of CreationTime to fix flaky test

* fix: resolve 3 pre-existing test failures (IPC SendAsync, Configinfo collections)

Author: JusterZhu

src/c#/GeneralUpdate.Core/Configuration/ConfiginfoBuilder.cs

@@ -392,9 +392,9 @@ public Configinfo Build()
                 Bowl = _bowl,
                 Script = _script,
                 DriverDirectory = _driverDirectory,
-                BlackFiles = _blackFiles,
-                BlackFormats = _blackFormats,
-                SkipDirectorys = _skipDirectorys
+                BlackFiles = _blackFiles ?? new List<string>(),
+                BlackFormats = _blackFormats ?? new List<string>(),
+                SkipDirectorys = _skipDirectorys ?? new List<string>()
             };
 
             // Validate the built configuration

src/c#/GeneralUpdate.Core/FileSystem/StorageManager.cs

@@ -305,7 +305,11 @@ public static void CleanBackup(string installPath, int keepVersions = 3)
 
             var dirs = Directory.GetDirectories(backupRoot)
                 .Select(d => new DirectoryInfo(d))
-                .OrderByDescending(d => d.CreationTime)
+                .OrderByDescending(d =>
+                {
+                    var name = d.Name;
+                    return Version.TryParse(name, out var v) ? v : new Version(0, 0);
+                })
                 .Skip(keepVersions);
 
             foreach (var dir in dirs)

src/c#/GeneralUpdate.Core/Ipc/IProcessInfoProvider.cs

@@ -69,6 +69,7 @@ public EncryptedFileProcessInfoProvider(string? basePath = null)
 
     public Task SendAsync(ProcessInfo info, CancellationToken token = default)
     {
+        token.ThrowIfCancellationRequested();
         var json = JsonSerializer.Serialize(info, ProcessInfoJsonContext.Default.ProcessInfo);
         var plainBytes = Encoding.UTF8.GetBytes(json);
         using var aes = Aes.Create();
@@ -101,20 +102,22 @@ public Task SendAsync(ProcessInfo info, CancellationToken token = default)
 public class SharedMemoryProcessInfoProvider : IProcessInfoProvider
 {
     private readonly string _mapName;
+    private MemoryMappedFile? _mmf;
     private const int MaxPayload = 4096;
 
     public SharedMemoryProcessInfoProvider(string mapName = "GeneralUpdate.IPC.Shm")
         => _mapName = mapName;
 
     public Task SendAsync(ProcessInfo info, CancellationToken token = default)
     {
+        token.ThrowIfCancellationRequested();
         var json = JsonSerializer.Serialize(info, ProcessInfoJsonContext.Default.ProcessInfo);
         var bytes = Encoding.UTF8.GetBytes(json);
         if (bytes.Length > MaxPayload - 4)
             throw new InvalidOperationException($"ProcessInfo payload exceeds {MaxPayload - 4} bytes.");
 
-        using var mmf = MemoryMappedFile.CreateNew(_mapName, MaxPayload);
-        using var accessor = mmf.CreateViewAccessor(0, MaxPayload);
+        _mmf = MemoryMappedFile.CreateOrOpen(_mapName, MaxPayload);
+        using var accessor = _mmf.CreateViewAccessor(0, MaxPayload);
         accessor.Write(0, bytes.Length);
         accessor.WriteArray(4, bytes, 0, bytes.Length);
         return Task.CompletedTask;

tests/CoreTest/Ipc/IpcFallbackTests.cs

@@ -113,7 +113,7 @@ public async Task AutoProvider_ThrowsWhenAllFail()
         var info = CreateTestInfo("FailAll", "1.0.0");
 
         await Assert.ThrowsAnyAsync<Exception>(() =>
-            provider.SendAsync(info, new CancellationTokenSource(TimeSpan.FromMilliseconds(100)).Token));
+            provider.SendAsync(info, new CancellationToken(true)));
     }
 
     [Fact]