search

WebJamApps / breakpointministries

MIT License
0 stars 0 forks source link

Finishes unit tests #139

Closed logrdog closed 2 years ago

logrdog commented 2 years ago

136

Also stumped why my coverage would be 0. Reason: TBD

JoshuaVSherman commented 2 years ago

@logrdog I can pair on this with you if you need help or finish it up and give you the solution if you are totally stuck, just let me know

logrdog commented 2 years ago

@JoshuaVSherman yeah sorry, I am stumped on why coverage fails even after the fact every test passes. I do need help on why that would be happening, whatever option would be easier :+1:

codeclimate[bot] commented 2 years ago

Code Climate has analyzed commit 54faf884 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Bug Risk 1

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 100.0% (0.5% change).

View more on Code Climate.

JoshuaVSherman commented 2 years ago

@logrdog I did some upgrades and added a new test, this branch works now for me and is at 100 percent coverage, so please review the changes and let me know when you are ready, then I will merge it. Talk to you later