No highlighting for 'no actuator' warning hovers.

This commit is contained in:
Kris De Volder
2017-11-01 10:52:49 -07:00
parent b1d8c988ff
commit 007d5c92a5
3 changed files with 4 additions and 4 deletions

View File

@@ -145,7 +145,8 @@ public class BootJavaHoverProvider implements HoverHandler {
result.addAll(hints);
}
} else {
ASTUtils.nameRange(doc, annotation).ifPresent(result::add);
//Do nothing... we don't want a highlight for the 'no actuator warning'
//ASTUtils.nameRange(doc, annotation).ifPresent(result::add);
}
});
}

View File

@@ -14,7 +14,6 @@ import java.util.Optional;
import org.eclipse.jdt.core.dom.Annotation;
import org.eclipse.jdt.core.dom.MethodDeclaration;
import org.springframework.ide.vscode.boot.java.beans.BeansSymbolProvider;
import org.springframework.ide.vscode.boot.java.utils.ASTUtils;
import org.springframework.ide.vscode.commons.boot.app.cli.livebean.LiveBean;
import org.springframework.ide.vscode.commons.util.Optionals;

View File

@@ -67,7 +67,7 @@ public class ActuatorWarningHoverTest {
"}"
);
editor.assertHighlights("@Profile");
editor.assertHighlights(/*NONE*/);
editor.assertHoverContains("@Profile", "No live hover information");
editor.assertHoverContains("@Profile", "Consider adding `spring-boot-actuator` as a dependency to your project `"+projectName+"`");
}
@@ -168,7 +168,7 @@ public class ActuatorWarningHoverTest {
" }\n" +
"}"
);
editor.assertHighlights("@Bean");
editor.assertHighlights(/*NONE*/);
editor.assertHoverContains("@Bean", "No live hover information");
editor.assertNoHover("the-bean-name");
}