nicklaw5 / helix

A Twitch Helix API client written in Go.
MIT License
240 stars 91 forks source link

Ensure HTTP failure is handled in tests #86

Closed Scorfly closed 3 years ago

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 783079358


Totals Coverage Status
Change from base Build 775330813: 8.9%
Covered Lines: 883
Relevant Lines: 922

💛 - Coveralls
nicklaw5 commented 3 years ago

@Scorfly are you able to explain this PR? I don't understand what it's trying to achieve. Is it purely to increase test code coverage?

Scorfly commented 3 years ago

On code coverage, http failure was never test on every func.

Yes, this PR only aim to increase code coverage