-
Notifications
You must be signed in to change notification settings - Fork 458
[Combobox] Fix presetting option #4255
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
|
✅ All tests passed successfully Details on your Workflow / Core Tests page. |
Summary - Unit Tests Code CoverageSummary
CoverageMicrosoft.FluentUI.AspNetCore.Components - 61.2%
|
Remove some test code
|
This We should also add a unit test in the current version 4 to verify this right away. |
This change only touches the |
That's not the only change. You also have a test on How can we be sure there won't be any regressions? |
|
Value was not being considered in this method before this change. I don't see how that could lead to a regression. |
|
@microsoft-github-policy-service rerun |
|
@microsoft-github-policy-service agree |
Fix #4250 by extending the
GetOptionSelectedcode