From 4f01a75887ae4be499f5cd6b58322524bcbdd492 Mon Sep 17 00:00:00 2001 From: Brian Hotopp Date: Tue, 31 Mar 2026 12:51:34 -0400 Subject: [PATCH] [2.x] test: Add test for error response to unknown JSON-RPC method MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This test fails against the current code — the server silently drops requests with unknown methods instead of responding with an error. Co-Authored-By: Claude Opus 4.6 (1M context) --- server-test/src/test/scala/testpkg/ResponseTest.scala | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/server-test/src/test/scala/testpkg/ResponseTest.scala b/server-test/src/test/scala/testpkg/ResponseTest.scala index 4f248fc01..48f914932 100644 --- a/server-test/src/test/scala/testpkg/ResponseTest.scala +++ b/server-test/src/test/scala/testpkg/ResponseTest.scala @@ -79,6 +79,16 @@ class ResponseTest extends AbstractServerTest { } } + test("unknown method returns error") { + val id = svr.session.nextId() + svr.session.sendJsonRpc(id, "build/foo", "{}").get + val response = svr.session.waitForResponseMsg(10.seconds, id).get + assert( + response.error.exists(_.code == -32601), + s"Expected method-not-found error, got: $response" + ) + } + private def neverReceiveResponse( duration: FiniteDuration )(predicate: sbt.internal.protocol.JsonRpcResponseMessage => Boolean): Unit =