Merge pull request #5362 from MDoerner/FixFunctionReturnValueDiscardedInspection

Handle output lists (Print statements), fixes FunctionReturnValue(Always)DiscardedInspection

Author: retailcoder

Rubberduck.CodeAnalysis/Inspections/Concrete/FunctionReturnValueAlwaysDiscardedInspection.cs

@@ -144,7 +144,14 @@ private static bool IsCalledAsProcedure(ParserRuleContext context)
                 ? methodCall
                 : context;
             var memberAccessParent = ownFunctionCallExpression.GetAncestor<VBAParser.MemberAccessExprContext>();
-            return memberAccessParent == null;
+            if (memberAccessParent != null)
+            {
+                return false;
+            }
+
+            //If we are in an output list, the value is used somewhere in defining the argument.
+            var outputListParent = context.GetAncestor<VBAParser.OutputListContext>();
+            return outputListParent == null;
         }
 
         protected override string ResultDescription(Declaration declaration)

Rubberduck.CodeAnalysis/Inspections/Concrete/FunctionReturnValueDiscardedInspection.cs

@@ -82,7 +82,9 @@ private static bool IsCalledAsProcedure(ParserRuleContext context)
                 return false;
             }
 
-            return true;
+            //If we are in an output list, the value is used somewhere in defining the argument.
+            var outputListParent = context.GetAncestor<VBAParser.OutputListContext>();
+            return outputListParent == null;
         }
 
         protected override string ResultDescription(IdentifierReference reference)

RubberduckTests/Inspections/FunctionReturnValueAlwaysDiscardedInspectionTests.cs

@@ -323,6 +323,23 @@ End Sub
             Assert.AreEqual(0, InspectionResultsForStandardModule(code).Count());
         }
 
+        [Test]
+        [Category("Inspections")]
+        [Category("Unused Value")]
+        public void OutputListFunctionCall_DoesNotReturnResult()
+        {
+            const string code = @"
+Public Function Foo(ByVal bar As String) As Integer
+    Foo = 42
+End Function
+
+Public Sub Baz()
+    Debug.Print Foo(""Test"")
+End Sub
+";
+            Assert.AreEqual(0, InspectionResultsForStandardModule(code).Count());
+        }
+
         [Test]
         [Category("Inspections")]
         [Category("Unused Value")]

RubberduckTests/Inspections/FunctionReturnValueDiscardedInspectionTests.cs

@@ -281,6 +281,23 @@ End Sub
             Assert.AreEqual(0, InspectionResultsForStandardModule(code).Count());
         }
 
+        [Test]
+        [Category("Inspections")]
+        [Category("Unused Value")]
+        public void FunctionReturnValueDiscarded_DoesNotReturnResult_OutputListFunctionCall()
+        {
+            const string code = @"
+Public Function Foo(ByVal bar As String) As Integer
+    Foo = 42
+End Function
+
+Public Sub Baz()
+    Debug.Print Foo(""Test"")
+End Sub
+";
+            Assert.AreEqual(0, InspectionResultsForStandardModule(code).Count());
+        }
+
         [Test]
         [Category("Inspections")]
         [Category("Unused Value")]