Update handling of empty AST passed to AnalyzeScript

liamjpeters · liamjpeters · commit 0abeb303579f · 2025-08-18T16:31:44.000+01:00
diff --git a/Rules/UseSingleValueFromPipelineParameter.cs b/Rules/UseSingleValueFromPipelineParameter.cs
@@ -36,7 +36,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
         {
             if (ast == null)
             {
-                yield break;
+                throw new ArgumentNullException(Strings.NullAstErrorMessage);
             }
             // Find all param blocks that have a Parameter attribute with
             // ValueFromPipeline set to true.

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)`
`36`	`36`	`{`
`37`	`37`	`if (ast == null)`
`38`	`38`	`{`
`39`		`- yield break;`
	`39`	`+ throw new ArgumentNullException(Strings.NullAstErrorMessage);`
`40`	`40`	`}`
`41`	`41`	`// Find all param blocks that have a Parameter attribute with`
`42`	`42`	`// ValueFromPipeline set to true.`