Cleanup | DbConnectionInternal

[benrr101]

Description: This is the second half of #2910 that applies cleanup to DbConnectionInternal. The PR is meant to be highly granular with each commit being individually reviewable. Ideally, I'd recommend going through the PR commit-by-commit and review each cleanup change.

Testing: Test cases are still passing, so I suspect this is a safe change.

[cheenamalhotra]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[codecov]

Codecov Report

Attention: Patch coverage is 80.79470% with 29 lines in your changes missing coverage. Please review.

Project coverage is 72.24%. Comparing base (39c4604) to head (e92fbf9).
Report is 21 commits behind head on main.

Files with missing lines	Patch %	Lines
...icrosoft/Data/ProviderBase/DbConnectionInternal.cs	81.08%	28 Missing ⚠️
...icrosoft/Data/SqlClient/SqlDelegatedTransaction.cs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2934      +/-   ##
==========================================
+ Coverage   71.92%   72.24%   +0.32%     
==========================================
  Files         294      291       -3     
  Lines       60342    59766     -576     
==========================================
- Hits        43398    43178     -220     
+ Misses      16944    16588     -356     

Flag	Coverage Δ
addons	92.90% <ø> (ø)
netcore	75.82% <79.72%> (-0.02%)	⬇️
netfx	70.61% <80.82%> (+0.15%)	⬆️

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.