From 731782e8e7ade9f1178b4cb082b41db4d46bad61 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kroi=C3=9F=2C=20Florian?= Date: Sun, 7 Jun 2026 15:46:16 +0200 Subject: [PATCH] fix(tests): Stop timer thread when shutting down dispatchers We call `LanguageServerWrapper.stopDispatcher()` after each test to clean up the spawned executors/threads. However, we did not include the Timer instance, which would cause many leaked Threads during test execution --- .../src/org/eclipse/lsp4e/LanguageServerWrapper.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/org.eclipse.lsp4e/src/org/eclipse/lsp4e/LanguageServerWrapper.java b/org.eclipse.lsp4e/src/org/eclipse/lsp4e/LanguageServerWrapper.java index c23c71eee..286e8ac6b 100644 --- a/org.eclipse.lsp4e/src/org/eclipse/lsp4e/LanguageServerWrapper.java +++ b/org.eclipse.lsp4e/src/org/eclipse/lsp4e/LanguageServerWrapper.java @@ -349,6 +349,9 @@ void stopDispatcher() { // Similarly, the error stream should also close when the input/output streams close. this.errorProcessor.shutdownNow(); + + // Make sure the timer thread shuts down. + this.timer.cancel(); } /**