palazzem / ha-econnect-alarm

Home Assistant integration that provides a full-fledged Alarm Panel to control your Elmo/IESS alarm systems.
BSD 3-Clause "New" or "Revised" License
10 stars 5 forks source link

tests: enable socket access for `responses` 0.24+ compatibility #104

Closed palazzem closed 7 months ago

palazzem commented 7 months ago

Related Issues

Proposed Changes:

After the change in responses 0.24.0 (https://github.com/getsentry/responses/pull/685/) all tests fail as socket.socket is accessed within the library. This change uses socket_enabled fixture from pytest-socket library so that any time an ElmoClient fixture is used, sockets are enabled.

This change should not have negative impacts as responses is still not making any call to external services.

Testing:

Run any test that access client fixture.

Extra Notes (optional):

n/a

Checklist

coveralls commented 7 months ago

Pull Request Test Coverage Report for Build 6746757894


Totals Coverage Status
Change from base Build 6677072536: 0.0%
Covered Lines: 475
Relevant Lines: 541

💛 - Coveralls