New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added -Scope to the Resource to fix bugs with narrower scope always m… #9
Conversation
…aking Test-TargetResource returning false. Signed-off-by: Hans O. Martinsen
Signed-off-by: Hans O. Martinsen
Signed-off-by: Hans O. Martinsen
Reviewed 3 of 4 files at r1, 2 of 2 files at r2. Comments from Reviewable |
} | ||
|
||
It 'Returns correct execution policy for the correct Scope' { | ||
$result = Get-TargetResource -ExecutionPolicy $(Get-ExecutionPolicy) -Scope 'LocalMachine' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ExecutionPolicy [](start = 68, length = 15)
I think you need to specify the parameter -Scope 'LocalMachine' in this and the line below.
|
||
It 'Returns correct execution policy for the correct Scope' { | ||
$result = Get-TargetResource -ExecutionPolicy $(Get-ExecutionPolicy) -Scope 'LocalMachine' | ||
$result.ExecutionPolicy | should be $(Get-ExecutionPolicy) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get-ExecutionPolicy [](start = 54, length = 19)
Same comment as above.
🕐 |
@HansOMartinsen are you able to work on this PR if I review? If so could you start by rebasing (git rebase) this PR to resolve the conflict? If you can’t work on this then let us know and someone else could continue the work. Thanks! |
Closing this as this is being worked on in PR #15 |
…aking Test-TargetResource returning false.
Signed-off-by: Hans O. Martinsen
This change is