Node: Fix flaky FLUSHALL timeout in commonjs-test.cjs

[xShinnRyuu]

Summary

Fix flaky FLUSHALL timeout in the commonjs-test.cjs hybrid node modules test by increasing the client's requestTimeout from the default 250ms to 5000ms.

Issue link

This Pull Request is linked to issue: [Node][Flaky Test] commonjs-test.cjs - FLUSHALL timeout in hybrid node modules test
Closes #6736

Features / Behaviour Changes

No behaviour changes. This PR fixes test flakiness only.

Implementation

Root cause: The GlideClient in commonjs-test.cjs was using the default requestTimeout of 250ms. In CI environments (GitHub Actions runners) under load, the system can be saturated even at command submission time, causing simple commands like FLUSHALL to exceed this timeout. The timeout_watchdog confirmed: "system was already saturated at submission time."

Fix: Set requestTimeout: 5000 (5 seconds) in the GlideClient.createClient configuration. This is generous enough for CI environments under load while still catching actual connection issues. This aligns with the pattern used in the main test suite (TestUtilities.ts uses 1000ms, and specific tests use up to 10000ms).

Limitations

None

Testing

• Verified the fix is consistent with the requestTimeout configuration pattern used throughout the test suite (e.g., node/tests/TestUtilities.ts sets 1000ms by default, GlideClient.test.ts uses 10000ms for certain operations)
• The change is a single configuration value addition with no logic changes

Checklist

• This Pull Request is related to one issue.
• Commit message has a detailed description of what changed and why.
• Tests are added or updated.
• CHANGELOG.md updated - N/A for flaky-test fixes; do NOT add a changelog entry.
• Lint checked manually (matched surrounding style; lint tools not run locally).
• Destination branch is correct - main or release

[valkey-review-bot]

Commit 2a71f22 needs two metadata fixes before merge:

• The DCO check fails because the author is Thomas Zhou <54688146+xShinnRyuu@users.noreply.github.com>, while the Signed-off-by line is xShinnRyuu <xshinnryuu@users.noreply.github.com>. Amend the commit so its sign-off matches the author identity.
• The subject Node: Increase requestTimeout in commonjs-test.cjs to fix flaky FLUSHALL timeout does not follow the repository's required <type>(<scope>): <description> format. A fitting subject would be test(node): increase request timeout in CommonJS hybrid test.

Please amend the commit and force-push the branch.

[valkey-review-bot]

The test-only timeout change matches the Node client configuration contract and the existing 5-second example. No code-level concerns found. The commit metadata issues are noted separately because they cannot be anchored to the diff.