`*-SqlDscServerPermission`: Add commands SQL Server permission management#2157

Merged

johlju merged 30 commits intomainfrom

copilot/fix-e806ea1d-2077-4c68-a272-f8dea52701f7

Sep 1, 2025

Commits on Aug 30, 2025

Initial plan
Copilot
committed
Add core server permission commands and private functions

Copilot
and
johlju
committed
Add integration tests and update CHANGELOG for new server permission commands

Copilot
and
johlju
committed
Update server permission commands to use State parameter and ServerPermissionSet type

Copilot
and
johlju
committed
Merge branch 'main' into copilot/fix-e806ea1d-2077-4c68-a272-f8dea52701f7
johlju
authored
Restructure server permission commands based on feedback - rename commands, add Deny command, and use parameter sets for Test command

Copilot
and
johlju
committed
Change Permission parameter to accept string array for easier usage

Copilot
and
johlju
committed

Commits on Aug 31, 2025

Implement all requested improvements to server permission commands

Copilot
and
johlju
committed
Address feedback: update server permission commands to use Login/ServerRole objects, fix localized strings, and ensure proper file endings

Copilot
and
johlju
committed
Clean up code
johlju
authored
Fix blank line EOF
johlju
authored
Change error handling to use PSCmdlet.ThrowTerminatingError in server permission commands

Copilot
and
johlju
committed
Merge branch 'main' into copilot/fix-e806ea1d-2077-4c68-a272-f8dea52701f7
johlju
authored
Remove unused $principalObject variables from server permission commands

Copilot
and
johlju
committed
Refactor SQL DSC Server Permission Tests
johlju
committed
Update integration tests
johlju
committed
Merge branch 'main' into copilot/fix-e806ea1d-2077-4c68-a272-f8dea52701f7
johlju
authored
Remove integration tests for Get-SqlDscServerPermission
johlju
committed
Refactor permission handling in integration and unit tests to eliminate unnecessary error handling syntax
johlju
committed
Update CHANGELOG.md

johlju
and
coderabbitai[bot]
authored
Update tests/Integration/Commands/Deny-SqlDscServerPermission.Integration.Tests.ps1

johlju
and
coderabbitai[bot]
authored
Update tests/Integration/Commands/Deny-SqlDscServerPermission.Integration.Tests.ps1

johlju
and
coderabbitai[bot]
authored
Remove unit test stub imports from server permission integration tests

Copilot
and
johlju
committed
Reorganize server permissions in SqlServerPermission enum for clarity and consistency
johlju
committed
Reorganize integration test commands by grouping and adding missing remove commands
johlju
committed
Refactor AfterAll block in Deny-SqlDscServerPermission integration tests to remove unnecessary parameter cleanup
johlju
committed
Update Pester guidelines to consolidate requirements and best practices for test structure
johlju
committed
Refactor integration tests to remove default parameter values and add ErrorAction for improved error handling
johlju
committed
Reorganize integration test commands to improve clarity and add missing remove commands
johlju
committed
Remove redundant module removal command from AfterAll blocks in integration tests
johlju
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

`*-SqlDscServerPermission`: Add commands SQL Server permission management#2157

`*-SqlDscServerPermission`: Add commands SQL Server permission management#2157
johlju merged 30 commits intomainfrom
copilot/fix-e806ea1d-2077-4c68-a272-f8dea52701f7

Commits on Aug 30, 2025

Initial plan

Add core server permission commands and private functions

Add integration tests and update CHANGELOG for new server permission commands

Update server permission commands to use State parameter and ServerPermissionSet type

Merge branch 'main' into copilot/fix-e806ea1d-2077-4c68-a272-f8dea52701f7

Restructure server permission commands based on feedback - rename commands, add Deny command, and use parameter sets for Test command

Change Permission parameter to accept string array for easier usage

Commits on Aug 31, 2025

Implement all requested improvements to server permission commands

Address feedback: update server permission commands to use Login/ServerRole objects, fix localized strings, and ensure proper file endings

Clean up code

Fix blank line EOF

Change error handling to use PSCmdlet.ThrowTerminatingError in server permission commands

Merge branch 'main' into copilot/fix-e806ea1d-2077-4c68-a272-f8dea52701f7

Remove unused $principalObject variables from server permission commands

Refactor SQL DSC Server Permission Tests

Update integration tests

Merge branch 'main' into copilot/fix-e806ea1d-2077-4c68-a272-f8dea52701f7

Remove integration tests for Get-SqlDscServerPermission

Refactor permission handling in integration and unit tests to eliminate unnecessary error handling syntax

Update CHANGELOG.md

Update tests/Integration/Commands/Deny-SqlDscServerPermission.Integration.Tests.ps1

Update tests/Integration/Commands/Deny-SqlDscServerPermission.Integration.Tests.ps1

Remove unit test stub imports from server permission integration tests

Reorganize server permissions in SqlServerPermission enum for clarity and consistency

Reorganize integration test commands by grouping and adding missing remove commands

Refactor AfterAll block in Deny-SqlDscServerPermission integration tests to remove unnecessary parameter cleanup

Update Pester guidelines to consolidate requirements and best practices for test structure

Refactor integration tests to remove default parameter values and add ErrorAction for improved error handling

Reorganize integration test commands to improve clarity and add missing remove commands

Remove redundant module removal command from AfterAll blocks in integration tests