Skip to content

[churn] Attempt to fix flaky StatusTrackerTest#4347

Merged
ggivo merged 2 commits intomasterfrom
topic/ggivo/fix-flaky-healthcheck-test
Nov 5, 2025
Merged

[churn] Attempt to fix flaky StatusTrackerTest#4347
ggivo merged 2 commits intomasterfrom
topic/ggivo/fix-flaky-healthcheck-test

Conversation

@ggivo
Copy link
Copy Markdown
Collaborator

@ggivo ggivo commented Nov 5, 2025

Error: redis.clients.jedis.mcf.StatusTrackerTest.testWaitForHealthStatus_EventDriven -- Time elapsed: 0.130 s <<< FAILURE!
org.opentest4j.AssertionFailedError: Listener should have been registered ==> expected: not
..
at redis.clients.jedis.mcf.StatusTrackerTest.testWaitForHealthStatus_EventDriven(StatusTrackerTest.java:78)

@ggivo
attempt to fix flaky StatusTrackerTest
8971a54 
Error:  redis.clients.jedis.mcf.StatusTrackerTest.testWaitForHealthStatus_EventDriven -- Time elapsed: 0.130 s <<< FAILURE!
org.opentest4j.AssertionFailedError: Listener should have been registered ==> expected: not <null>
..
	at redis.clients.jedis.mcf.StatusTrackerTest.testWaitForHealthStatus_EventDriven(StatusTrackerTest.java:78)
@ggivo ggivo added testing churn Chore: Formatting and comment cleanup (non-functional change) skip-changelog Ignore pull request from release note labels Nov 5, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 5, 2025

Test Results

   280 files  ±0    280 suites  ±0   11m 28s ⏱️ -20s
10 188 tests ±0  9 123 ✅  - 1 011  1 065 💤 +1 011  0 ❌ ±0 
 2 703 runs  ±0  2 703 ✅ ±    0      0 💤 ±    0  0 ❌ ±0 

Results for commit 8d08002. ± Comparison against base commit 0bcac5c.

This pull request skips 1011 tests. 
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHgetdel
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHgetdelBinary
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHgetex
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHgetexBinary
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHsetex
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHsetexBinary
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[2] ‑ testHgetdel
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[2] ‑ testHgetdelBinary
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[2] ‑ testHgetex
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[2] ‑ testHgetexBinary
…

@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 5, 2025 •
edited
Loading

Test Results

   280 files  ±0     280 suites  ±0   11m 46s ⏱️ -2s
10 188 tests ±0  10 134 ✅ ±0  54 💤 ±0  0 ❌ ±0 
 2 703 runs  ±0   2 703 ✅ ±0   0 💤 ±0  0 ❌ ±0 

Results for commit 8d08002. ± Comparison against base commit 0bcac5c.

♻️ This comment has been updated with latest results.

@ggivo ggivo merged commit 9e0632c into master Nov 5, 2025
13 of 14 checks passed
@ggivo ggivo deleted the topic/ggivo/fix-flaky-healthcheck-test branch March 24, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

churn Chore: Formatting and comment cleanup (non-functional change) skip-changelog Ignore pull request from release note testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ggivo