Skip to content

Commit dccd428

Browse files
Youssef1313Youssef1313
committed
Update to MTP
1 parent d4ee787 commit dccd428

File tree

6 files changed

+31
-64
lines changed

6 files changed

+31
-64
lines changed

‎Directory.Build.props‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,9 @@
2929
</PropertyGroup>
3030

3131
<PropertyGroup>
32-
<VSTestLogger Condition="'$(VSTestLogger)' == ''">trx%3bLogFileName=$(MSBuildProjectName).$(TargetFramework).$(OS).trx</VSTestLogger>
33-
<VSTestResultsDirectory Condition="'$(VSTestResultsDirectory)' == ''">$(ArtifactsTestResultsDir)</VSTestResultsDirectory>
32+
<UseMicrosoftTestingPlatformRunner>true</UseMicrosoftTestingPlatformRunner>
33+
<_MTPResultsDirectory>$(ArtifactsTestResultsDir)</_MTPResultsDirectory>
34+
<TestingPlatformCommandLineArguments>$(TestingPlatformCommandLineArguments) --results-directory $(_MTPResultsDirectory) --report-trx --report-trx-filename $(MSBuildProjectName).$(TargetFramework).$(OS).trx</TestingPlatformCommandLineArguments>
3435
</PropertyGroup>
3536

3637
<PropertyGroup Condition="'$(GITHUB_ACTIONS)' == 'true'">

‎Directory.Packages.props‎

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
<System9Version>9.0.10</System9Version>
55
<System10Version>10.0.0-rc.2.25502.107</System10Version>
66
<MicrosoftExtensionsAIVersion>9.10.2</MicrosoftExtensionsAIVersion>
7+
<MicrosoftTestingPlatformVersion>2.0.2</MicrosoftTestingPlatformVersion>
78
</PropertyGroup>
89

910
<!-- Product dependencies netstandard -->
@@ -55,10 +56,10 @@
5556

5657
<!-- Testing dependencies -->
5758
<PackageVersion Include="Anthropic.SDK" Version="5.6.0" />
58-
<PackageVersion Include="coverlet.collector" Version="6.0.4">
59-
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
60-
<PrivateAssets>all</PrivateAssets>
61-
</PackageVersion>
59+
<PackageVersion Include="Microsoft.Testing.Extensions.CodeCoverage" Version="18.1.0" />
60+
<PackageVersion Include="Microsoft.Testing.Extensions.HangDump" Version="$(MicrosoftTestingPlatformVersion)" />
61+
<PackageVersion Include="Microsoft.Testing.Extensions.CrashDump" Version="$(MicrosoftTestingPlatformVersion)" />
62+
<PackageVersion Include="Microsoft.Testing.Extensions.TrxReport" Version="$(MicrosoftTestingPlatformVersion)" />
6263
<PackageVersion Include="GitHubActionsTestLogger" Version="2.4.1" />
6364
<PackageVersion Include="Microsoft.Extensions.AI.OpenAI" Version="9.10.2-preview.1.25552.1" />
6465
<PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="$(System9Version)" />
@@ -67,7 +68,6 @@
6768
<PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="$(System9Version)" />
6869
<PackageVersion Include="Microsoft.Extensions.Options" Version="$(System9Version)" />
6970
<PackageVersion Include="Microsoft.Extensions.TimeProvider.Testing" Version="9.10.0" />
70-
<PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.14.1" />
7171
<PackageVersion Include="Moq" Version="4.20.72" />
7272
<PackageVersion Include="OpenTelemetry" Version="1.12.0" />
7373
<PackageVersion Include="OpenTelemetry.Exporter.InMemory" Version="1.12.0" />
@@ -82,8 +82,7 @@
8282
<PackageVersion Include="Serilog.Sinks.File" Version="7.0.0" />
8383
<PackageVersion Include="Serilog" Version="4.3.0" />
8484
<PackageVersion Include="System.Linq.AsyncEnumerable" Version="$(System10Version)" />
85-
<PackageVersion Include="xunit.v3" Version="3.0.1" />
86-
<PackageVersion Include="xunit.runner.visualstudio" Version="3.1.4" />
85+
<PackageVersion Include="xunit.v3.mtp-v2" Version="3.2.0" />
8786
<PackageVersion Include="System.Net.Http" Version="4.3.4" />
8887
<PackageVersion Include="JsonSchema.Net" Version="7.4.0" />
8988
</ItemGroup>

‎Makefile‎

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,12 @@ test: build
1919
dotnet test \
2020
--no-build \
2121
--configuration $(CONFIGURATION) \
22-
--filter '(Execution!=Manual)' \
23-
--blame \
24-
--blame-crash \
25-
--blame-hang-timeout 7m \
26-
--diag "$(ARTIFACT_PATH)/diag.txt" \
27-
--logger "trx" \
28-
--logger "GitHubActions;summary.includePassedTests=true;summary.includeSkippedTests=true" \
29-
--collect "XPlat Code Coverage" \
30-
--results-directory $(ARTIFACT_PATH)/testresults \
22+
--filter-not-trait 'Execution=Manual' \
23+
--crashdump \
24+
--hangdump \
25+
--hangdump-timeout 7m \
26+
--coverage
27+
-p:_MTPResultsDirectory=$(ARTIFACT_PATH)/testresults \
3128
-- \
3229
RunConfiguration.CollectSourceInformation=true
3330

‎global.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,8 @@
33
"version": "10.0.100-rc.2",
44
"rollForward": "minor",
55
"allowPrerelease": true
6+
},
7+
"test": {
8+
"runner": "Microsoft.Testing.Platform"
69
}
710
}

‎tests/ModelContextProtocol.AspNetCore.Tests/ModelContextProtocol.AspNetCore.Tests.csproj‎

Lines changed: 7 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@
44
<TargetFrameworks>net10.0;net9.0;net8.0</TargetFrameworks>
55
<ImplicitUsings>enable</ImplicitUsings>
66
<Nullable>enable</Nullable>
7-
<IsPackable>false</IsPackable>
8-
<IsTestProject>true</IsTestProject>
7+
<OutputType>Exe</OutputType>
98
<RootNamespace>ModelContextProtocol.AspNetCore.Tests</RootNamespace>
109
</PropertyGroup>
1110

@@ -14,41 +13,26 @@
1413
<JsonSerializerIsReflectionEnabledByDefault>false</JsonSerializerIsReflectionEnabledByDefault>
1514
</PropertyGroup>
1615

17-
<PropertyGroup>
18-
<!-- Without this, tests are currently not showing results until all tests complete
19-
https://xunit.net/docs/getting-started/v3/microsoft-testing-platform
20-
-->
21-
<DisableTestingPlatformServerCapability>true</DisableTestingPlatformServerCapability>
22-
</PropertyGroup>
23-
2416
<ItemGroup>
2517
<Compile Include="..\Common\**\*.cs" />
2618
</ItemGroup>
2719

2820
<ItemGroup>
29-
<PackageReference Include="coverlet.collector">
30-
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
31-
<PrivateAssets>all</PrivateAssets>
32-
</PackageReference>
33-
<PackageReference Include="GitHubActionsTestLogger">
34-
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
35-
<PrivateAssets>all</PrivateAssets>
36-
</PackageReference>
21+
<PackageReference Include="Microsoft.Testing.Extensions.CodeCoverage" />
22+
<PackageReference Include="Microsoft.Testing.Extensions.HangDump" />
23+
<PackageReference Include="Microsoft.Testing.Extensions.CrashDump" />
24+
<PackageReference Include="Microsoft.Testing.Extensions.TrxReport" />
25+
<PackageReference Include="GitHubActionsTestLogger" />
3726
<PackageReference Include="Microsoft.AspNetCore.Authentication.JwtBearer" />
3827
<PackageReference Include="Microsoft.Extensions.AI" />
3928
<PackageReference Include="Microsoft.Extensions.AI.OpenAI" />
4029
<PackageReference Include="Microsoft.Extensions.Logging" />
4130
<PackageReference Include="Microsoft.Extensions.Logging.Console" />
4231
<PackageReference Include="Microsoft.Extensions.TimeProvider.Testing" />
43-
<PackageReference Include="Microsoft.NET.Test.Sdk" />
4432
<PackageReference Include="Moq" />
4533
<PackageReference Include="OpenTelemetry" />
4634
<PackageReference Include="OpenTelemetry.Exporter.InMemory" />
47-
<PackageReference Include="xunit.v3" />
48-
<PackageReference Include="xunit.runner.visualstudio">
49-
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
50-
<PrivateAssets>all</PrivateAssets>
51-
</PackageReference>
35+
<PackageReference Include="xunit.v3.mtp-v2" />
5236
</ItemGroup>
5337

5438
<ItemGroup Condition="!$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net10.0'))">

‎tests/ModelContextProtocol.Tests/ModelContextProtocol.Tests.csproj‎

Lines changed: 6 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@
66
<ImplicitUsings>enable</ImplicitUsings>
77
<Nullable>enable</Nullable>
88

9-
<IsPackable>false</IsPackable>
10-
<IsTestProject>true</IsTestProject>
119
<RootNamespace>ModelContextProtocol.Tests</RootNamespace>
1210
<!-- https://github.com/dotnet/sdk/issues/51060 -->
1311
<NoWarn>$(NoWarn);NU1903;NU1902</NoWarn>
@@ -18,13 +16,6 @@
1816
<JsonSerializerIsReflectionEnabledByDefault>false</JsonSerializerIsReflectionEnabledByDefault>
1917
</PropertyGroup>
2018

21-
<PropertyGroup>
22-
<!-- Without this, tests are currently not showing results until all tests complete
23-
https://xunit.net/docs/getting-started/v3/microsoft-testing-platform
24-
-->
25-
<DisableTestingPlatformServerCapability>true</DisableTestingPlatformServerCapability>
26-
</PropertyGroup>
27-
2819
<ItemGroup>
2920
<Compile Include="..\Common\**\*.cs" />
3021
</ItemGroup>
@@ -35,29 +26,21 @@
3526
</ItemGroup>
3627

3728
<ItemGroup>
38-
<PackageReference Include="coverlet.collector">
39-
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
40-
<PrivateAssets>all</PrivateAssets>
41-
</PackageReference>
42-
<PackageReference Include="GitHubActionsTestLogger">
43-
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
44-
<PrivateAssets>all</PrivateAssets>
45-
</PackageReference>
29+
<PackageReference Include="Microsoft.Testing.Extensions.CodeCoverage" />
30+
<PackageReference Include="Microsoft.Testing.Extensions.HangDump" />
31+
<PackageReference Include="Microsoft.Testing.Extensions.CrashDump" />
32+
<PackageReference Include="Microsoft.Testing.Extensions.TrxReport" />
33+
<PackageReference Include="GitHubActionsTestLogger" />
4634
<PackageReference Include="Microsoft.Extensions.AI" />
4735
<PackageReference Include="Microsoft.Extensions.AI.OpenAI" />
4836
<PackageReference Include="Microsoft.Extensions.Logging" />
4937
<PackageReference Include="Microsoft.Extensions.Logging.Console" />
50-
<PackageReference Include="Microsoft.NET.Test.Sdk" />
5138
<PackageReference Include="Moq" />
5239
<PackageReference Include="OpenTelemetry" />
5340
<PackageReference Include="OpenTelemetry.Exporter.InMemory" />
5441
<PackageReference Include="Serilog" />
5542
<PackageReference Include="JsonSchema.Net" />
56-
<PackageReference Include="xunit.v3" />
57-
<PackageReference Include="xunit.runner.visualstudio">
58-
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
59-
<PrivateAssets>all</PrivateAssets>
60-
</PackageReference>
43+
<PackageReference Include="xunit.v3.mtp-v2" />
6144
</ItemGroup>
6245

6346
<ItemGroup Condition="!$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net10.0'))">

0 commit comments

Comments
 (0)