Skip to content

Commit 5d49ccb

Browse files
author
unknown
committed
Handle relative paths for New/Set PolicyDefinition
1 parent 2d14285 commit 5d49ccb

File tree

3 files changed

+9
-5
lines changed

3 files changed

+9
-5
lines changed

src/ResourceManager/Resources/Commands.ResourceManager/Cmdlets/Implementation/Policy/NewAzurePolicyDefinition.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,8 +126,10 @@ private JToken GetResource()
126126
/// </summary>
127127
private JToken GetPolicyRuleObject()
128128
{
129-
return File.Exists(this.Policy)
130-
? JToken.FromObject(FileUtilities.DataStore.ReadFileAsText(this.TryResolvePath(this.Policy)))
129+
string policyFilePath = this.TryResolvePath(this.Policy);
130+
131+
return File.Exists(policyFilePath)
132+
? JToken.FromObject(FileUtilities.DataStore.ReadFileAsText(policyFilePath))
131133
: JToken.FromObject(this.Policy);
132134
}
133135
}

src/ResourceManager/Resources/Commands.ResourceManager/Cmdlets/Implementation/Policy/SetAzurePolicyDefinition.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -171,8 +171,10 @@ protected string GetResourceId()
171171
/// </summary>
172172
protected JToken GetPolicyRuleObject()
173173
{
174-
return File.Exists(this.Policy)
175-
? JToken.FromObject(FileUtilities.DataStore.ReadFileAsText(this.TryResolvePath(this.Policy)))
174+
string policyFilePath = this.TryResolvePath(this.Policy);
175+
176+
return File.Exists(policyFilePath)
177+
? JToken.FromObject(FileUtilities.DataStore.ReadFileAsText(policyFilePath))
176178
: JToken.FromObject(this.Policy);
177179
}
178180
}

src/ResourceManager/Resources/Commands.Resources.Test/ScenarioTests/PolicyTests.ps1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ function Test-PolicyDefinitionCRUD
2828
Assert-AreEqual $expected.PolicyDefinitionId $actual.PolicyDefinitionId
2929
Assert-NotNull($actual.Properties.PolicyRule)
3030

31-
$actual = Set-AzureRMPolicyDefinition -Name $policyName -DisplayName testDisplay -Description testDescription
31+
$actual = Set-AzureRMPolicyDefinition -Name $policyName -DisplayName testDisplay -Description testDescription -Policy ".\SamplePolicyDefinition.json"
3232
$expected = Get-AzureRMPolicyDefinition -Name $policyName
3333
Assert-AreEqual $expected.Properties.DisplayName $actual.Properties.DisplayName
3434
Assert-AreEqual $expected.Properties.Description $actual.Properties.Description

0 commit comments

Comments
 (0)