From 89d56ca658ef8d457d269e6e75a91e4998b5837b Mon Sep 17 00:00:00 2001 From: aboyko Date: Mon, 28 Aug 2023 22:49:45 -0400 Subject: [PATCH] Turn on logging for a test suite class with the failing test --- .../vscode/boot/app/SpringSymbolIndex.java | 8 +++---- .../boot/java/utils/SpringIndexerXML.java | 2 +- .../utils/test/XmlBeansHyperlinkTest.java | 24 ++++++++++++++++++- 3 files changed, 28 insertions(+), 6 deletions(-) diff --git a/headless-services/spring-boot-language-server/src/main/java/org/springframework/ide/vscode/boot/app/SpringSymbolIndex.java b/headless-services/spring-boot-language-server/src/main/java/org/springframework/ide/vscode/boot/app/SpringSymbolIndex.java index 1fad39d26..8330acab9 100644 --- a/headless-services/spring-boot-language-server/src/main/java/org/springframework/ide/vscode/boot/app/SpringSymbolIndex.java +++ b/headless-services/spring-boot-language-server/src/main/java/org/springframework/ide/vscode/boot/app/SpringSymbolIndex.java @@ -861,18 +861,18 @@ public class SpringSymbolIndex implements InitializingBean, SpringIndex { this.project = project; this.indexer = indexer; this.clean = clean; - log.debug("{} created ", this); + log.debug("{} created ", indexer.getClass().getSimpleName()); } @Override public void run() { - log.debug("{} starting...", this); + log.debug("{} starting...", indexer.getClass().getSimpleName()); try { indexer.initializeProject(project, this.clean); - log.debug("{} completed", this); + log.debug("{} completed", indexer.getClass().getSimpleName()); } catch (Throwable e) { - log.error("{} threw exception", this, e); + log.error("{} threw exception", indexer.getClass().getSimpleName(), e); } } } diff --git a/headless-services/spring-boot-language-server/src/main/java/org/springframework/ide/vscode/boot/java/utils/SpringIndexerXML.java b/headless-services/spring-boot-language-server/src/main/java/org/springframework/ide/vscode/boot/java/utils/SpringIndexerXML.java index f151c534e..70aba1c89 100644 --- a/headless-services/spring-boot-language-server/src/main/java/org/springframework/ide/vscode/boot/java/utils/SpringIndexerXML.java +++ b/headless-services/spring-boot-language-server/src/main/java/org/springframework/ide/vscode/boot/java/utils/SpringIndexerXML.java @@ -146,7 +146,7 @@ public class SpringIndexerXML implements SpringIndexer { long endTime = System.currentTimeMillis(); - log.info("scan xml files for symbols for project: " + project.getElementName() + " took ms: " + (endTime - startTime)); + log.info("scan xml files for symbols for project: " + project.getElementName() + " took ms: " + (endTime - startTime) + " Symbols Found: " + symbols.length); } @Override diff --git a/headless-services/spring-boot-language-server/src/test/java/org/springframework/ide/vscode/boot/java/utils/test/XmlBeansHyperlinkTest.java b/headless-services/spring-boot-language-server/src/test/java/org/springframework/ide/vscode/boot/java/utils/test/XmlBeansHyperlinkTest.java index d77b734b2..60f52f6f9 100644 --- a/headless-services/spring-boot-language-server/src/test/java/org/springframework/ide/vscode/boot/java/utils/test/XmlBeansHyperlinkTest.java +++ b/headless-services/spring-boot-language-server/src/test/java/org/springframework/ide/vscode/boot/java/utils/test/XmlBeansHyperlinkTest.java @@ -23,8 +23,12 @@ import org.eclipse.lsp4j.Location; import org.eclipse.lsp4j.LocationLink; import org.eclipse.lsp4j.Position; import org.eclipse.lsp4j.Range; +import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; +import org.slf4j.LoggerFactory; +import ch.qos.logback.classic.Level; +import ch.qos.logback.classic.Logger; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.boot.test.autoconfigure.OverrideAutoConfiguration; import org.springframework.boot.test.context.SpringBootTest; @@ -56,6 +60,9 @@ import com.google.gson.Gson; @DirtiesContext(classMode=ClassMode.AFTER_EACH_TEST_METHOD) public class XmlBeansHyperlinkTest { + + private static final org.slf4j.Logger log = LoggerFactory.getLogger(XmlBeansHyperlinkTest.class); + private static final int DEFAULT_INDEX_WAIT_TIME = 5; @Autowired private BootLanguageServerHarness harness; @Autowired private SpringSymbolIndex indexer; @@ -64,9 +71,16 @@ public class XmlBeansHyperlinkTest { private ProjectsHarness projects = ProjectsHarness.INSTANCE; private MavenJavaProject project; + + private Level originalLevel; @BeforeEach public void setup() throws Exception { + final Logger logger = (Logger)LoggerFactory.getLogger(Logger.ROOT_LOGGER_NAME); + originalLevel = logger.getLevel(); + logger.setLevel(Level.DEBUG); + + log.debug("-------------------------------------------------"); harness.intialize(null); Map supportXML = new HashMap<>(); @@ -87,7 +101,14 @@ public class XmlBeansHyperlinkTest { projectObserver.doWithListeners(l -> l.created(project)); CompletableFuture initProject = indexer.waitOperation(); - initProject.get(DEFAULT_INDEX_WAIT_TIME, TimeUnit.SECONDS); + initProject.get(1500, TimeUnit.SECONDS); + } + + @AfterEach + public void tearDown() { + log.debug("-------------------------------------------------"); + final Logger logger = (Logger)LoggerFactory.getLogger(Logger.ROOT_LOGGER_NAME); + logger.setLevel(originalLevel); } @Test @@ -149,6 +170,7 @@ public class XmlBeansHyperlinkTest { @Test void testBeanRefHyperlink() throws Exception { + log.debug("------------------ testBeanRefHyperlink ----------------------"); Path xmlFilePath = Paths.get(project.getLocationUri()).resolve("beans.xml"); Editor editor = harness.newEditor(LanguageId.XML, "\n" +