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
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

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

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
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