Handle AlreadyDisposedException when hiding tool windows.

JetBrains-Research · Oct 13, 2024 · a631791 · a631791
1 parent c0373d7
commit a631791
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/src/main/kotlin/org/jetbrains/research/testspark/display/TestSparkDisplayManager.kt b/src/main/kotlin/org/jetbrains/research/testspark/display/TestSparkDisplayManager.kt
@@ -4,6 +4,7 @@ import com.intellij.openapi.editor.Editor
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.wm.ToolWindow
 import com.intellij.openapi.wm.ToolWindowManager
+import com.intellij.serviceContainer.AlreadyDisposedException
 import com.intellij.ui.content.ContentManager
 import org.jetbrains.research.testspark.bundles.plugin.PluginLabelsBundle
 import org.jetbrains.research.testspark.bundles.plugin.PluginMessagesBundle
@@ -94,7 +95,8 @@ class TestSparkDisplayManager {
                 }
             }
         }
-
-        toolWindow?.hide()
+        try {
+            toolWindow?.hide()
+        } catch (_: AlreadyDisposedException) {} // Make sure the process continues if the tool window is already closed
     }
 }
diff --git a/...otlin/org/jetbrains/research/testspark/display/generatedTests/GeneratedTestsTabBuilder.kt b/...otlin/org/jetbrains/research/testspark/display/generatedTests/GeneratedTestsTabBuilder.kt
@@ -3,6 +3,7 @@ package org.jetbrains.research.testspark.display.generatedTests
 import com.intellij.openapi.editor.Editor
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.wm.ToolWindowManager
+import com.intellij.serviceContainer.AlreadyDisposedException
 import com.intellij.ui.content.ContentFactory
 import com.intellij.ui.content.ContentManager
 import org.jetbrains.research.testspark.bundles.plugin.PluginLabelsBundle
@@ -233,9 +234,11 @@ class GeneratedTestsTabBuilder(
      * Closes the tool window by removing the content and hiding the window.
      */
     private fun closeToolWindow() {
-        generatedTestsTabData.contentManager?.removeContent(generatedTestsTabData.content!!, true)
-        ToolWindowManager.getInstance(project).getToolWindow("TestSpark")?.hide()
-        coverageVisualisationTabBuilder.closeToolWindowTab()
+        try {
+            generatedTestsTabData.contentManager?.removeContent(generatedTestsTabData.content!!, true)
+            ToolWindowManager.getInstance(project).getToolWindow("TestSpark")?.hide()
+            coverageVisualisationTabBuilder.closeToolWindowTab()
+        } catch (_: AlreadyDisposedException) {} // Make sure the process continues if the tool window is already closed
     }
 
     /**