quickfix needs evaluator to return true when Set keyword is present

Author: retailcoder

Rubberduck.Inspections/VariableRequiresSetAssignmentEvaluator.cs

@@ -32,6 +32,7 @@ public static bool RequiresSetAssignment(IdentifierReference reference, Rubberdu
             if (setStmtContext != null)
             {
                 // assignment already has a Set keyword
+                return true;
                 // (but is it misplaced? ...hmmm... beyond the scope of *this* inspection though)
                 // if we're only ever assigning to 'Nothing', might as well flag it though
                 if (reference.Declaration.References.Where(r => r.IsAssignment).All(r =>

RubberduckTests/Inspections/ObjectVariableNotSetInspectionTests.cs

@@ -117,7 +117,7 @@ End Sub
         [Category("Inspections")]
         public void ObjectVariableNotSet_GivenIndexerObjectAccess_ReturnsResult()
         {
-            var expectResultCount = 0;
+            var expectResultCount = 1;
             var input =
 @"
 Private Sub DoSomething()
@@ -175,9 +175,9 @@ Private Sub TestSub(ByRef testParam As Variant)
 
         [Test]
         [Category("Inspections")]
-        public void ObjectVariableNotSet_GivenVariantVariableAssignedRange_ReturnsNoResult()
+        public void ObjectVariableNotSet_GivenVariantVariableAssignedRange_ReturnsResult()
         {
-            var expectResultCount = 0;
+            var expectResultCount = 1;
             var input =
 @"
 Private Sub TestSub(ByRef testParam As Variant)
@@ -188,9 +188,9 @@ Private Sub TestSub(ByRef testParam As Variant)
 
         [Test]
         [Category("Inspections")]
-        public void ObjectVariableNotSet_GivenVariantVariableAssignedDeclaredRange_ReturnsNoResult()
+        public void ObjectVariableNotSet_GivenVariantVariableAssignedDeclaredRange_ReturnsResult()
         {
-            var expectResultCount = 0;
+            var expectResultCount = 1;
             var input =
 @"
 Private Sub TestSub(ByRef testParam As Variant, target As Range)