Fix flaky test_idle_heartbeat and CI workflow YAML syntax

The test_idle_heartbeat test could fail with a KeyError when
shard-aware reconnection replaced connections during the sleep
interval. Skip connections not present in the original snapshot.

Also fix a trailing whitespace/dash in integration-tests.yml
that caused a YAML syntax issue in the paths-ignore list.

Author: dkropachev

.github/workflows/integration-tests.yml

@@ -29,7 +29,7 @@ on:
      - .github/dependabot.yml
      - .github/pull_request_template.md
       - "*.md"
-      - .github/workflows/docs-*     -
+      - .github/workflows/docs-*
   workflow_dispatch:
 
 jobs:

tests/integration/standard/test_cluster.py

@@ -746,8 +746,11 @@ def test_idle_heartbeat(self):
 
         connections = [c for holders in cluster.get_connection_holders() for c in holders.get_connections()]
 
-        # make sure requests were sent on all connections
+        # make sure requests were sent on all connections that existed before the sleep
+        # (shard-aware reconnection may replace connections during the sleep interval)
         for c in connections:
+            if id(c) not in connection_request_ids:
+                continue
             expected_ids = connection_request_ids[id(c)]
             expected_ids.rotate(-1)
             with c.lock: