[cheenamalhotra]

Description

The Windows and Linux Azure integration jobs in #4890 failed because project-reference builds now reach the SqlClient reference assembly, whose documentation trimming requires the local pwsh tool.

Invoke the existing shared tool-restore template after SDK installation and before building the Azure tests. Restore remains unconditional for both reference modes. Add a regression test that checks this ordering. No public API or runtime behavior changes.

Issues

Failures seen in #4890. Addresses the missing-tool failures in build 187521.

Testing

Added a Pester regression test for the Azure integration job bootstrap.

Passed: the following isolated run selected one test, which passed:

@'
Import-Module Pester -RequiredVersion 5.3.1
$config = New-PesterConfiguration
$config.Run.Path = 'eng\pipelines\scripts\tests\Build-Orchestration.Tests.ps1'
$config.Filter.FullName = 'Azure integration tool bootstrap*'
$config.Run.Exit = $true
$config.Output.Verbosity = 'Detailed'
Invoke-Pester -Configuration $config
'@ | dotnet tool run pwsh -- -NoProfile -NonInteractive -Command -

Passed: dotnet msbuild src\Microsoft.Data.SqlClient\ref\Microsoft.Data.SqlClient.csproj -nologo -t:_CheckPwshToolRestored -p:TargetFramework=net10.0 -v:minimal.

Passed: git diff --check.

The initial test invocation unexpectedly ran unrelated orchestration tests and was stopped after Windows argument-handling failures; the explicit Pester 5 invocation above correctly isolated the new test. Full Azure integration tests and an Azure Pipelines rerun were not performed locally.

Validations

• Tests added or updated
• Public API changes documented (N/A: no public API changes)
• Verified against customer repro (N/A: pipeline failure confirmed in build logs)
• Ensure no breaking changes introduced

[Copilot]

🔵 Needs a closer look

Required trusted-base review-policy references were not supplied with an immutable source, leaving policy verification incomplete.

No open issues 🎉

What changed in this PR

Restores local tools before Azure integration builds, addressing the missing pwsh dependency introduced when #4890 made project-reference builds include SqlClient’s reference assembly. No public API or runtime behavior changes.

Changes:

• Invoke the shared tool-restore template after SDK installation for both reference modes.
• Add a Pester regression test checking unconditional restoration and bootstrap ordering.

File	Description
eng/​pipelines/​scripts/​tests/​Build-Orchestration.Tests.ps1	Tests that tool restoration occurs after SDK installation and before building.
eng/​pipelines/​jobs/​test-azure-package-ci-job.yml	Adds shared local-tool restoration before Azure integration builds.

_{Review effort: Balanced}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.64%. Comparing base (cb66150) to head (7efc6be).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4952      +/-   ##
==========================================
- Coverage   67.91%   66.64%   -1.28%     
==========================================
  Files         311      305       -6     
  Lines       45286    68341   +23055     
==========================================
+ Hits        30757    45543   +14786     
- Misses      14529    22798    +8269     

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

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.