From 016c99f0f3ecfa221fa281d1e409cfaa57f70a61 Mon Sep 17 00:00:00 2001 From: Stephen Toub Date: Tue, 15 Apr 2025 11:16:45 -0400 Subject: [PATCH] Delete faulty assert We need better coverage of Subscribe/Unsubscribe separately, but for now this assert is bad. --- tests/ModelContextProtocol.Tests/ClientIntegrationTests.cs | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tests/ModelContextProtocol.Tests/ClientIntegrationTests.cs b/tests/ModelContextProtocol.Tests/ClientIntegrationTests.cs index d6944df9..dc01c92c 100644 --- a/tests/ModelContextProtocol.Tests/ClientIntegrationTests.cs +++ b/tests/ModelContextProtocol.Tests/ClientIntegrationTests.cs @@ -307,11 +307,6 @@ public async Task UnsubscribeResource_Stdio() // unsubscribe await client.UnsubscribeFromResourceAsync("test://static/resource/1", TestContext.Current.CancellationToken); - receivedNotification = new(); - - // wait a bit to validate we don't receive another. this is best effort only; - // false negatives are possible. - await Assert.ThrowsAsync(() => receivedNotification.Task.WaitAsync(TimeSpan.FromSeconds(1), TestContext.Current.CancellationToken)); } [Theory]