Skip to content

Fix for intermittent test failure on x86 platform 'BindingContext_UpdateBinding_UpdateListBindingWithoutDataMember_Success' - #14359

Merged
Shyam-Gupta merged 1 commit into
mainfrom
dev/shgu/bindingtest
Mar 6, 2026
Merged

Shyam-Gupta merged 1 commit into
mainfrom
dev/shgu/bindingtest

Conversation

@Shyam-Gupta

@Shyam-Gupta Shyam-Gupta commented Mar 5, 2026 •

Copy link
Copy Markdown
Member

Issue: Intermittent BindingContext_UpdateBinding_UpdateListBindingWithoutDataMember_Success test failure on x86 platform

CI test failure against 11.0-preview2: https://dev.azure.com/dnceng-public/public/_build/results?buildId=1321935&view=ms.vss-test-web.build-test-results-tab&runId=36910776&resultId=116411&paneView=debug

Root cause: context1 was getting garbage collected after UpdateBinding call to context2 causing the assertions to fail.

Fix: Obtain a strong reference to manager1 before calling UpdateBinding.

Microsoft Reviewers: Open in CodeFlow

Issue: context1 was getting garbage collected after UpdateBinding call to context2 causing the assertions to fail.

Fix: Obtain a strong reference to manager1 before calling UpdateBinding.
@Shyam-Gupta Shyam-Gupta self-assigned this Mar 5, 2026
@Shyam-Gupta
Shyam-Gupta requested a review from a team as a code owner March 5, 2026 21:18
@Shyam-Gupta Shyam-Gupta added test-bug Problem in test source code (most likely) and removed area-Infrastructure labels Mar 5, 2026
@Shyam-Gupta Shyam-Gupta changed the title Fix for intermittent test failure on x86 platform Fix for intermittent test failure on x86 platform 'BindingContext_UpdateBinding_UpdateListBindingWithoutDataMember_Success' Mar 5, 2026
@codecov

codecov Bot commented Mar 5, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.31528%. Comparing base (8e560d0) to head (fc966b4).
⚠️ Report is 9 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #14359         +/-   ##
===================================================
+ Coverage   77.31249%   77.31528%   +0.00279%     
===================================================
  Files           3265        3265                 
  Lines         644447      644447                 
  Branches       47631       47631                 
===================================================
+ Hits          498238      498256         +18     
+ Misses        142520      142501         -19     
- Partials        3689        3690          +1     
Flag Coverage Δ
Debug 77.31528% <100.00000%> (+0.00279%) ⬆️
integration 19.12461% <ø> (+0.01606%) ⬆️
production 52.28362% <ø> (+0.00209%) ⬆️
test 97.41916% <100.00000%> (+0.00336%) ⬆️
unit 49.64256% <ø> (-0.02544%) ⬇️

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.

@Shyam-Gupta
Shyam-Gupta merged commit 82ed134 into main Mar 6, 2026
10 checks passed
@Shyam-Gupta
Shyam-Gupta deleted the dev/shgu/bindingtest branch March 6, 2026 20:24
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure test-bug Problem in test source code (most likely)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants