fix(discord): avoid duplicate schedule send after new session

2026-05-26 13:40:59 +03:00
parent 682dd3fdec
commit 6ed0a120a0
2 changed files with 31 additions and 13 deletions
@@ -127,6 +127,18 @@ public sealed class DiscordNewSessionHandlerTests
        Assert.Contains("RollbackAsync", source, StringComparison.Ordinal);
    }

+    [Fact]
+    public void Handler_ShouldNotRollbackCommittedTransactionAfterPostCommitFailure()
+    {
+        var repoRoot = GetRepoRoot();
+        var handlerPath = Path.Combine(repoRoot, "src", "GmRelay.DiscordBot", "Features", "Sessions", "DiscordNewSessionHandler.cs");
+        var source = File.ReadAllText(handlerPath);
+
+        Assert.Contains("transactionCommitted = false", source, StringComparison.Ordinal);
+        Assert.Contains("transactionCommitted = true", source, StringComparison.Ordinal);
+        Assert.Contains("if (!transactionCommitted)", source, StringComparison.Ordinal);
+    }
+
    [Fact]
    public void Handler_ShouldRespectCancellationToken()
    {
@@ -148,6 +160,17 @@ public sealed class DiscordNewSessionHandlerTests
        Assert.Contains("message.Embeds = embeds", source, StringComparison.Ordinal);
    }

+    [Fact]
+    public void Handler_ShouldLeaveScheduleMessageCreationToInteractionResponse()
+    {
+        var repoRoot = GetRepoRoot();
+        var handlerPath = Path.Combine(repoRoot, "src", "GmRelay.DiscordBot", "Features", "Sessions", "DiscordNewSessionHandler.cs");
+        var source = File.ReadAllText(handlerPath);
+
+        Assert.DoesNotContain("SendScheduleAsync", source, StringComparison.Ordinal);
+        Assert.DoesNotContain("PlatformScheduleMessage", source, StringComparison.Ordinal);
+    }
+
    private static DateTimeOffset FutureDateAt1930()
    {
        var future = DateTimeOffset.UtcNow.AddDays(7);