Completion Unit Test: complete returns null

Please show the whole test method's code and verify you have "<caret>" marker in the test data file where completion is invoked on.

Sure, please find them below. I can confirm that getVariants on my reference gets invoked correctly.

Test class:

public class CompletionTest extends LightCodeInsightFixtureTestCase {
  @Override
  protected String getTestDataPath() {
    return "src/test/resources/testData/completion";
  }

  public void testStaticImportCompletion() {
    myFixture.configureByFiles("CompletionWithStaticImport.es", "zzz.es");
    LookupElement[] complete = myFixture.complete(CompletionType.BASIC, 1); // this is null
    List<String> strings = myFixture.getLookupElementStrings(); // this is null too
//    assertNotNull(strings);
//    assertEquals(1, strings.size());
//    assertContainsElements(strings, "ZZLong");
  }
}

CompletionWithStaticImport.es:

namespace some

import zzz.ZZLong

record RRR {
  field: Z<caret>
}

zzz.es:

namespace zzz

long ZZLong

thanks

WRONG, PLEASE IGNORE

It seems the javadoc for com.intellij.testFramework.fixtures.CodeInsightTestFixture#configureByFiles is misleading, good catch.

Please change order in arguments in

myFixture.configureByFiles("CompletionWithStaticImport.es", "zzz.es");

so that file containing <caret> is last.

AFAIU copyFileToProject() for zzz.es should be enough and slightly faster.

Hm, I'm not sure you're correct.. I did as you said and got non-null results, but looks like it tried to invoke completion at the very beginning of zzz.es. When I added a <caret> to zzz.es, I got nulls again.

As a test I reverted configureByFiles call and added this:

myFixture.configureByFiles("CompletionWithStaticImport.es", "zzz.es");
PsiElement element = myFixture.getFile().findElementAt(myFixture.getCaretOffset()); // <--
LookupElement[] complete = myFixture.complete(CompletionType.BASIC, 1);

Inspecting `element` in the debugger shows that the framework got it right:

(field_decl ('field' ':' ' ' (type_ref ...)) ' ' '}'

element is that last PsiWhiteSpace before '}'.

Besides, I have another test that uses configureByFiles(..) with 2 arguments, and having <caret> in the first of them works just fine.

You're right, I missed the loop iterates backwards and got additionally confused by looking at a sample where both files have <caret>.

Did you check there are multiple completion variants at this position? <null> is expected result if "if the only item was auto-completed" (javadoc).

There's only one variant indeed.. how do I check for it then?

Edit: nevermind, found it. Just another method in CodeInsightTestFixture.

Thanks!

Konstantin Sobolev hey mate, I also have similar situation. Which method did u use instead of complete()?

Otto Dvali I didn't touch this project for a few years now, but the answer is probably here: https://github.com/SumoLogic/epigraph/blob/master/idea-plugin/src/test/java/ws/epigraph/ideaplugin/schema/features/completion/CompletionTest.java

thanks Konstantin Sobolev