entlicher commented on a change in pull request #3047:
URL: https://github.com/apache/netbeans/pull/3047#discussion_r669610318
##########
File path:
java/java.lsp.server/test/unit/src/org/netbeans/modules/java/lsp/server/protocol/ServerTest.java
##########
@@ -1604,6 +1604,21 @@ public void showMessage(MessageParams params) {
public void logMessage(MessageParams arg0) {
throw new UnsupportedOperationException("Not supported yet.");
}
+
+ @Override
+ public CompletableFuture<String>
createTextEditorDecoration(DecorationRenderOptions params) {
+ throw new UnsupportedOperationException("Not supported yet.");
Review comment:
Well, all other methods in this class use
`UnsupportedOperationException`. It's a dummy test client, which is used just
in the test of imports, which is unrelated to a bunch of methods that the
client has.
It'd not be a mistake if the test call these methods, thus no need for
`fail()`. It's more like we know that the test will not touch it and thus we do
not bother to implement it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
