Add RuleSuppressionIDs for rules

Quoc Truong · Quoc Truong · commit cd8176ea473b · 2015-04-20T12:08:39.000-07:00
diff --git a/Engine/Generic/AvoidParameterGeneric.cs b/Engine/Generic/AvoidParameterGeneric.cs
@@ -35,7 +35,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
                     {
                         if (ParameterCondition(cmdAst, ceAst))
                         {
-                            yield return new DiagnosticRecord(GetError(fileName, cmdAst), cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);
+                            yield return new DiagnosticRecord(GetError(fileName, cmdAst), cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, cmdAst.GetCommandName());
                         }
                     }
                 }
diff --git a/Engine/Generic/RuleSuppression.cs b/Engine/Generic/RuleSuppression.cs
@@ -26,6 +26,15 @@ public class RuleSuppression
     {
         private string _ruleName;
 
+        /// <summary>
+        /// The start offset of the rule suppression attribute (not where it starts to apply)
+        /// </summary>
+        public int StartAttributeLine
+        {
+            get;
+            set;
+        }
+
         /// <summary>
         /// The start offset of the rule suppression
         /// </summary>
@@ -133,6 +142,7 @@ public RuleSuppression(AttributeAst attrAst, int start, int end)
 
             if (attrAst != null)
             {
+                StartAttributeLine = attrAst.Extent.StartLineNumber;
                 var positionalArguments = attrAst.PositionalArguments;
                 var namedArguments = attrAst.NamedArguments;
 
@@ -262,24 +272,26 @@ public RuleSuppression(AttributeAst attrAst, int start, int end)
 
             if (!String.IsNullOrWhiteSpace(Error))
             {
-                Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, attrAst.Extent.StartLineNumber,
+                Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, StartAttributeLine,
                     System.IO.Path.GetFileName(attrAst.Extent.File), Error);
             }
         }
 
         /// <summary>
-        /// Constructs rule expression from rule name, id, start and end
+        /// Constructs rule expression from rule name, id, start, end and startAttributeLine
         /// </summary>
         /// <param name="ruleName"></param>
         /// <param name="ruleSuppressionID"></param>
         /// <param name="start"></param>
         /// <param name="end"></param>
-        public RuleSuppression(string ruleName, string ruleSuppressionID, int start, int end)
+        /// <param name="startAttributeLine"></param>
+        public RuleSuppression(string ruleName, string ruleSuppressionID, int start, int end, int startAttributeLine)
         {
             RuleName = ruleName;
             RuleSuppressionID = ruleSuppressionID;
             StartOffset = start;
             EndOffset = end;
+            StartAttributeLine = startAttributeLine;
         }
 
         /// <summary>
@@ -336,15 +348,15 @@ public static List<RuleSuppression> GetSuppressions(IEnumerable<AttributeAst> at
                     {
                         if (targetAsts.Count() == 0)
                         {
-                            ruleSupp.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSupp.StartOffset,
+                            ruleSupp.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSupp.StartAttributeLine,
                                 System.IO.Path.GetFileName(scopeAst.Extent.File), String.Format(Strings.TargetCannotBeFoundError, ruleSupp.Target, ruleSupp.Scope));
                             result.Add(ruleSupp);
                             continue;
                         }
 
                         foreach (Ast targetAst in targetAsts)
                         {
-                            result.Add(new RuleSuppression(ruleSupp.RuleName, ruleSupp.RuleSuppressionID, targetAst.Extent.StartOffset, targetAst.Extent.EndOffset));
+                            result.Add(new RuleSuppression(ruleSupp.RuleName, ruleSupp.RuleSuppressionID, targetAst.Extent.StartOffset, targetAst.Extent.EndOffset, attributeAst.Extent.StartLineNumber));
                         }
                     }
 
diff --git a/Engine/Helper.cs b/Engine/Helper.cs
@@ -773,7 +773,7 @@ public List<DiagnosticRecord> SuppressRule(string ruleName, Dictionary<string, L
                     // If we cannot found any error but the rulesuppression has a rulesuppressionid then it must be used wrongly
                     if (!String.IsNullOrWhiteSpace(ruleSuppression.RuleSuppressionID) && suppressionCount == 0)
                     {
-                        ruleSuppression.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSuppression.StartOffset,
+                        ruleSuppression.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSuppression.StartAttributeLine,
                                 System.IO.Path.GetFileName(record.Extent.File), String.Format(Strings.RuleSuppressionIDError, ruleSuppression.RuleSuppressionID));
                         Helper.Instance.MyCmdlet.WriteError(new ErrorRecord(new ArgumentException(ruleSuppression.Error), ruleSuppression.Error, ErrorCategory.InvalidArgument, ruleSuppression));
                     }
@@ -811,7 +811,7 @@ public List<DiagnosticRecord> SuppressRule(string ruleName, Dictionary<string, L
                     // If we cannot found any error but the rulesuppression has a rulesuppressionid then it must be used wrongly
                     if (!String.IsNullOrWhiteSpace(ruleSuppression.RuleSuppressionID) && suppressionCount == 0)
                     {
-                        ruleSuppression.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSuppression.StartOffset,
+                        ruleSuppression.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSuppression.StartAttributeLine,
                                 System.IO.Path.GetFileName(record.Extent.File), String.Format(Strings.RuleSuppressionIDError, ruleSuppression.RuleSuppressionID));
                         Helper.Instance.MyCmdlet.WriteError(new ErrorRecord(new ArgumentException(ruleSuppression.Error), ruleSuppression.Error, ErrorCategory.InvalidArgument, ruleSuppression));
                     }
diff --git a/Rules/AvoidAlias.cs b/Rules/AvoidAlias.cs
@@ -35,17 +35,18 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
             foreach (Ast foundAst in foundAsts)
             {
                 CommandAst cmdAst = (CommandAst)foundAst;
+                string aliasName = cmdAst.GetCommandName();
                 // Handles the exception caused by commands like, {& $PLINK $args 2> $TempErrorFile}.
                 // You can also review the remark section in following document,
                 // MSDN: CommandAst.GetCommandName Method
-                if (cmdAst.GetCommandName() == null) continue;
+                if (aliasName == null) continue;
 
-                string cmdletName = Microsoft.Windows.Powershell.ScriptAnalyzer.Helper.Instance.GetCmdletNameFromAlias(cmdAst.GetCommandName());
+                string cmdletName = Microsoft.Windows.Powershell.ScriptAnalyzer.Helper.Instance.GetCmdletNameFromAlias(aliasName);
 
                 if (!String.IsNullOrEmpty(cmdletName))
                 {
-                    yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.AvoidUsingCmdletAliasesError, cmdAst.GetCommandName(), cmdletName),
-                        cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);
+                    yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.AvoidUsingCmdletAliasesError, aliasName, cmdletName),
+                        cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, aliasName);
                 }
             }
         }
diff --git a/Rules/AvoidGlobalVars.cs b/Rules/AvoidGlobalVars.cs
@@ -40,7 +40,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
                         yield return
                             new DiagnosticRecord(
                                 string.Format(CultureInfo.CurrentCulture, Strings.AvoidGlobalVarsError,
-                                    varAst.VariablePath.UserPath), varAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);
+                                    varAst.VariablePath.UserPath), varAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, varAst.VariablePath.UserPath);
                     }
                 }
             }
diff --git a/Rules/AvoidParameterGeneric.cs b/Rules/AvoidParameterGeneric.cs
diff --git a/Rules/AvoidPositionalParameters.cs b/Rules/AvoidPositionalParameters.cs
@@ -45,7 +45,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
                     && Helper.Instance.PositionalParameterUsed(cmdAst))
                 {
                     yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.AvoidUsingPositionalParametersError, cmdAst.GetCommandName()),
-                        cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);
+                        cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, cmdAst.GetCommandName());
                 }
             }
         }
diff --git a/Rules/AvoidReservedParams.cs b/Rules/AvoidReservedParams.cs
@@ -63,7 +63,8 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName) {
                                     StringComparison.OrdinalIgnoreCase))
                                 {
                                     string funcName = string.Format(CultureInfo.CurrentCulture,Strings.ReservedParamsCmdletPrefix, (parentAst as FunctionDefinitionAst).Name);
-                                    yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.ReservedParamsError, funcName,paramName), paramAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);
+                                    yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.ReservedParamsError, funcName,paramName),
+                                        paramAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, paramName);
                                    
                                 }
                             }
diff --git a/Rules/UseDeclaredVarsMoreThanAssignments.cs b/Rules/UseDeclaredVarsMoreThanAssignments.cs
@@ -93,7 +93,8 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
 
             foreach (string key in assignments.Keys)
             {
-                yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.UseDeclaredVarsMoreThanAssignmentsError, key), assignments[key].Extent, GetName(), DiagnosticSeverity.Warning, fileName);
+                yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.UseDeclaredVarsMoreThanAssignmentsError, key),
+                    assignments[key].Extent, GetName(), DiagnosticSeverity.Warning, fileName, key);
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)`
`35`	`35`	`{`
`36`	`36`	`if (ParameterCondition(cmdAst, ceAst))`
`37`	`37`	`{`
`38`		`- yield return new DiagnosticRecord(GetError(fileName, cmdAst), cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);`
	`38`	`+ yield return new DiagnosticRecord(GetError(fileName, cmdAst), cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, cmdAst.GetCommandName());`
`39`	`39`	`}`
`40`	`40`	`}`
`41`	`41`	`}`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,15 @@ public class RuleSuppression`
`26`	`26`	`{`
`27`	`27`	`private string _ruleName;`
`28`	`28`
	`29`	`+ /// <summary>`
	`30`	`+ /// The start offset of the rule suppression attribute (not where it starts to apply)`
	`31`	`+ /// </summary>`
	`32`	`+ public int StartAttributeLine`
	`33`	`+ {`
	`34`	`+ get;`
	`35`	`+ set;`
	`36`	`+ }`
	`37`	`+`
`29`	`38`	`/// <summary>`
`30`	`39`	`/// The start offset of the rule suppression`
`31`	`40`	`/// </summary>`
`@@ -133,6 +142,7 @@ public RuleSuppression(AttributeAst attrAst, int start, int end)`
`133`	`142`
`134`	`143`	`if (attrAst != null)`
`135`	`144`	`{`
	`145`	`+ StartAttributeLine = attrAst.Extent.StartLineNumber;`
`136`	`146`	`var positionalArguments = attrAst.PositionalArguments;`
`137`	`147`	`var namedArguments = attrAst.NamedArguments;`
`138`	`148`
`@@ -262,24 +272,26 @@ public RuleSuppression(AttributeAst attrAst, int start, int end)`
`262`	`272`
`263`	`273`	`if (!String.IsNullOrWhiteSpace(Error))`
`264`	`274`	`{`
`265`		`- Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, attrAst.Extent.StartLineNumber,`
	`275`	`+ Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, StartAttributeLine,`
`266`	`276`	`System.IO.Path.GetFileName(attrAst.Extent.File), Error);`
`267`	`277`	`}`
`268`	`278`	`}`
`269`	`279`
`270`	`280`	`/// <summary>`
`271`		`- /// Constructs rule expression from rule name, id, start and end`
	`281`	`+ /// Constructs rule expression from rule name, id, start, end and startAttributeLine`
`272`	`282`	`/// </summary>`
`273`	`283`	`/// <param name="ruleName"></param>`
`274`	`284`	`/// <param name="ruleSuppressionID"></param>`
`275`	`285`	`/// <param name="start"></param>`
`276`	`286`	`/// <param name="end"></param>`
`277`		`- public RuleSuppression(string ruleName, string ruleSuppressionID, int start, int end)`
	`287`	`+ /// <param name="startAttributeLine"></param>`
	`288`	`+ public RuleSuppression(string ruleName, string ruleSuppressionID, int start, int end, int startAttributeLine)`
`278`	`289`	`{`
`279`	`290`	`RuleName = ruleName;`
`280`	`291`	`RuleSuppressionID = ruleSuppressionID;`
`281`	`292`	`StartOffset = start;`
`282`	`293`	`EndOffset = end;`
	`294`	`+ StartAttributeLine = startAttributeLine;`
`283`	`295`	`}`
`284`	`296`
`285`	`297`	`/// <summary>`
`@@ -336,15 +348,15 @@ public static List<RuleSuppression> GetSuppressions(IEnumerable<AttributeAst> at`
`336`	`348`	`{`
`337`	`349`	`if (targetAsts.Count() == 0)`
`338`	`350`	`{`
`339`		`- ruleSupp.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSupp.StartOffset,`
	`351`	`+ ruleSupp.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSupp.StartAttributeLine,`
`340`	`352`	`System.IO.Path.GetFileName(scopeAst.Extent.File), String.Format(Strings.TargetCannotBeFoundError, ruleSupp.Target, ruleSupp.Scope));`
`341`	`353`	`result.Add(ruleSupp);`
`342`	`354`	`continue;`
`343`	`355`	`}`
`344`	`356`
`345`	`357`	`foreach (Ast targetAst in targetAsts)`
`346`	`358`	`{`
`347`		`- result.Add(new RuleSuppression(ruleSupp.RuleName, ruleSupp.RuleSuppressionID, targetAst.Extent.StartOffset, targetAst.Extent.EndOffset));`
	`359`	`+ result.Add(new RuleSuppression(ruleSupp.RuleName, ruleSupp.RuleSuppressionID, targetAst.Extent.StartOffset, targetAst.Extent.EndOffset, attributeAst.Extent.StartLineNumber));`
`348`	`360`	`}`
`349`	`361`	`}`
`350`	`362`
Original file line number	Diff line number	Diff line change
`@@ -773,7 +773,7 @@ public List<DiagnosticRecord> SuppressRule(string ruleName, Dictionary<string, L`
`773`	`773`	`// If we cannot found any error but the rulesuppression has a rulesuppressionid then it must be used wrongly`
`774`	`774`	`if (!String.IsNullOrWhiteSpace(ruleSuppression.RuleSuppressionID) && suppressionCount == 0)`
`775`	`775`	`{`
`776`		`- ruleSuppression.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSuppression.StartOffset,`
	`776`	`+ ruleSuppression.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSuppression.StartAttributeLine,`
`777`	`777`	`System.IO.Path.GetFileName(record.Extent.File), String.Format(Strings.RuleSuppressionIDError, ruleSuppression.RuleSuppressionID));`
`778`	`778`	`Helper.Instance.MyCmdlet.WriteError(new ErrorRecord(new ArgumentException(ruleSuppression.Error), ruleSuppression.Error, ErrorCategory.InvalidArgument, ruleSuppression));`
`779`	`779`	`}`
`@@ -811,7 +811,7 @@ public List<DiagnosticRecord> SuppressRule(string ruleName, Dictionary<string, L`
`811`	`811`	`// If we cannot found any error but the rulesuppression has a rulesuppressionid then it must be used wrongly`
`812`	`812`	`if (!String.IsNullOrWhiteSpace(ruleSuppression.RuleSuppressionID) && suppressionCount == 0)`
`813`	`813`	`{`
`814`		`- ruleSuppression.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSuppression.StartOffset,`
	`814`	`+ ruleSuppression.Error = String.Format(CultureInfo.CurrentCulture, Strings.RuleSuppressionErrorFormat, ruleSuppression.StartAttributeLine,`
`815`	`815`	`System.IO.Path.GetFileName(record.Extent.File), String.Format(Strings.RuleSuppressionIDError, ruleSuppression.RuleSuppressionID));`
`816`	`816`	`Helper.Instance.MyCmdlet.WriteError(new ErrorRecord(new ArgumentException(ruleSuppression.Error), ruleSuppression.Error, ErrorCategory.InvalidArgument, ruleSuppression));`
`817`	`817`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,17 +35,18 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)`
`35`	`35`	`foreach (Ast foundAst in foundAsts)`
`36`	`36`	`{`
`37`	`37`	`CommandAst cmdAst = (CommandAst)foundAst;`
	`38`	`+ string aliasName = cmdAst.GetCommandName();`
`38`	`39`	`// Handles the exception caused by commands like, {& $PLINK $args 2> $TempErrorFile}.`
`39`	`40`	`// You can also review the remark section in following document,`
`40`	`41`	`// MSDN: CommandAst.GetCommandName Method`
`41`		`- if (cmdAst.GetCommandName() == null) continue;`
	`42`	`+ if (aliasName == null) continue;`
`42`	`43`
`43`		`- string cmdletName = Microsoft.Windows.Powershell.ScriptAnalyzer.Helper.Instance.GetCmdletNameFromAlias(cmdAst.GetCommandName());`
	`44`	`+ string cmdletName = Microsoft.Windows.Powershell.ScriptAnalyzer.Helper.Instance.GetCmdletNameFromAlias(aliasName);`
`44`	`45`
`45`	`46`	`if (!String.IsNullOrEmpty(cmdletName))`
`46`	`47`	`{`
`47`		`- yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.AvoidUsingCmdletAliasesError, cmdAst.GetCommandName(), cmdletName),`
`48`		`- cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);`
	`48`	`+ yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.AvoidUsingCmdletAliasesError, aliasName, cmdletName),`
	`49`	`+ cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, aliasName);`
`49`	`50`	`}`
`50`	`51`	`}`
`51`	`52`	`}`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)`
`40`	`40`	`yield return`
`41`	`41`	`new DiagnosticRecord(`
`42`	`42`	`string.Format(CultureInfo.CurrentCulture, Strings.AvoidGlobalVarsError,`
`43`		`- varAst.VariablePath.UserPath), varAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);`
	`43`	`+ varAst.VariablePath.UserPath), varAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, varAst.VariablePath.UserPath);`
`44`	`44`	`}`
`45`	`45`	`}`
`46`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)`
`45`	`45`	`&& Helper.Instance.PositionalParameterUsed(cmdAst))`
`46`	`46`	`{`
`47`	`47`	`yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.AvoidUsingPositionalParametersError, cmdAst.GetCommandName()),`
`48`		`- cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);`
	`48`	`+ cmdAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, cmdAst.GetCommandName());`
`49`	`49`	`}`
`50`	`50`	`}`
`51`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,8 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName) {`
`63`	`63`	`StringComparison.OrdinalIgnoreCase))`
`64`	`64`	`{`
`65`	`65`	`string funcName = string.Format(CultureInfo.CurrentCulture,Strings.ReservedParamsCmdletPrefix, (parentAst as FunctionDefinitionAst).Name);`
`66`		`- yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.ReservedParamsError, funcName,paramName), paramAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName);`
	`66`	`+ yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.ReservedParamsError, funcName,paramName),`
	`67`	`+ paramAst.Extent, GetName(), DiagnosticSeverity.Warning, fileName, paramName);`
`67`	`68`
`68`	`69`	`}`
`69`	`70`	`}`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,8 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)`
`93`	`93`
`94`	`94`	`foreach (string key in assignments.Keys)`
`95`	`95`	`{`
`96`		`- yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.UseDeclaredVarsMoreThanAssignmentsError, key), assignments[key].Extent, GetName(), DiagnosticSeverity.Warning, fileName);`
	`96`	`+ yield return new DiagnosticRecord(string.Format(CultureInfo.CurrentCulture, Strings.UseDeclaredVarsMoreThanAssignmentsError, key),`
	`97`	`+ assignments[key].Extent, GetName(), DiagnosticSeverity.Warning, fileName, key);`
`97`	`98`	`}`
`98`	`99`	`}`
`99`	`100`