Skip to content

Conversation

@gpetrou
Copy link
Contributor

@gpetrou gpetrou commented Jan 31, 2024

Proposed changes

  • Add ComboBoxDesigner tests.
Microsoft Reviewers: Open in CodeFlow

@gpetrou gpetrou requested a review from a team as a code owner January 31, 2024 05:38
@ghost ghost assigned gpetrou Jan 31, 2024
@gpetrou gpetrou force-pushed the ComboBoxDesignerTests branch from b897bf7 to fad1d69 Compare January 31, 2024 05:38
@codecov
Copy link

codecov bot commented Jan 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (82a15d4) 72.65624% compared to head (fad1d69) 72.67741%.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #10772         +/-   ##
===================================================
+ Coverage   72.65624%   72.67741%   +0.02116%     
===================================================
  Files           2925        2926          +1     
  Lines         622899      622924         +25     
  Branches       46925       46925                 
===================================================
+ Hits          452575      452725        +150     
+ Misses        161985      161850        -135     
- Partials        8339        8349         +10     
Flag Coverage Δ
Debug 72.67741% <100.00000%> (+0.02116%) ⬆️
production 43.53199% <ø> (+0.04377%) ⬆️
test 97.33495% <100.00000%> (+0.00020%) ⬆️

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


Action action = () => _ = comboBoxDesigner.SelectionRules;

action.Should().ThrowExactly<NullReferenceException>();
Copy link
Member

@lonitra lonitra Jan 31, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is something that we'll want to change to throw a different exception or not at all to avoid NRE as we enable nullability in the designer space.

@lonitra lonitra enabled auto-merge (squash) January 31, 2024 17:29
@lonitra lonitra merged commit d152b3c into dotnet:main Jan 31, 2024
@ghost ghost added this to the 9.0 Preview2 milestone Jan 31, 2024
@gpetrou gpetrou deleted the ComboBoxDesignerTests branch January 31, 2024 19:32
@github-actions github-actions bot locked and limited conversation to collaborators Mar 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants