modelcontextprotocol/csharp-sdk
Watch
1
0
Fork
You've already forked csharp-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Delete faulty assert (#313)

Browse source 
We need better coverage of Subscribe/Unsubscribe separately, but for now this assert is bad.
This commit is contained in:
Stephen Toub 2025-04-15 19:00:43 -04:00 • committed by GitHub
commit 90748b1198
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 0 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

5
tests/ModelContextProtocol.Tests/ClientIntegrationTests.cs
View file

@ -307,11 +307,6 @@ public partial class ClientIntegrationTests : LoggedTest, IClassFixture<ClientIn
// unsubscribe // unsubscribe
await client.UnsubscribeFromResourceAsync("test://static/resource/1", TestContext.Current.CancellationToken); 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<TimeoutException>(() => receivedNotification.Task.WaitAsync(TimeSpan.FromSeconds(1), TestContext.Current.CancellationToken));
} }
[Theory] [Theory]