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 tests for the failing cases of PyDictManager #151

Merged
merged 2 commits into from
Nov 23, 2022

Conversation

MegaRedHand
Copy link
Contributor

This PR adds more unit tests for PyDictManager and PyDictTracker, implemented on #118

@MegaRedHand MegaRedHand self-assigned this Nov 18, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #151 (a0bc804) into dict-manager (f10e775) will increase coverage by 1.41%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           dict-manager     #151      +/-   ##
================================================
+ Coverage         72.80%   74.22%   +1.41%     
================================================
  Files                13       13              
  Lines               706      706              
================================================
+ Hits                514      524      +10     
+ Misses              192      182      -10     
Impacted Files Coverage Δ
src/dict_manager.rs 88.88% <ø> (+5.55%) ⬆️
src/ids.rs 43.75% <0.00%> (+5.46%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Jrigada Jrigada merged commit 6128817 into dict-manager Nov 23, 2022
@Jrigada Jrigada deleted the dict-manager-test-errors branch November 23, 2022 13:03
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.

5 participants