Skip import quickfix hierarchy check if base type is unresolved (eclipse-jdt#2120)

The commit 546a697 extended the import
check to compare the interfaces of the suggested types with the
interface of the missing type.

In case the base type is unresolved, this then skips all suggestions,
even though it should let all pass so that the user can decide which one
to pick.

Fixes eclipse-jdt#2119

Author: ptziegler

org.eclipse.jdt.core.manipulation/common/org/eclipse/jdt/internal/ui/text/correction/UnresolvedElementsBaseSubProcessor.java

@@ -1007,7 +1007,7 @@ private void addSimilarTypeProposals(int kind, ICompilationUnit cu, Name node, i
 			if ((elem.getKind() & TypeKinds.ALL_TYPES) != 0) {
 				String fullName= elem.getName();
 				if (!fullName.equals(resolvedTypeName)) {
-					if (simpleBinding != null && !simpleBinding.isPrimitive()) {
+					if (simpleBinding != null && !simpleBinding.isPrimitive() && !simpleBinding.isRecovered()) {
 						// If we have an expected type, we should verify that any classes we suggest to import
 						// inherit directly or indirectly from the type
 						ITypeBinding qualifiedTypeBinding= null;

org.eclipse.jdt.ui.tests/ui/org/eclipse/jdt/ui/tests/quickfix/UnresolvedTypesQuickFixTest.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2000, 2023 IBM Corporation and others.
+ * Copyright (c) 2000, 2025 IBM Corporation and others.
  *
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License 2.0
@@ -18,6 +18,7 @@
 
 import java.util.ArrayList;
 import java.util.Hashtable;
+import java.util.List;
 
 import org.junit.After;
 import org.junit.Before;
@@ -1788,4 +1789,58 @@ public void foo() {
 
 		assertExpectedExistInProposals(proposals, expected);
 	}
+
+	@Test
+	public void testIssue2119() throws Exception {
+		IPackageFragment other = fSourceFolder.createPackageFragment("other", false, null);
+		String source1 = """
+				package other;
+				public interface IPath {}
+				""";
+		String source2 = """
+				package other;
+				public class Path implements IPath {}
+				""";
+		other.createCompilationUnit("IPath.java", source1, false, null);
+		other.createCompilationUnit("Path.java", source2, false, null);
+		IPackageFragment test= fSourceFolder.createPackageFragment("test", false, null);
+		String source = """
+				package test;
+				class Test {
+					private IPath p = new Path();
+				}
+				""";
+		ICompilationUnit cu = test.createCompilationUnit("Test.java", source, false, null);
+		CompilationUnit astRoot = getASTRoot(cu);
+
+		// import java.util.List
+		List<IJavaCompletionProposal> proposals1 = collectCorrections(cu, astRoot, 2, 0);
+		assertCorrectLabels(proposals1);
+		String[] expected1 = new String[1];
+		expected1[0] = """
+				package test;
+
+				import other.IPath;
+
+				class Test {
+					private IPath p = new Path();
+				}
+				""";
+		assertExpectedExistInProposals(proposals1, expected1);
+
+		// import java.util.ArrayList
+		List<IJavaCompletionProposal> proposals2 = collectCorrections(cu, astRoot, 2, 1);
+		assertCorrectLabels(proposals2);
+		String[] expected2 = new String[1];
+		expected2[0] = """
+				package test;
+
+				import other.Path;
+
+				class Test {
+					private IPath p = new Path();
+				}
+				""";
+		assertExpectedExistInProposals(proposals2, expected2);
+	}
 }