Update after review and rebase: Fix FIXME

SonarSource · Apr 13, 2023 · 37e7dd6 · 37e7dd6
1 parent 9dbaae6
commit 37e7dd6
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/...lyzer.VisualBasic/SymbolicExecution/Roslyn/PublicMethodArgumentsShouldBeCheckedForNull.cs b/...lyzer.VisualBasic/SymbolicExecution/Roslyn/PublicMethodArgumentsShouldBeCheckedForNull.cs
@@ -80,7 +80,6 @@ public ArgumentDereferenceWalker(ISet<string> argumentNames) =>
 
         public override void Visit(SyntaxNode node)
         {
-            // FIXME: UTs
             if (!DereferencesMethodArguments && !node.IsAnyKind(SingleLineSubLambdaExpression, MultiLineSubLambdaExpression, SingleLineFunctionLambdaExpression, MultiLineFunctionLambdaExpression))
             {
                 base.Visit(node);

diff --git a/...nitTest/TestCases/SymbolicExecution/Roslyn/PublicMethodArgumentsShouldBeCheckedForNull.vb b/...nitTest/TestCases/SymbolicExecution/Roslyn/PublicMethodArgumentsShouldBeCheckedForNull.vb
@@ -535,3 +535,18 @@ Public Class Nancy_Repros
     End Sub
 
 End Class
+
+Public Class ShouldExecuteLambdas
+
+    Public Sub Lambdas(Arg As Object)
+        Dim SingleLineFunction As Func(Of String) = Function() Arg.ToString
+        Dim MultiLineFunction As Func(Of String) = Function()
+                                                       Return Arg.ToString
+                                                   End Function
+        Dim SingleLineAction As Action = Sub() Arg.ToString()
+        Dim MultiLineAction As Action = Sub()
+                                            Arg.ToString()
+                                        End Sub
+    End Sub
+
+End Class