raise on invalid cancel request

Author: lukaszsamson

apps/elixir_ls_debugger/lib/debugger/server.ex

@@ -521,11 +521,12 @@ defmodule ElixirLS.Debugger.Server do
           }
 
         _ ->
-          Output.debugger_console(
-            "Received cancel request for unknown requestId: #{inspect(request_or_progress_id)}\n"
-          )
-
-          state
+          raise ServerError,
+            message: "invalidRequest",
+            format: "Request or progress {reguestOrProgressId} cannot be cancelled",
+            variables: %{
+              "reguestOrProgressId" => inspect(request_or_progress_id)
+            }
       end
 
     {%{}, state}

apps/elixir_ls_debugger/test/debugger_test.exs

@@ -2914,7 +2914,7 @@ defmodule ElixirLS.Debugger.ServerTest do
     end)
   end
 
-  describe "Watch section" do
+  describe "evaluate" do
     defp gen_watch_expression_packet(seq, expr) do
       %{
         "arguments" => %{
@@ -3372,7 +3372,16 @@ defmodule ElixirLS.Debugger.ServerTest do
         cancel_req(2, %{"requestId" => 1})
       )
 
-      assert_receive(response(_, 2, "cancel", _))
+      assert_receive(
+        error_response(
+          _,
+          2,
+          "cancel",
+          "invalidRequest",
+          "Request or progress {reguestOrProgressId} cannot be cancelled",
+          %{"reguestOrProgressId" => "1"}
+        )
+      )
 
       assert Process.alive?(server)
     end)