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 undo tests #217

Merged
merged 10 commits into from
Feb 27, 2024
Merged

Add undo tests #217

merged 10 commits into from
Feb 27, 2024

Conversation

rewin123
Copy link
Owner

@rewin123 rewin123 commented Feb 25, 2024

image

~85% code coverage for undo crate

@rewin123 rewin123 marked this pull request as draft February 25, 2024 13:28
Copy link

codecov bot commented Feb 25, 2024

Codecov Report

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

Project coverage is 22.03%. Comparing base (b33cd11) to head (4c65bb9).
Report is 30 commits behind head on main.

Files Patch % Lines
crates/undo/src/lib.rs 50.00% 1 Missing ⚠️
crates/undo/src/tests.rs 99.40% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
+ Coverage   12.74%   22.03%   +9.28%     
==========================================
  Files          54       54              
  Lines        7085     7503     +418     
==========================================
+ Hits          903     1653     +750     
+ Misses       6182     5850     -332     

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

@rewin123 rewin123 requested a review from naomijub February 25, 2024 14:23
@rewin123 rewin123 marked this pull request as ready for review February 25, 2024 14:56
crates/undo/src/lib.rs Outdated Show resolved Hide resolved
crates/undo/src/lib.rs Outdated Show resolved Hide resolved
@rewin123 rewin123 merged commit a2d3951 into main Feb 27, 2024
6 checks passed
@rewin123 rewin123 deleted the add-undo-tests branch February 27, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants