Skip to content
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

Add code coverage for RichTextBox #11034

Closed
wants to merge 2 commits into from
Closed

Add code coverage for RichTextBox #11034

wants to merge 2 commits into from

Conversation

Zheng-Li01
Copy link
Member

@Zheng-Li01 Zheng-Li01 commented Mar 12, 2024

Fixes #10453

Proposed changes

  • Add unit tests for RichTextBox to test its DrawToBitmap()/LoadFile()/SaveFile()/Paste() methods
  • Add unit tests for RichTextBoxto test its events: DragDrop, DragEnter, DragLeave, DragOver, GiveFeedback,QueryContinueDrag

Customer Impact

  • None

Regression?

  • No

Risk

  • Minimal

Test methodology

  • Unit tests
Microsoft Reviewers: Open in CodeFlow

@Zheng-Li01 Zheng-Li01 requested a review from a team as a code owner March 12, 2024 09:22
@Zheng-Li01 Zheng-Li01 marked this pull request as draft March 12, 2024 09:39
Copy link

codecov bot commented Mar 12, 2024

Codecov Report

Attention: Patch coverage is 97.97980% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 73.23675%. Comparing base (5e635c5) to head (fbc7a22).
Report is 33 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #11034         +/-   ##
===================================================
+ Coverage   73.19881%   73.23675%   +0.03794%     
===================================================
  Files           3097        3096          -1     
  Lines         634289      633817        -472     
  Branches       47364       47338         -26     
===================================================
- Hits          464292      464187        -105     
+ Misses        166453      166090        -363     
+ Partials        3544        3540          -4     
Flag Coverage Δ
Debug 73.23675% <97.97980%> (+0.03794%) ⬆️
integration 18.42765% <ø> (-0.01961%) ⬇️
production 46.72236% <ø> (+0.01333%) ⬆️
test 94.98148% <97.97980%> (+0.01015%) ⬆️
unit 43.62874% <ø> (+0.02197%) ⬆️

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

@Zheng-Li01 Zheng-Li01 closed this Mar 19, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Apr 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
draft draft PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure that all WinForms controls have unit tests coverage for public APIs
2 participants