PY-6955 Unused import not greyed out if also failing

python/src/com/jetbrains/python/inspections/PyUnresolvedReferencesInspection.java

@@ -984,6 +984,10 @@ public class PyUnresolvedReferencesInspection extends PyInspection {
             final PyImportElement importElement = (PyImportElement)unusedImport;
             final PsiElement element = importElement.resolve();
             if (element == null) {
+              if (importElement.getImportedQName() != null) {
+                //Mark import as unused even if it can't be resolved
+                result.add(importElement.getParent());
+              }
               continue;
             }
             if (dunderAll != null && dunderAll.contains(importElement.getVisibleName())) {

python/testData/inspections/PyUnresolvedReferencesInspection/FromPackageImportBuiltin/a.py

@@ -1 +1,3 @@
 from importSource import <error descr="Unresolved reference 'len'">len</error>
+
+len()

python/testData/inspections/PyUnresolvedReferencesInspection/UnusedUnresolvedNameImported/a.py

@@ -0,0 +1 @@
+<warning descr="Unused import statement">from my_module import <error descr="Unresolved reference 'eggs'">eggs</error></warning>

python/testData/inspections/PyUnresolvedReferencesInspection/UnusedUnresolvedNameImported/my_module.py

@@ -0,0 +1,2 @@
+def my_func():
+    pass

python/testData/inspections/PyUnresolvedReferencesInspection/UnusedUnresolvedNameImportedSeveralTimes/a.py

@@ -0,0 +1,2 @@
+<warning descr="Unused import statement">from my_module import <error descr="Unresolved reference 'eggs'">eggs</error></warning>
+<warning descr="Unused import statement">from my_module import <error descr="Unresolved reference 'eggs'">eggs</error></warning>

python/testData/inspections/PyUnresolvedReferencesInspection/UnusedUnresolvedNameImportedSeveralTimes/my_module.py

@@ -0,0 +1,2 @@
+def my_func():
+    pass

python/testData/inspections/PyUnresolvedReferencesInspection/UsedUnresolvedNameImportedSeveralTimes/a.py

@@ -0,0 +1,4 @@
+from my_package.my_module import <error descr="Unresolved reference 'eggs'">eggs</error>
+from my_package.my_module import <error descr="Unresolved reference 'eggs'">eggs</error>
+
+eggs()

python/testData/inspections/PyUnresolvedReferencesInspection/UsedUnresolvedNameImportedSeveralTimes/my_package/__init__.py

@@ -0,0 +1 @@
+__author__ = 'Ilya.Kazakevich'

python/testData/inspections/PyUnresolvedReferencesInspection/UsedUnresolvedNameImportedSeveralTimes/my_package/my_module.py

@@ -0,0 +1,2 @@
+def my_func():
+    pass

python/testData/inspections/PyUnresolvedReferencesInspection/importFunction.py

@@ -1 +1,3 @@
 import collections.<warning descr="No module named OrderedDict">OrderedDict</warning>
+
+collections.OrderedDict()

python/testData/inspections/PyUnresolvedReferencesInspection/unresolvedImport.py

@@ -1 +1,3 @@
 import <error descr="No module named wurm">wurm</error>
+
+wurm()

python/testData/inspections/PyUnresolvedReferencesInspection/unusedUnresolvedModuleImported.py

@@ -0,0 +1 @@
+<warning descr="Unused import statement">import <error descr="No module named spam">spam</error></warning>

python/testData/inspections/PyUnresolvedReferencesInspection/unusedUnresolvedPackageImported.py

@@ -0,0 +1 @@
+<warning descr="Unused import statement">import <error descr="No module named spam">spam</error>.eggs</warning>

python/testSrc/com/jetbrains/python/fixtures/PyInspectionTestCase.java

@@ -28,6 +28,9 @@ public abstract class PyInspectionTestCase extends PyTestCase {
     configureInspection();
   }
 
+  protected void doMultiFileTest() {
+    doMultiFileTest("a.py");
+  }
   protected void doMultiFileTest(@NotNull String filename) {
     myFixture.copyDirectoryToProject(getTestDirectory(false), "");
     myFixture.configureFromTempProjectFile(filename);

python/testSrc/com/jetbrains/python/inspections/PyUnresolvedReferencesInspectionTest.java

@@ -335,6 +335,31 @@ public class PyUnresolvedReferencesInspectionTest extends PyInspectionTestCase {
     doTest();
   }
 
+  // PY-6955
+  public void testUnusedUnresolvedModuleImported() {
+    doTest();
+  }
+
+  // PY-6955
+  public void testUnusedUnresolvedNameImported() {
+    doMultiFileTest();
+  }
+
+  // PY-6955
+  public void testUnusedUnresolvedNameImportedSeveralTimes() {
+    doMultiFileTest();
+  }
+
+  // PY-6955
+  public void testUsedUnresolvedNameImportedSeveralTimes() {
+    doMultiFileTest();
+  }
+
+  // PY-6955
+  public void testUnusedUnresolvedPackageImported() {
+    doTest();
+  }
+
   @NotNull
   @Override
   protected Class<? extends PyInspection> getInspectionClass() {