Feat/streaming disconnect logging

[Tymek]

Refactor streaming connector to add disconnect logging and simplify _run.

I changed iteration to client.all from client.events to log disconnection warnings. After that _run had deep nesting. I pulled setup, event handling, error handling, and cleanup into smaller, flatter methods.

Example log when disconnected from Edge now looks like this:

INFO:UnleashClient:Connecting to stream at http://localhost:3063/api/client/streaming
DEBUG:urllib3.connectionpool:Resetting dropped connection: localhost
WARNING:UnleashClient:SSE stream error: <urllib3.connection.HTTPConnection object at 0x7fb2863b02d0>: Failed to establish a new connection: [Errno 111] Connection refused - will retry
INFO:UnleashClient:Will reconnect after delay of 3.090826s

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

• Unit tests
• Spec Tests
• Integration tests / Manual Tests

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[coveralls]

Pull Request Test Coverage Report for Build 17670269762

Details

• 33 of 41 (80.49%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.1%) to 94.083%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
UnleashClient/connectors/streaming_connector.py	33	41	80.49%

Totals
Change from base Build 17488384346:	0.1%
Covered Lines:	636
Relevant Lines:	676

---

💛 - Coveralls

[kwasniew]

not sure if for the unknown events we want warnings or maybe debug instead

[kwasniew]

we may add new event types over time and it's not sth to warn about and pollute logs at this level

[stale]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.