From cdcc4d610f71002eaa60d4d5d395edacc3c64bd1 Mon Sep 17 00:00:00 2001
From: Liam Peters <liamjpeters@gmail.com>
Date: Thu, 28 Mar 2024 14:43:24 +0000
Subject: [PATCH] PSReservedParams: Make severity Error instead of Warning

---
 Rules/AvoidReservedParams.cs                 | 13 +++++++++++--
 Tests/Engine/GetScriptAnalyzerRule.tests.ps1 |  6 +++---
 2 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/Rules/AvoidReservedParams.cs b/Rules/AvoidReservedParams.cs
index 7582d4571..4035a9c89 100644
--- a/Rules/AvoidReservedParams.cs
+++ b/Rules/AvoidReservedParams.cs
@@ -60,7 +60,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName) {
                         if (commonParamNames.Contains(paramName, StringComparer.OrdinalIgnoreCase))
                         {
                             yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.ReservedParamsError, funcAst.Name, paramName),
-                                paramAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);
+                                paramAst.Extent, GetName(), GetDiagnosticSeverity(), fileName);
                         }
                     }
                 }
@@ -107,7 +107,16 @@ public SourceType GetSourceType()
         /// <returns></returns>
         public RuleSeverity GetSeverity()
         {
-            return RuleSeverity.Warning;
+            return RuleSeverity.Error;
+        }
+
+        /// <summary>
+        /// Gets the severity of the returned diagnostic record: error, warning, or information.
+        /// </summary>
+        /// <returns></returns>
+        public DiagnosticSeverity GetDiagnosticSeverity()
+        {
+            return DiagnosticSeverity.Error;
         }
 
         /// <summary>
diff --git a/Tests/Engine/GetScriptAnalyzerRule.tests.ps1 b/Tests/Engine/GetScriptAnalyzerRule.tests.ps1
index 93824060a..8d61c1c7f 100644
--- a/Tests/Engine/GetScriptAnalyzerRule.tests.ps1
+++ b/Tests/Engine/GetScriptAnalyzerRule.tests.ps1
@@ -154,17 +154,17 @@ Describe "Test RuleExtension" {
 Describe "TestSeverity" {
     It "filters rules based on the specified rule severity" {
         $rules = Get-ScriptAnalyzerRule -Severity Error
-        $rules.Count | Should -Be 7
+        $rules.Count | Should -Be 8
     }
 
     It "filters rules based on multiple severity inputs"{
         $rules = Get-ScriptAnalyzerRule -Severity Error,Information
-        $rules.Count | Should -Be 18
+        $rules.Count | Should -Be 19
     }
 
         It "takes lower case inputs" {
         $rules = Get-ScriptAnalyzerRule -Severity error
-        $rules.Count | Should -Be 7
+        $rules.Count | Should -Be 8
     }
 }