Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Exception coverage. #88

Merged
merged 1 commit into from
Oct 31, 2022
Merged

Exception coverage. #88

merged 1 commit into from
Oct 31, 2022

Conversation

tristanmorgan
Copy link
Owner

Description

Pushing the Coverage of Rspec tests closer to 100% and found a couple of rescue blocks were redundant.

Did you run the Tests?

  • Rubocop
  • Rspec
  • Filemode
  • Yard

@servian-sheldon-gill
Copy link

99.66% is pretty good ;)

@tristanmorgan tristanmorgan merged commit d648dcf into main Oct 31, 2022
@tristanmorgan tristanmorgan deleted the exception_coverage branch October 31, 2022 02:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@servian-sheldon-gill servian-sheldon-gill servian-sheldon-gill approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tristanmorgan @servian-sheldon-gill