[6.1] Add new trusted AKV URLs for FR and DE

[paulmedynski]

Port of #3482 from main to release/6.1

• Added 4 new trusted AKV URLs.
• Fixed existing manual tests and added unit tests.
• Removed unnecessary strings and console output from TrustedUrlsTest.

[Copilot]

Pull Request Overview

Port the addition of new Azure Key Vault trusted endpoints and update related tests for the 6.1 release branch.

• Add four new trusted AKV URLs (France & Germany) to the domain list.
• Update exception helper to return ArgumentException and centralize error message generation.
• Register the new TrustedUrlsTest in the manual tests project and refactor existing tests to use the helper.

Reviewed Changes

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

Show a summary per file

File	Description
src/Microsoft.Data.SqlClient/tests/ManualTests/Microsoft.Data.SqlClient.ManualTesting.Tests.csproj	Re-add compile entries for new/updated AlwaysEncrypted tests.
src/Microsoft.Data.SqlClient/tests/ManualTests/AlwaysEncrypted/TrustedUrlsTest.cs	Introduce a test for valid/invalid vault URLs and helper.
src/Microsoft.Data.SqlClient/tests/ManualTests/AlwaysEncrypted/ExceptionTestAKVStore.cs	Replace hard-coded expected message with the new helper call.
src/Microsoft.Data.SqlClient/add-ons/AzureKeyVaultProvider/Utils.cs	Change exception factory to return ArgumentException.
src/Microsoft.Data.SqlClient/add-ons/AzureKeyVaultProvider/Constants.cs	Expand AKV domain list with new regions using C# 12 syntax.

Comments suppressed due to low confidence (1)

src/Microsoft.Data.SqlClient/add-ons/AzureKeyVaultProvider/Utils.cs:144

• [nitpick] The method name InvalidAKVUrlTrustedEndpoints doesn’t match the resource key InvalidAkvKeyPathTrustedTemplate. Consider renaming it to InvalidAKVKeyPathTrustedEndpoints for consistency and clearer intent.

        internal static ArgumentException InvalidAKVUrlTrustedEndpoints(string masterKeyPath, string endpoints) =>

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.91%. Comparing base (5daddba) to head (097041c).
Report is 2 commits behind head on release/6.1.

Additional details and impacted files

@@               Coverage Diff               @@
##           release/6.1    #3486      +/-   ##
===============================================
+ Coverage        66.09%   66.91%   +0.82%     
===============================================
  Files              281      281              
  Lines            62408    62416       +8     
===============================================
+ Hits             41247    41768     +521     
+ Misses           21161    20648     -513     

Flag	Coverage Δ
addons	91.04% <100.00%> (+0.22%)	⬆️
netcore	68.98% <ø> (+0.02%)	⬆️
netfx	69.25% <ø> (+1.09%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 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.