Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.1.5] Fix | Minor fixes to support different test environments (#2045) #2325

Merged
merged 1 commit into from Jan 27, 2024

Conversation

JRahnama
Copy link
Member

backporting #2045 to 5.1.5

  • Test Fixes:
  • Set correct condition on AE tests that require an available key store (Cert Store on Windows, else AKV)
  • ServerSPN test should get FQDN of target, not local machine
  • Instance name test shouldn't try to connect via IP if using integrated security in the context of a remote domain-based connection (Kerberos fails)

* Test Fixes:
- Set correct condition on AE tests that require an available key store (Cert Store on Windows, else AKV)
- ServerSPN test should get FQDN of target, not local machine
- Instance name test shouldn't try to connect via IP if using integrated security in the context of a remote domain-based connection (Kerberos fails)
@JRahnama JRahnama added ✔️ Tests Identifier for test related changes netcore labels Jan 26, 2024
@JRahnama JRahnama added this to In Progress in SqlClient v5.1.5 hotfix Jan 26, 2024
Copy link

codecov bot commented Jan 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (release/5.1@b92637e). Click here to learn what that means.

Additional details and impacted files
@@              Coverage Diff               @@
##             release/5.1    #2325   +/-   ##
==============================================
  Coverage               ?   71.05%           
==============================================
  Files                  ?      293           
  Lines                  ?    61562           
  Branches               ?        0           
==============================================
  Hits                   ?    43741           
  Misses                 ?    17821           
  Partials               ?        0           
Flag Coverage Δ
addons 92.38% <0.00%> (?)
netcore 75.82% <0.00%> (?)
netfx 68.36% <0.00%> (?)

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.

@JRahnama JRahnama merged commit 759dc69 into dotnet:release/5.1 Jan 27, 2024
128 of 130 checks passed
SqlClient v5.1.5 hotfix automation moved this from In Progress to Done Jan 27, 2024
@JRahnama JRahnama deleted the backport-test-fixes-to-5.1.5 branch March 8, 2024 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✔️ Tests Identifier for test related changes netcore
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants