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

Added missing test cases for LoggerBuilder and index #445

Merged
merged 1 commit into from
Feb 6, 2023

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Feb 4, 2023

Add more tests, for better code coverage:

before

code coverage 50% statements, 30% functions

after

code coverage 74% statements, 69% functions

@susnux susnux added the enhancement New feature or request label Feb 4, 2023
@ChristophWurst ChristophWurst merged commit 54edaaa into master Feb 6, 2023
@delete-merged-branch delete-merged-branch bot deleted the feat/tests branch February 6, 2023 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants