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.

Sign up for GitHub

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

Jump to bottom

📈 Increase Code Coverage to 100% #110

Merged
merged 18 commits into from
Aug 18, 2021
Merged

📈 Increase Code Coverage to 100% #110

merged 18 commits into from
Aug 18, 2021

Conversation

TeoZosa
Copy link
Owner

@TeoZosa TeoZosa commented Aug 18, 2021

No description provided.

TeoZosa added 16 commits August 18, 2021 12:48
@TeoZosa
🔧 Exclude private files not in the VCS from coverage reports
b56dbb5
@TeoZosa
♻️ For closed set membership testing, make final case an else
c675d86
@TeoZosa
💥 Throw exception on pre-condition violation
d948e23 
Namely, that the input will always contain an element that appears once
and only once.
@TeoZosa
⚰️ Remove deprecated functions
f64ef63
@TeoZosa
📝 Add docstring for reverse_sub_list() driver fn
7cace77
@TeoZosa
✅ Add trivial reverse_sub_list() driver fn test
9c0399d
@TeoZosa
♻️ Make merge_alt() divide & conquer step self-recursive
621ee39
@TeoZosa
👽 Add optional do_iterative_merge parameter
621f44f 
Allow users to switch on which internal merge implementation is used.
@TeoZosa
♻️ Dynamically generate test list nodes
931a4fa
@TeoZosa
✅ Test internal recursive merge implementation
e08eabe
@TeoZosa
♻️ Refactor variadic arguments into positional arguments
d1bbd8a
@TeoZosa
✅ Add missing subtraction functionality test
1dca01a
@TeoZosa
✅ Add missing edge case tests
9a99475
@TeoZosa
🎨 Assign left & right chars at start of iteration
2e24b25 
Eschewing walrus operator assignment for more standard (read:
easier to follow when reading the code) tuple assignment pattern.
@TeoZosa
📈 Exclude intentional structurally partial branches from coverage
2265dfa 
See: https://coverage.readthedocs.io/en/coverage-5.5/branch.html#structurally-partial-branches
@TeoZosa
👷 Fail coverage runs under 100% code coverage
ff73186
@TeoZosa TeoZosa added ci Changes to CI configuration files and scripts refactoring A code change that neither fixes a bug nor adds a feature testing Adding missing tests or correcting existing tests labels Aug 18, 2021
@TeoZosa TeoZosa self-assigned this Aug 18, 2021
@TeoZosa TeoZosa changed the title 📈 Increase Code Coverage to 100 Percent 📈 Increase Code Coverage to 100% Aug 18, 2021
TeoZosa added 2 commits August 18, 2021 14:29
@TeoZosa
✨ Add _clear_cache class method
e901b79
@TeoZosa
✅ Fix climb_stairs_tabulate() tests
97973c2 
Primary code path was not being executed due to an already
populated class cache (from prior tests).
@TeoZosa TeoZosa merged commit ab9f6ea into master Aug 18, 2021
@TeoZosa TeoZosa deleted the increase-code-coverage-to-100-percent branch August 18, 2021 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@TeoZosa TeoZosa

Labels
ci Changes to CI configuration files and scripts refactoring A code change that neither fixes a bug nor adds a feature testing Adding missing tests or correcting existing tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@TeoZosa