IntervalRunner: don't wait for interval_seconds on shutdown

[KostyaEsmukov]

Summary

IntervalRunner.shutdown method currently has to wait the full interval_seconds period due to using a time.sleep call. This change speeds up the shutdown by replacing it with a threading.Event.wait call.

Category (place an x in each of the [ ])

• slack_sdk.web.WebClient (sync/async) (Web API client)
• slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
• slack_sdk.models (UI component builders)
• slack_sdk.oauth (OAuth Flow Utilities)
• slack_sdk.socket_mode (Socket Mode client)
• slack_sdk.audit_logs (Audit Logs API client)
• slack_sdk.scim (SCIM API client)
• slack_sdk.rtm (RTM client)
• slack_sdk.signature (Request Signature Verifier)
• /docs-src (Documents, have you run ./docs.sh?)
• /docs-src-v2 (Documents, have you run ./docs-v2.sh?)
• /tutorial (PythOnBoardingBot tutorial)
• tests/integration_tests (Automated tests for this library)

Requirements (place an x in each [ ])

• I've read and understood the Contributing Guidelines and have done my best effort to follow them.
• I've read and agree to the Code of Conduct.
• I've run python setup.py validate after making the changes. [Some e2e tests don't pass with a ConnectionResetError: [Errno 54] Connection reset by peer error]

[KostyaEsmukov]

BTW, the python setup.py validate command fails in a clean venv when running tests with the following error: E ModuleNotFoundError: No module named 'aiohttp'.

[seratch]

@KostyaEsmukov

BTW, the python setup.py validate command fails in a clean venv when running tests with the following error: E ModuleNotFoundError: No module named 'aiohttp'.

Thanks for pointing this out. It needs to be updated. I will do it later.

[codecov]

Codecov Report

Merging #1035 (e421260) into main (fc90f6f) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1035      +/-   ##
==========================================
- Coverage   84.34%   84.34%   -0.01%     
==========================================
  Files          95       95              
  Lines        8878     8877       -1     
==========================================
- Hits         7488     7487       -1     
  Misses       1390     1390              

Impacted Files	Coverage Δ
slack_sdk/socket_mode/interval_runner.py	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fc90f6f...e421260. Read the comment docs.

[seratch]

Thanks for the solid improvement - LGTM 👍