Do not report variable not used if it has assignments. Closes #2427

Author: comintern

Rubberduck.CodeAnalysis/Inspections/Concrete/VariableNotUsedInspection.cs

@@ -22,7 +22,7 @@ protected override IEnumerable<IInspectionResult> DoGetInspectionResults()
                 .Where(declaration =>
                     !declaration.IsWithEvents
                     && !IsIgnoringInspectionResultFor(declaration, AnnotationName)
-                    && declaration.References.All(reference => reference.IsAssignment));
+                    && !declaration.References.Any());
 
             return declarations.Select(issue => 
                 new DeclarationInspectionResult(this,

RubberduckTests/Inspections/VariableNotUsedInspectionTests.cs

@@ -148,12 +148,24 @@ Dim var1 As String
 
         [Test]
         [Category("Inspections")]
-        public void InspectionName()
+        public void VariableUsed_DoesNotReturnResultIfAssigned()
         {
-            const string inspectionName = "VariableNotUsedInspection";
-            var inspection = new VariableNotUsedInspection(null);
+            const string inputCode =
+                @"Function Foo() As Boolean
+    Dim var1 as String
+    var1 = ""test""
+End Function";
+
+            var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out _);
+            using (var state = MockParser.CreateAndParse(vbe.Object))
+            {
+
+                var inspection = new VariableNotUsedInspection(state);
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
-            Assert.AreEqual(inspectionName, inspection.Name);
+                Assert.AreEqual(0, inspectionResults.Count());
+            }
         }
+
     }
 }