Skip to content

Conversation

@muskan124947
Copy link
Contributor

@muskan124947 muskan124947 commented Nov 5, 2025

Temporarily commented out the testFourPartSyntaxCallEscapeSyntax() test case due to failures encountered with linked server setup on SQL Server 2025. The issue is related to stricter encryption and trust requirements introduced in the new SQL Server version. The test will be re-enabled once compatibility adjustments for linked server configuration are implemented.

@codecov
Copy link

codecov bot commented Nov 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.37%. Comparing base (ed649c4) to head (5626769).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2832      +/-   ##
============================================
+ Coverage     56.35%   56.37%   +0.02%     
- Complexity     4531     4534       +3     
============================================
  Files           149      149              
  Lines         34376    34376              
  Branches       5734     5734              
============================================
+ Hits          19373    19381       +8     
+ Misses        12393    12392       -1     
+ Partials       2610     2603       -7     

☔ 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.

@muskan124947 muskan124947 changed the title [FIX]: drop linked server logins before dropping linked server Comment out test testFourPartSyntaxCallEscapeSyntax() due to linked server issue with sql server 2025 Nov 6, 2025
@muskan124947 muskan124947 self-assigned this Nov 6, 2025
@muskan124947 muskan124947 added this to the 13.3.0 milestone Nov 6, 2025
@muskan124947 muskan124947 merged commit 0e695fb into main Nov 6, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants