Skip to content

Commit 8f6e91f

Browse files
committed
Fix undisposed resources
1 parent db2baaa commit 8f6e91f

3 files changed

Lines changed: 14 additions & 14 deletions

File tree

tests/ModelContextProtocol.Tests/Configuration/McpServerBuilderExtensionsTransportsTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,15 @@ namespace ModelContextProtocol.Tests.Configuration;
99
public class McpServerBuilderExtensionsTransportsTests
1010
{
1111
[Fact]
12-
public void WithStdioServerTransport_Sets_Transport()
12+
public async Task WithStdioServerTransport_Sets_Transport()
1313
{
1414
var services = new ServiceCollection();
1515
services.AddMcpServer().WithStdioServerTransport();
1616

1717
var transportServiceType = services.FirstOrDefault(s => s.ServiceType == typeof(ITransport));
1818
Assert.NotNull(transportServiceType);
1919

20-
var serviceProvider = services.BuildServiceProvider();
20+
await using var serviceProvider = services.BuildServiceProvider();
2121
Assert.IsType<StdioServerTransport>(serviceProvider.GetRequiredService<ITransport>());
2222
}
2323

tests/ModelContextProtocol.Tests/Server/McpServerLoggingLevelTests.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,29 +15,29 @@ public McpServerLoggingLevelTests()
1515
}
1616

1717
[Fact]
18-
public void CanCreateServerWithLoggingLevelHandler()
18+
public async Task CanCreateServerWithLoggingLevelHandler()
1919
{
2020
var services = new ServiceCollection();
2121

2222
services.AddMcpServer()
2323
.WithStdioServerTransport()
2424
.WithSetLoggingLevelHandler(async (ctx, ct) => new EmptyResult());
2525

26-
var provider = services.BuildServiceProvider();
26+
await using var provider = services.BuildServiceProvider();
2727

2828
provider.GetRequiredService<McpServer>();
2929
}
3030

3131
[Fact]
32-
public void AddingLoggingLevelHandlerSetsLoggingCapability()
32+
public async Task AddingLoggingLevelHandlerSetsLoggingCapability()
3333
{
3434
var services = new ServiceCollection();
3535

3636
services.AddMcpServer()
3737
.WithStdioServerTransport()
3838
.WithSetLoggingLevelHandler(async (ctx, ct) => new EmptyResult());
3939

40-
var provider = services.BuildServiceProvider();
40+
await using var provider = services.BuildServiceProvider();
4141

4242
var server = provider.GetRequiredService<McpServer>();
4343

@@ -46,12 +46,12 @@ public void AddingLoggingLevelHandlerSetsLoggingCapability()
4646
}
4747

4848
[Fact]
49-
public void ServerWithoutCallingLoggingLevelHandlerDoesNotSetLoggingCapability()
49+
public async Task ServerWithoutCallingLoggingLevelHandlerDoesNotSetLoggingCapability()
5050
{
5151
var services = new ServiceCollection();
5252
services.AddMcpServer()
5353
.WithStdioServerTransport();
54-
var provider = services.BuildServiceProvider();
54+
await using var provider = services.BuildServiceProvider();
5555
var server = provider.GetRequiredService<McpServer>();
5656
Assert.Null(server.ServerOptions.Capabilities?.Logging);
5757
}

tests/ModelContextProtocol.Tests/Server/McpServerResourceTests.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public McpServerResourceTests()
2929
}
3030

3131
[Fact]
32-
public void CanCreateServerWithResource()
32+
public async Task CanCreateServerWithResource()
3333
{
3434
var services = new ServiceCollection();
3535

@@ -58,14 +58,14 @@ public void CanCreateServerWithResource()
5858
};
5959
});
6060

61-
var provider = services.BuildServiceProvider();
61+
await using var provider = services.BuildServiceProvider();
6262

6363
provider.GetRequiredService<McpServer>();
6464
}
6565

6666

6767
[Fact]
68-
public void CanCreateServerWithResourceTemplates()
68+
public async Task CanCreateServerWithResourceTemplates()
6969
{
7070
var services = new ServiceCollection();
7171

@@ -94,13 +94,13 @@ public void CanCreateServerWithResourceTemplates()
9494
};
9595
});
9696

97-
var provider = services.BuildServiceProvider();
97+
await using var provider = services.BuildServiceProvider();
9898

9999
provider.GetRequiredService<McpServer>();
100100
}
101101

102102
[Fact]
103-
public void CreatingReadHandlerWithNoListHandlerSucceeds()
103+
public async Task CreatingReadHandlerWithNoListHandlerSucceeds()
104104
{
105105
var services = new ServiceCollection();
106106
services.AddMcpServer()
@@ -117,7 +117,7 @@ public void CreatingReadHandlerWithNoListHandlerSucceeds()
117117
}]
118118
};
119119
});
120-
var sp = services.BuildServiceProvider();
120+
await using var sp = services.BuildServiceProvider();
121121

122122
sp.GetRequiredService<McpServer>();
123123
}

0 commit comments

Comments
 (0)