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 ControlCodeDomSerializer #13011

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Zheng-Li01
Copy link
Member

@Zheng-Li01 Zheng-Li01 commented Feb 24, 2025

Related #10773

Proposed changes

Add unit test file: ControlCodeDomSerializerTests.cs for public methods of the ControlCodeDomSerializer.cs.

Microsoft Reviewers: Open in CodeFlow

Copy link

codecov bot commented Feb 24, 2025

Codecov Report

Attention: Patch coverage is 72.22222% with 10 lines in your changes missing coverage. Please review.

Project coverage is 76.05659%. Comparing base (227fd3b) to head (10afd24).
Report is 1 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #13011         +/-   ##
===================================================
+ Coverage   76.04966%   76.05659%   +0.00692%     
===================================================
  Files           3269        3270          +1     
  Lines         643544      643580         +36     
  Branches       47429       47429                 
===================================================
+ Hits          489413      489485         +72     
+ Misses        150566      150536         -30     
+ Partials        3565        3559          -6     
Flag Coverage Δ
Debug 76.05659% <72.22222%> (+0.00692%) ⬆️
integration 18.03440% <ø> (-0.00280%) ⬇️
production 50.04676% <ø> (+0.01534%) ⬆️
test 96.93533% <72.22222%> (-0.00194%) ⬇️
unit 47.47920% <ø> (-0.00803%) ⬇️

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

@LeafShi1
Copy link
Member

The file name should match the class name ControlCodeDomSerializerTests

@Zheng-Li01 Zheng-Li01 added the waiting-review This item is waiting on review by one or more members of team label Feb 24, 2025
@LeafShi1 LeafShi1 removed the waiting-review This item is waiting on review by one or more members of team label Feb 24, 2025
@Zheng-Li01 Zheng-Li01 added the waiting-review This item is waiting on review by one or more members of team label Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-review This item is waiting on review by one or more members of team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants