Fix test projects that don't follow folder structure#2640
Merged
hallipr merged 1 commit intoMay 14, 2026
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR reorganizes several test projects to match the expected tests/<ProjectName>/<ProjectName>.csproj folder structure, while adding/updating related tests and documentation for Fabric and distributed HTTP server components.
Changes:
- Updated solution and project references to point to nested test project folders.
- Added/reorganized Fabric OneLake and Fabric Core test coverage, helpers, and test docs.
- Updated distributed HTTP server tests and docs with minor cleanup and validation coverage.
Reviewed changes
Copilot reviewed 15 out of 43 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| Microsoft.Mcp.slnx | Updates test project paths. |
| servers/Fabric.Mcp.Server/Fabric.Mcp.Server.slnx | Updates Fabric test project paths. |
| core/Microsoft.ModelContextProtocol.HttpServer.Distributed/Microsoft.ModelContextProtocol.HttpServer.Distributed.slnx | Updates distributed HTTP server test path. |
| core/.../Microsoft.ModelContextProtocol.HttpServer.Distributed.Tests.csproj | Fixes project reference after folder move. |
| core/.../TESTSERVER_SESSION_AFFINITY.md | Adds TestServer/session-affinity design notes. |
| core/.../KeyedServiceTests.cs | Minor formatting/cancellation-token cleanup. |
| core/.../ListeningEndpointResolverTests.cs | Simplifies test initialization and collection setup. |
| core/.../RealServerIntegrationTests.cs | Uses test cancellation tokens and primary constructors. |
| core/.../SessionAffinityEndpointFilterTests.cs | Simplifies helper test types and assertions. |
| core/.../SessionAffinityOptionsValidationTests.cs | Adds options/attribute validation tests. |
| core/.../SessionOwnerInfoSerializerTests.cs | Simplifies serializer field initialization. |
| tools/Fabric.Mcp.Tools.Core/tests/Fabric.Mcp.Tools.Core.Tests/Fabric.Mcp.Tools.Core.Tests.csproj | Fixes source project reference. |
| tools/Fabric.Mcp.Tools.Core/tests/Fabric.Mcp.Tools.Core.Tests/FabricCoreSetupTests.cs | Adds Fabric Core setup tests. |
| tools/Fabric.Mcp.Tools.Core/tests/Fabric.Mcp.Tools.Core.Tests/README.md | Updates test project title. |
| tools/Fabric.Mcp.Tools.Core/tests/Fabric.Mcp.Tools.Core.Tests/Commands/ItemCreateCommandTests.cs | Updates namespace after folder rename. |
| tools/Fabric.Mcp.Tools.OneLake/src/AssemblyInfo.cs | Updates test assembly visibility. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Fabric.Mcp.Tools.OneLake.Tests.csproj | Fixes source project reference. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/FabricOneLakeSetupTests.cs | Removes unused imports. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/README.md | Adds OneLake test documentation. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/TestImplementationSummary.md | Adds OneLake test implementation summary. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/TestSupport/CapturingHttpMessageHandler.cs | Adds HTTP request capture test helper. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/TestSupport/FakeTokenCredential.cs | Adds fake Azure credential test helper. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Services/OneLakePathTraversalTests.cs | Adds traversal validation tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Services/OneLakeServiceTests.cs | Adds OneLake service architecture/example tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Services/OneLakeServiceUserAgentTests.cs | Adds User-Agent header tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/BlobDeleteCommandTests.cs | Adds blob delete command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/BlobGetCommandTests.cs | Adds blob download command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/BlobListCommandTests.cs | Adds blob list command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/BlobPutCommandTests.cs | Adds blob upload command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/DirectoryCreateCommandTests.cs | Adds directory create command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/DirectoryDeleteCommandTests.cs | Adds directory delete command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/FileDeleteCommandTests.cs | Adds file delete command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/FileReadCommandTests.cs | Adds file read command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/FileWriteCommandTests.cs | Adds file write command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/OneLakeItemDataListCommandTests.cs | Adds item data-list command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/OneLakeItemListCommandTests.cs | Adds item list command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/OneLakeWorkspaceListCommandTests.cs | Adds workspace list command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/PathListCommandTests.cs | Adds path list command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/Table/TableConfigGetCommandTests.cs | Adds table config command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/Table/TableGetCommandTests.cs | Adds table get command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/Table/TableListCommandTests.cs | Adds table list command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/Table/TableNamespaceGetCommandTests.cs | Adds table namespace get command tests. |
| tools/Fabric.Mcp.Tools.OneLake/tests/Fabric.Mcp.Tools.OneLake.Tests/Commands/Table/TableNamespaceListCommandTests.cs | Adds table namespace list command tests. |
hallipr
approved these changes
May 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Fixes test projects that don't follow folder structure expectations.
GitHub issue number?
[Link to the GitHub issue this PR addresses]Pre-merge Checklist
servers/Azure.Mcp.Server/README.mdand/orservers/Fabric.Mcp.Server/README.mddocumentationREADME.mdchanges running the script./eng/scripts/Process-PackageReadMe.ps1. See Package READMEToolDescriptionEvaluatorand obtained a score of0.4or more and a top 3 ranking for all related test promptsconsolidated-tools.jsonbreaking-changelabelservers/Azure.Mcp.Server/docs/azmcp-commands.md./eng/scripts/Update-AzCommandsMetadata.ps1to update tool metadata inazmcp-commands.md(required for CI)servers/Azure.Mcp.Server/docs/e2eTestPrompts.mdcrypto mining, spam, data exfiltration, etc.)/azp run mcp - pullrequest - liveto run Live Test Pipeline