Add integration test for Set-SqlDscServerPermission command

[Copilot]

• Analyze existing codebase and integration test patterns
• Examine Set-SqlDscServerPermission command structure
• Study Grant-SqlDscServerPermission, Deny-SqlDscServerPermission, and Revoke-SqlDscServerPermission integration tests as templates
• Initialize build environment
• Create integration test file following established patterns
• Test Grant, Deny, and Revoke states using Set-SqlDscServerPermission
• Test pipeline input scenarios
• Test WithGrant parameter scenarios
• Add proper cleanup and setup for each test
• Add integration test to azure-pipelines.yml configuration
• Update CHANGELOG.md with entry for new integration test
• Verify integration test syntax and structure
• Validate build process completes successfully
• Address PR feedback: remove error message test, fix task name, ensure proper file ending, update README.md
• Fix test cleanup to restore expected shared login permissions state

Fix for Test Failure:

Added proper cleanup logic in AfterAll block to restore the expected permissions state for the shared IntegrationTestSqlLogin. The Test-SqlDscServerPermission ExactMatch test expects the login to have ConnectSql, ViewServerState, and ViewAnyDefinition permissions granted, but the Set-SqlDscServerPermission test was modifying this state without proper restoration.

The fix ensures that after Set-SqlDscServerPermission tests complete, the shared login is restored to the state that subsequent tests depend on:

• Revokes any permissions that may have been set during testing
• Restores ViewServerState and ViewAnyDefinition permissions to granted state
• Maintains compatibility with other integration tests that depend on shared login state

Fixes #2234.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

---

This change is 

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94%. Comparing base (52e707f) to head (babb538).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files

@@         Coverage Diff          @@
##           main   #2265   +/-   ##
====================================
  Coverage    94%     94%           
====================================
  Files       149     149           
  Lines      9163    9163           
====================================
  Hits       8687    8687           
  Misses      476     476           

Flag	Coverage Δ
unit	94% <ø> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[FrokenSonja]

@coderabbitai create an issue to track that we need to change the task from build to noop

[FrokenSonja]

@coderabbitai create an issue to track that we need to change the task from build to noop

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}