Add integration test for Disconnect-SqlDscDatabaseEngine command

[Copilot]

• Explored repository structure and understand the codebase
• Verified build environment and ran existing unit tests for Disconnect-SqlDscDatabaseEngine
• Analyzed existing integration test patterns from Connect-SqlDscDatabaseEngine and Remove-SqlDscAgentAlert
• Create integration test file for Disconnect-SqlDscDatabaseEngine
• Implement test scenarios covering basic disconnect functionality
• Test both default and named instance scenarios
• Verify tests work with proper connection/disconnection patterns
• Update azure-pipelines.yml to include the new integration test
• Validate test file syntax and structure
• Confirm unit tests still pass
• Verify test discovery works correctly
• Address all feedback comments from review
• Fix build.ps1 file mode to match main branch

Summary

Successfully implemented integration test coverage for the Disconnect-SqlDscDatabaseEngine command and addressed all review feedback:

Changes Made

1. Integration Test: tests/Integration/Commands/Disconnect-SqlDscDatabaseEngine.Integration.Tests.ps1

   • 6 comprehensive test scenarios
   • Tests both default and named instances
   • Covers Windows and SQL authentication
   • Validates connection state after disconnect

2. CI Pipeline: Updated azure-pipelines.yml to include test in Group 1

3. Documentation: Added test entry to tests/Integration/Commands/README.md

4. Changelog: Updated CHANGELOG.md with new integration test entry

Review Feedback Addressed

• ✅ Reverted unintended permission changes to build.ps1
• ✅ Updated error message to use 'noop' task instead of 'build'
• ✅ Added README.md entry with correct dependencies (Group 1, depends on Prerequisites)
• ✅ Ensured file ends with single blank line
• ✅ Updated changelog with proper entry format
• ✅ Fixed build.ps1 file mode to match main branch (644, non-executable)

The integration test provides comprehensive coverage for disconnect functionality across different SQL Server versions and authentication methods.

Fixes #2224.

---

💬 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 (36ba010).
⚠️ Report is 18 commits behind head on main.

Additional details and impacted files

@@         Coverage Diff          @@
##           main   #2255   +/-   ##
====================================
  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.

[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.}