PR Pipeline - Azure + AE

[benrr101]

Description

This PR adds to the PR pipeline in the previous PR #4308 and adds the following test jobs:

• SqlClient Manual test sets 123 against Azure
• SqlClient Manual test sets AE against localhost (in release mode)

Testing

• ✅ pr-pipeline on internal ADO
  https://dev.azure.com/SqlClientDrivers/ADO.Net/_build/results?buildId=155927&view=results
• ✅ sqlclient-pr on public ADO
  https://sqlclientdrivers.visualstudio.com/public/_build/results?buildId=155941&view=results
• ✅ sqlient-non-official
  https://dev.azure.com/SqlClientDrivers/ADO.Net/_build/results?buildId=155730&view=results

[Copilot]

Pull request overview

This PR extends the new PR validation pipeline to run additional SqlClient manual test coverage in CI-like PR validation scenarios, specifically adding Azure SQL manual test runs and an Always Encrypted (AE) manual test run in Release configuration.

Changes:

• Adds SqlClient manual test set 123 execution against Azure SQL, alongside existing localhost manual coverage.
• Adds a localhost Always Encrypted manual test run (TestSet=AE) forced to Release configuration.
• Simplifies PR pipeline templates by removing PR-specific wrapper step templates and switching PR jobs/stages to directly use the shared eng/pipelines/common/steps/* templates.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/Microsoft.Data.SqlClient/tests/ManualTests/BulkCopy/CopyAllFromReaderConnectionCloseOnEventAsync.cs	Skips this BulkCopy manual test on Azure servers via an additional IsNotAzureServer condition.
eng/pipelines/pr/stages/test-stages.yml	Adds Azure manual test execution for set 123, and adds localhost AE manual execution (AE) in Release mode.
eng/pipelines/pr/jobs/test-sqlclientmanual-job.yml	Parameterizes manual test jobs by config/test set; adds conditional local server setup and passes TestSet to build.proj.
eng/pipelines/pr/jobs/test-buildproj-job.yml	Updates job bootstrap to use shared install/restore steps and inlines the build.proj test invocation.
eng/pipelines/pr/stages/pack-stage.yml	Replaces PR-specific dotnet install wrapper with shared common install + tool restore steps.
eng/pipelines/pr/steps/pack-buildproj-step.yml	Renames the task display name to clarify it performs packing.
eng/pipelines/pr/steps/test-buildproj-step.yml	Deleted (previously a PR-specific wrapper step for running build.proj test targets).
eng/pipelines/pr/steps/install-dotnet.yml	Deleted (previously a PR-specific wrapper around the common install-dotnet and tool restore steps).
eng/pipelines/pr/steps/build-buildproj-step.yml	Deleted (previously a PR-specific wrapper step for running build.proj build targets).

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.62%. Comparing base (5ac26c9) to head (631d50c).
⚠️ Report is 9 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4338      +/-   ##
==========================================
- Coverage   66.50%   64.62%   -1.89%     
==========================================
  Files         285      280       -5     
  Lines       43311    66162   +22851     
==========================================
+ Hits        28806    42759   +13953     
- Misses      14505    23403    +8898     

Flag	Coverage Δ
CI-SqlClient	?
PR-SqlClient-Project	64.62% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

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

[benrr101]

@paulmedynski I'm going to assume it has to do wit the branch not being up to date with main and didn't have the config.jsonc changes. Merge has been pushed, so fingers crossed.

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated no new comments.