Incorrect signatures returned assertEquals

[yyoncho]

        long num = 10;
        assertEquals(num, num);

signature help shows (Object[], Object[]) as the active signature instead of (long, long).

[NikolasKomonen]

Hi, I tried to recreate your issue and I got the following using the code you provided.

Could you share some more information about your setup, Java version, if everything is up to date,
a screenshot of the signature if possible, ...

[yyoncho]

@NikolasKomonen AFAICS you are showing the information from textDocument/hover. I am talking about the information retrieved via textDocument/signatureHelp. Let me know if you still cannot reproduce the issue.

[fbricon]

@NikolasKomonen in vscode, you need to enable signatureHelp with "java.signatureHelp.enabled": true in the settings.
Then use ctrl+shift+space. This is definitively wonky:

[NikolasKomonen]

Thanks, I just tested it. For sure not doing what it is supposed to. I'll look into it.

[yyoncho]

@fbricon one more thing: I noticed that ParameterInformation.documentation field is not populated. Is this intentional?

[fbricon]

@yyoncho I don't think so. Please open a separate ticket for that problem

[NikolasKomonen]

I found the problem area, but I think @snjeza would be more knowledgeable since she worked on it.

package com.example.demo;

import static org.junit.Assert.assertEquals;
import org.junit.Test;

public class DemoApplicationTests {

	@Test
	public void contextLoads() {
		long num = 3;
		assertEquals(num,num);
	}

}

If I do Ctrl+Shift+Space here:

assertEquals(|num,num);

the variable help2 stays null

but when I do it here:

assertEquals(n|um,num);

It triggers this case and populates the help2 variable with completely wrong/different signatures, which I'm not sure exactly what this specific case is for:
https://github.com/eclipse/eclipse.jdt.ls/blob/master/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/SignatureHelpHandler.java#L86

I just wanted to see if you knew what to fix before I dove deeper into the issue.