[Instrumentation.EFCore] Support use with SqlClient Instrumentation

[martincostello]

Fixes #1764

Changes

Takes over from #2280 which went stale. Just need to add tests.

Merge requirement checklist

• CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
• Unit tests added/updated
• Appropriate CHANGELOG.md files updated for non-trivial changes
• Changes in public API reviewed (if applicable)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.44%. Comparing base (71655ce) to head (1996b69).
Report is 880 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2829      +/-   ##
==========================================
+ Coverage   73.91%   81.44%   +7.52%     
==========================================
  Files         267       14     -253     
  Lines        9615      652    -8963     
==========================================
- Hits         7107      531    -6576     
+ Misses       2508      121    -2387     

Flag	Coverage Δ
unittests-Instrumentation.EntityFrameworkCore	66.14% <100.00%> (?)
unittests-Instrumentation.SqlClient	87.82% <ø> (?)

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

Files with missing lines	Coverage Δ
...mplementation/EntityFrameworkDiagnosticListener.cs	63.15% <100.00%> (+10.77%)	⬆️

... and 269 files with indirect coverage changes

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[martincostello]

From manual testing:

OpenTelemetry.Instrumentation.EntityFrameworkCore 1.12.0-beta.1

This PR

[martincostello]

Tests added based on the existing SqlClient integration tests. Reverting the original change in the first commit will cause the tests to fail.

[Copilot]

Pull Request Overview

This PR adds support for using SqlClient instrumentation with EntityFrameworkCore by updating package references, adjusting activity source usage, and extending integration test coverage.

• Updated package references in test projects for Microsoft.Data.SqlClient, EFCore providers, and test containers.
• Added new integration tests and refactored activity source usage in the diagnostic listener.
• Updated changelog to document support for SqlClient instrumentation.

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
test/OpenTelemetry.Instrumentation.SqlClient.Tests/OpenTelemetry.Instrumentation.SqlClient.Tests.csproj	Updated SqlClient package version
test/OpenTelemetry.Instrumentation.EntityFrameworkCore.Tests/OpenTelemetry.Instrumentation.EntityFrameworkCore.Tests.csproj	Updated EFCore provider versions and added project references
test/OpenTelemetry.Instrumentation.EntityFrameworkCore.Tests/ItemsContext.cs	Added new ItemsContext for EFCore tests using C# primary constructor syntax
test/OpenTelemetry.Instrumentation.EntityFrameworkCore.Tests/Item.cs	Added simple POCO for EFCore tests
test/OpenTelemetry.Instrumentation.EntityFrameworkCore.Tests/EntityFrameworkIntegrationTests.cs	Introduced integration tests for raw SQL and DataContext scenarios
test/OpenTelemetry.Instrumentation.EntityFrameworkCore.Tests/EntityFrameworkDiagnosticListenerTests.cs	Refactored test item instantiation to use object initializer syntax
src/OpenTelemetry.Instrumentation.EntityFrameworkCore/Implementation/EntityFrameworkDiagnosticListener.cs	Updated activity source usage to reflect SqlClient vs. EFCore differences
src/OpenTelemetry.Instrumentation.EntityFrameworkCore/CHANGELOG.md	Documented new SqlClient instrumentation support

Comments suppressed due to low confidence (1)

test/OpenTelemetry.Instrumentation.EntityFrameworkCore.Tests/EntityFrameworkIntegrationTests.cs:290

• The error message uses an extra '$' inside the string interpolation which results in an unintended literal. Consider removing the extraneous '$' to correctly interpolate the container type name.

throw new InvalidOperationException($"Container type '${this.fixture.DatabaseContainer.GetType().Name}' is not supported.");

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 7 days.