fix bug: issue #3595 and add unit test

[kael-aiur]

Make sure that:

• You have read the contribution guidelines.
• You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
• You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
• You submit test cases (unit or integration tests) that back your changes.

---

Note

Medium Risk
Changes the stale-connection eviction logic for pooled cluster connections, which can affect when connections are closed during topology/role changes. Risk is moderate because it alters behavior during failover and could cause extra reconnects if misclassified.

Overview
Fixes stale pooled-connection detection during cluster reconfiguration by treating connections as stale when a node’s role no longer matches the connection intent (e.g., cached READ connection to a node that is now UPSTREAM, or WRITE connection to a node that is now REPLICA).

Adds a unit test covering replica promotion to master to ensure existing cached read connections are closed after setPartitions() updates the cluster view.

^{Reviewed by Cursor Bugbot for commit 69be339. Bugbot is set up for automated code reviews on this repo. Configure here.}

[a-TODO-rov]

Thanks @kael-aiur
Will review this soon.

[jit-ci]

🛡️ Jit Security Scan Results

✅ No security findings were detected in this PR

---

^{Security scan by Jit}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

[kael-aiur]

I think the test fail on redis 7.4 and 8.2 is appears to be a flaky test unrelated to my changes. The test failure is in scriptLoad() which is unrelated to the clusterMyId() fallback logic I modified？