[WIP]test(integration): Add Streamable HTTP transport integration tests #486

jskjw157 · 2026-01-20T09:29:47Z

Summary

Add integration tests for Prompts, Resources, and Tools over Streamable HTTP transport as outlined in #183.

Changes

Infrastructure: Enable STREAMABLE_HTTP transport in test base
Prompts: Add functional integration tests
Resources: Add functional integration tests
Tools: Add functional integration tests

Related to #332

kpavlov

Thanks, @jskjw157!
Could we take a look at the test scenarios and see, if we can move any use cases that aren’t specific to the protocol to the base class?
Also, could we tidy up those long TODO comments?
Let’s give another go, this time with a focus on making the tests easier to maintain.

...t/src/jvmTest/kotlin/io/modelcontextprotocol/kotlin/sdk/integration/kotlin/KotlinTestBase.kt

...Test/kotlin/io/modelcontextprotocol/kotlin/sdk/integration/utils/MockAuthorizationWrapper.kt

...ntextprotocol/kotlin/sdk/integration/kotlin/streamablehttp/StreamableHttpToolSpecificTest.kt

...mTest/kotlin/io/modelcontextprotocol/kotlin/sdk/integration/utils/StreamableHttpTestUtils.kt

...xtprotocol/kotlin/sdk/integration/kotlin/streamablehttp/ToolIntegrationTestStreamableHttp.kt

jskjw157 · 2026-01-21T23:40:29Z

Fixed in 97f1fa9

kpavlov

Please rebase onto main and squash the commits so we maintain a clear commit history.

...commonMain/kotlin/io/modelcontextprotocol/kotlin/sdk/server/StreamableHttpServerTransport.kt

jskjw157 · 2026-01-22T14:29:56Z

Done! Rebased onto main and squashed the commits.

Are there any other issues or features you'd recommend I work on next?

kpavlov

Thanks, @jskjw157!

While this PR does add some useful integration tests for the Streamable HTTP protocol, the security tests could use a bit more thought.

To start, I’d suggest moving the security tests to a separate PR and keeping this one focused on adding Streamable HTTP transport support to the existing tests (we can remove the ‘Security tests’ tag).

The access control system isn’t quite ready to go on the server yet, but we’re running some tests to check it out a bit early. Also, the list resource tests should make sure that resources are only shown to users who have the right to see them. Maybe we could replace MockAuthorizationWrapper with something more like a production-code solution, such as a chain of generic (resource+principal)-aware predicates. You might want to have a look at Spring Security, where you can find examples of how these ideas are put into action.

TL;DR - Let's remove security tests from this PR and keep it focused on adding Streamable HTTP protocol tests.

kpavlov · 2026-01-26T12:07:39Z

...t/src/jvmTest/kotlin/io/modelcontextprotocol/kotlin/sdk/integration/kotlin/KotlinTestBase.kt

    private var stdioClientOutput: Sink? = null

+    // StreamableHTTP-specific fields
+    private var streamableHttpServerTransport: StreamableHttpServerTransport? = null


Just a heads-up: this should probably be in the streamable test subclass, but since we already have transport-specific fields here, let’s tackle it separately.

...textprotocol/kotlin/sdk/integration/kotlin/security/AbstractPromptSecurityIntegrationTest.kt

kpavlov · 2026-01-26T12:46:38Z

...textprotocol/kotlin/sdk/integration/kotlin/security/AbstractPromptSecurityIntegrationTest.kt

+    fun testListPromptsDenied() {
+        runBlocking {
+            val result = client.listPrompts()
+            assertNotNull(result, "List prompts should succeed")
+        }
+    }


This test does not redundant

...textprotocol/kotlin/sdk/integration/kotlin/security/AbstractPromptSecurityIntegrationTest.kt

...Test/kotlin/io/modelcontextprotocol/kotlin/sdk/integration/utils/MockAuthorizationWrapper.kt

...mTest/kotlin/io/modelcontextprotocol/kotlin/sdk/integration/utils/StreamableHttpTestUtils.kt

...xtprotocol/kotlin/sdk/integration/kotlin/security/AbstractResourceSecurityIntegrationTest.kt

...ontextprotocol/kotlin/sdk/integration/kotlin/security/AbstractToolSecurityIntegrationTest.kt

jskjw157 · 2026-02-04T13:33:20Z

Thanks, @kpavlov!

I've updated the PR to address the review feedback:

Moved the security tests (and MockAuthorizationWrapper) out of this PR to keep it focused on Streamable HTTP transport integration tests only. I'll follow up with a separate PR for security once we align on the auth model.
Squashed the PR down to a single commit.
Cleaned up Streamable HTTP test utilities (header naming + removed stale TODO/comment blocks) and kept production code changes out of this PR.

There are still a couple of notes around transport-specific fields in KotlinTestBase that I'm leaving for a follow-up refactor, as suggested.

jskjw157 · 2026-02-04T13:33:57Z

Follow-up: there are a couple of inline notes about transport-specific fields living in KotlinTestBase (line ~65). Ack — agreed they should live in the streamable-http subclass; I'll handle that in a dedicated follow-up PR to keep this one focused.

kpavlov · 2026-02-04T14:06:46Z

integration-test/src/jvmTest/typescript/client/stdio-client.ts

        console.error('Error:', error);
-        process.exit(1);
+        // Don't hard-exit; allow finally{} to run so the client always closes cleanly.
+        process.exitCode = 1;


…ing in MockMcp - remove unised StreamableHttpTestUtils - fix session id header is [MCP-Session-Id"](https://modelcontextprotocol.io/specification/2025-11-25/basic/transports)

kpavlov

Let’s merge this and focus more on testing in the follow-up PRs.

jskjw157 force-pushed the test/streamable-http-integration branch from e71221c to 56cd2d7 Compare January 20, 2026 09:31

kpavlov reviewed Jan 20, 2026

View reviewed changes

kpavlov added the tests label Jan 20, 2026

kpavlov reviewed Jan 22, 2026

View reviewed changes

...commonMain/kotlin/io/modelcontextprotocol/kotlin/sdk/server/StreamableHttpServerTransport.kt Show resolved Hide resolved

kpavlov force-pushed the test/streamable-http-integration branch from 97f1fa9 to 921bed3 Compare January 22, 2026 07:19

jskjw157 force-pushed the test/streamable-http-integration branch 2 times, most recently from c2a5d9f to 8e9557c Compare January 22, 2026 13:46

kpavlov force-pushed the test/streamable-http-integration branch from 8e9557c to e253b80 Compare January 26, 2026 06:47

kpavlov requested changes Jan 26, 2026

View reviewed changes

kpavlov force-pushed the test/streamable-http-integration branch from e253b80 to d1b3f67 Compare February 4, 2026 09:26

jskjw157 force-pushed the test/streamable-http-integration branch 2 times, most recently from 5ac3cba to 5d71104 Compare February 4, 2026 12:07

kpavlov reviewed Feb 4, 2026

View reviewed changes

kpavlov force-pushed the test/streamable-http-integration branch from 5d71104 to 600285a Compare February 4, 2026 15:04

test(integration): add Streamable HTTP transport tests (no security)

3c65404

kpavlov force-pushed the test/streamable-http-integration branch from 600285a to 3c65404 Compare February 5, 2026 11:24

refactor: remove unused StreamableHttpTestUtils and update header cas…

2063649

…ing in MockMcp - remove unised StreamableHttpTestUtils - fix session id header is [MCP-Session-Id"](https://modelcontextprotocol.io/specification/2025-11-25/basic/transports)

kpavlov changed the title ~~test(integration): Add Streamable HTTP transport integration tests~~ [WIP]test(integration): Add Streamable HTTP transport integration tests Feb 5, 2026

kpavlov approved these changes Feb 5, 2026

View reviewed changes

kpavlov merged commit def0e92 into modelcontextprotocol:main Feb 5, 2026
11 checks passed

[WIP]test(integration): Add Streamable HTTP transport integration tests #486

[WIP]test(integration): Add Streamable HTTP transport integration tests #486

Uh oh!

Conversation

jskjw157 commented Jan 20, 2026 • edited by kpavlov Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Uh oh!

kpavlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jskjw157 commented Jan 21, 2026

Uh oh!

kpavlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

jskjw157 commented Jan 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kpavlov left a comment

Choose a reason for hiding this comment

Uh oh!

kpavlov Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

kpavlov Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jskjw157 commented Feb 4, 2026

Uh oh!

jskjw157 commented Feb 4, 2026

Uh oh!

kpavlov Feb 4, 2026

Choose a reason for hiding this comment

Uh oh!

kpavlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

jskjw157 commented Jan 20, 2026 •

edited by kpavlov

Loading

jskjw157 commented Jan 22, 2026 •

edited

Loading