winforms icon indicating copy to clipboard operation
winforms copied to clipboard

Add code coverage for DataGridVieLinkCell

Open Zheng-Li01 opened this issue 7 months ago • 2 comments

Related https://github.com/dotnet/winforms/issues/13442

Proposed changes

Add unit test file: DataGridVieLinkCellTests.cs for DataGridVieLinkCell.cs file.

Zheng-Li01 avatar May 26 '25 08:05 Zheng-Li01

Codecov Report

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

Project coverage is 76.64262%. Comparing base (5821493) to head (8c40d58). Report is 117 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                  @@
##                main      #13524          +/-   ##
====================================================
+ Coverage   62.21083%   76.64262%   +14.43178%     
====================================================
  Files           3213        3254          +41     
  Lines         638302      641399        +3097     
  Branches       47201       47471         +270     
====================================================
+ Hits          397093      491585       +94492     
+ Misses        234153      146272       -87881     
+ Partials        7056        3542        -3514     
Flag Coverage Δ
Debug 76.64262% <99.00744%> (+14.43178%) :arrow_up:
integration 18.47423% <ø> (-0.31456%) :arrow_down:
production 51.11418% <ø> (+32.32540%) :arrow_up:
test 97.41357% <99.00744%> (+0.00946%) :arrow_up:
unit 48.52683% <ø> (?)

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

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar May 26 '25 08:05 codecov[bot]

@ricardobossan, @Zheng-Li01:

If you want me to take another look at particular issues, can you tag me at the respective spots? Otherwise, if you got all approvals, merge the PRs as quickly as possible, so we're getting a bit lower on the PR count! (And, yes I know, I also have quite some PRs to go through... 😄)

KlausLoeffelmann avatar Jun 13 '25 17:06 KlausLoeffelmann