Skip to content

Commit f24af8d

Browse files
authored
Fix recently introduced bug (PR #1391) for UseCorrectCasing (#1408)
1 parent add05d5 commit f24af8d

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

Rules/UseCorrectCasing.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,7 @@ public override IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string file
7373
foreach (var commandParameterAst in commandParameterAsts)
7474
{
7575
var parameterName = commandParameterAst.ParameterName;
76-
var parameterMetaData = availableParameters[parameterName];
77-
if (parameterMetaData != null)
76+
if (availableParameters.TryGetValue(parameterName, out ParameterMetadata parameterMetaData))
7877
{
7978
var correctlyCasedParameterName = parameterMetaData.Name;
8079
if (!parameterName.Equals(correctlyCasedParameterName, StringComparison.Ordinal))

Tests/Rules/UseCorrectCasing.tests.ps1

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,4 +65,9 @@ Describe "UseCorrectCasing" {
6565
Invoke-Formatter 'Invoke-DummyFunction -parametername: $parameterValue' |
6666
Should -Be 'Invoke-DummyFunction -ParameterName: $parameterValue'
6767
}
68+
69+
It "Should not throw when using parameter name that does not exist" {
70+
Invoke-Formatter 'Get-Process -NonExistingParameterName' -ErrorAction Stop
71+
}
72+
6873
}

0 commit comments

Comments
 (0)