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 more unittests for untested lines of code #142

Merged
merged 1 commit into from
Feb 13, 2022

Conversation

joro75
Copy link
Collaborator

@joro75 joro75 commented Feb 12, 2022

Checked the lines of code that didn't get covered by the unittests, and just added more unittests to ensure that more lines of code are being tested.

Copy link
Owner

@DurgNomis-drol DurgNomis-drol left a comment

Choose a reason for hiding this comment

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

LGTM!

Awesome work you are doing!

@DurgNomis-drol DurgNomis-drol merged commit 4456499 into DurgNomis-drol:master Feb 13, 2022
@joro75 joro75 deleted the MoreUnitTests branch February 14, 2022 20:58
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.

3 participants