Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Oct 31, 2024

This PR intends to add more unit tests to increase the code coverage.

Signed-off-by: Patrick Zheng <[email protected]>
@codecov
Copy link

codecov bot commented Oct 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.35%. Comparing base (eb76e2a) to head (9b05a6d).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1075      +/-   ##
==========================================
- Coverage   72.02%   70.35%   -1.68%     
==========================================
  Files          46       46              
  Lines        2302     2911     +609     
==========================================
+ Hits         1658     2048     +390     
- Misses        442      670     +228     
+ Partials      202      193       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Patrick Zheng added 3 commits October 31, 2024 17:39
Signed-off-by: Patrick Zheng <[email protected]>
Signed-off-by: Patrick Zheng <[email protected]>
Signed-off-by: Patrick Zheng <[email protected]>
Signed-off-by: Patrick Zheng <[email protected]>
@ghost ghost requested a review from shizhMSFT November 1, 2024 02:54
Copy link
Contributor

@shizhMSFT shizhMSFT left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@JeyJeyGao JeyJeyGao left a comment

Choose a reason for hiding this comment

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

LGTM

@priteshbandi priteshbandi merged commit 445030b into notaryproject:main Nov 1, 2024
@ghost ghost deleted the addTest branch November 3, 2024 12:13
7h3-3mp7y-m4n pushed a commit to 7h3-3mp7y-m4n/notation that referenced this pull request Mar 29, 2025
This PR intends to add more unit tests to increase the code coverage.

---------

Signed-off-by: Patrick Zheng <[email protected]>
FeynmanZhou pushed a commit to FeynmanZhou/notation that referenced this pull request May 15, 2025
This PR intends to add more unit tests to increase the code coverage.

---------

Signed-off-by: Patrick Zheng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants