From 995c635c2e9827a06abb8eb9e90f2176b6611f6b Mon Sep 17 00:00:00 2001 From: Aparna Michael Date: Fri, 27 Oct 2023 11:32:25 +0530 Subject: [PATCH] Fixed the focus listener leak issue. Enabled integrated test. --- build.gradle | 5 ----- .../tools/intellij/lsp4jakarta/it/core/BaseJakartaTest.java | 4 +--- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/build.gradle b/build.gradle index 07b92ea89..4c2d08cf0 100644 --- a/build.gradle +++ b/build.gradle @@ -157,11 +157,6 @@ test { showStandardStreams = true exceptionFormat = 'full' } - - filter { - // disable lsp4jakarta integration tests until listener leak issue is resolved: https://github.com/OpenLiberty/liberty-tools-intellij/issues/541 - excludeTestsMatching "io.openliberty.tools.intellij.lsp4jakarta.it.*" - } } tasks { diff --git a/src/test/java/io/openliberty/tools/intellij/lsp4jakarta/it/core/BaseJakartaTest.java b/src/test/java/io/openliberty/tools/intellij/lsp4jakarta/it/core/BaseJakartaTest.java index 8f01b8841..15e00c072 100644 --- a/src/test/java/io/openliberty/tools/intellij/lsp4jakarta/it/core/BaseJakartaTest.java +++ b/src/test/java/io/openliberty/tools/intellij/lsp4jakarta/it/core/BaseJakartaTest.java @@ -48,9 +48,7 @@ public abstract class BaseJakartaTest extends MavenImportingTestCase { @Override protected void setUpFixtures() throws Exception { - super.setUpFixtures(); - //myTestFixture = IdeaTestFixtureFactory.getFixtureFactory().createFixtureBuilder(getName()).getFixture(); - //myTestFixture.setUp(); + //Don't call super.setUpFixtures() here, that will create FocusListener leak. myProjectBuilder = IdeaTestFixtureFactory.getFixtureFactory().createFixtureBuilder(getName()); final JavaTestFixtureFactory factory = JavaTestFixtureFactory.getFixtureFactory(); ModuleFixtureBuilder moduleBuilder = myProjectBuilder.addModule(JavaModuleFixtureBuilder.class);