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

Added code coverage for FuzzyObject. #203

Merged
merged 6 commits into from
Jan 16, 2023
Merged

Added code coverage for FuzzyObject. #203

merged 6 commits into from
Jan 16, 2023

Conversation

rneudorf-godaddy
Copy link
Contributor

@rneudorf-godaddy rneudorf-godaddy commented Jan 13, 2023 •
edited by unfulvio-godaddy
Loading

Summary

This PR adds missing code coverage for FuzzyObject.

Closes: N/A

Details

  • PR adds coverage for all functions.
  • Test for __toString() does not explicitly cover every single possible permutation of variables, just the three main cases.

Contributor checklist

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly
  • I have added tests to cover changes introduced by this pull request
  • All new and existing tests pass

Testing

Reviewer checklist

  • Code changes review
  • Documentation changes review
  • Unit tests pass
  • Static analysis passes

@coveralls
Copy link

coveralls commented Jan 13, 2023 •
edited
Loading

Coverage Status

Coverage: 41.94% (+5.2%) from 36.749% when pulling fd1fcb1 on chore/add-code-coverage-for-fuzzyobject into 9355ebf on trunk.

@unfulvio-godaddy unfulvio-godaddy added the Tests Updates to tests label Jan 16, 2023
@unfulvio-godaddy unfulvio-godaddy added this to the 0.6.0 milestone Jan 16, 2023
@unfulvio-godaddy unfulvio-godaddy merged commit 4a46042 into trunk Jan 16, 2023
@unfulvio-godaddy unfulvio-godaddy deleted the chore/add-code-coverage-for-fuzzyobject branch January 16, 2023 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@unfulvio-godaddy unfulvio-godaddy unfulvio-godaddy approved these changes

Assignees

@rneudorf-godaddy rneudorf-godaddy

Labels
Tests Updates to tests
Projects
None yet
Milestone
v1.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@rneudorf-godaddy @coveralls @unfulvio-godaddy @ohryan