Skip to content

Revert "Fix unit tests native dependencies" #10919

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

Conversation

ThomasGoulet73
Copy link
Contributor

@ThomasGoulet73 ThomasGoulet73 commented Jun 6, 2025

Description

This reverts commit 2ff7cad (#10338). The linked issue (dotnet/runtime#111825) was closed when the fix (dotnet/runtime#114651) was merged in the runtime. The fix flowed in this repo and I'm no longer able to reproduce it locally.

Customer Impact

None, tests only.

Regression

No.

Testing

Ran the tests locally and tested the initial test I used when I pushed the workaround.

Risk

Low to none, tests only.

Microsoft Reviewers: Open in CodeFlow

@ThomasGoulet73 ThomasGoulet73 requested a review from a team as a code owner June 6, 2025 23:20
@dotnet-policy-service dotnet-policy-service bot added PR metadata: Label to tag PRs, to facilitate with triage Community Contribution A label for all community Contributions labels Jun 6, 2025
Copy link

codecov bot commented Jun 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 2.98754%. Comparing base (416cfa6) to head (adec62f).
Report is 2 commits behind head on main.

Additional details and impacted files
@@                Coverage Diff                @@
##               main     #10919         +/-   ##
=================================================
- Coverage   2.98812%   2.98754%   -0.00058%     
=================================================
  Files          3320       3319          -1     
  Lines        664900     664894          -6     
  Branches      74675      74674          -1     
=================================================
- Hits          19868      19864          -4     
+ Misses       644851     644849          -2     
  Partials        181        181                 
Flag Coverage Δ
Debug 2.98754% <ø> (-0.00058%) ⬇️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@harshit7962 harshit7962 merged commit 9beecf3 into dotnet:main Jun 9, 2025
8 checks passed
@harshit7962
Copy link
Member

Thank you for the follow-up @ThomasGoulet73

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community Contribution A label for all community Contributions PR metadata: Label to tag PRs, to facilitate with triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants