Get-DbaPermission does not process DENY

[pollusb]

Type of Change

• [x] Bug fix (non-breaking change, fixes # )
• [ ] New feature (non-breaking change, adds functionality, fixes # )
• [ ] Breaking change (affects multiple commands or functionality, fixes # )
• [ ] Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
• [ ] Adding code coverage to existing functionality
• [ ] Pester test is included
• [ ] If new file reference added for test, has is been added to github.com/dataplat/appveyor-lab ?
• [ ] Unit test is included
• [ ] Documentation
• [ ] Build system

Purpose

Resolve wrong GrantStatement

Commands to test

PS> $perm = Get-DbaPermission @splat -ExcludeSystemObjects 
PS> $perm|? PermState -eq DENY|select -f 1 PermState,PermissionName,GrantStatement

PermState      : DENY
PermissionName : DELETE
GrantStatement : GRANT DELETE ON [dbo].[table1] TO [user1]

PS> $perm = Get-DbaPermission @splat -ExcludeSystemObjects 
PS> $perm|? PermState -eq DENY|select -f 1 PermState,PermissionName,GrantStatement

PermState      : DENY
PermissionName : DELETE
GrantStatement : DENY DELETE ON [dbo].[table1] TO [user1]

[pollusb]

What can I do to help it pass the last check?

Thanks

[niphlod]

hi @pollusb , failure was seemingly unrelated. I restarted the CI process. PS: If I'm not around just pushing a new commit will trigger the CI automatically.

[pollusb]

Thanks for the tip

Le ven. 5 juill. 2024 03:00, Simone Bizzotto @.***> a écrit :

hi @pollusb https://github.com/pollusb , failure was seemingly unrelated. I restarted the CI process. PS: If I'm not around just pushing a new commit will trigger the CI automatically.

— Reply to this email directly, view it on GitHub https://github.com/dataplat/dbatools/pull/9416#issuecomment-2210309907, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFFZ4P7BCBCPM3DSXL4XOTZKZACRAVCNFSM6AAAAABKLX4CAWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJQGMYDSOJQG4 . You are receiving this because you were mentioned.Message ID: @.***>