FileReceiverTest is flaky #467

Activity

1. mgaffigan

   added

   bugSomething isn't workingSomething isn't working

   on Sep 30, 2026
2. e210 commented on Oct 4, 2026

   

   e210

   on Oct 4, 2026

   Contributor

   More actions

   Found the culprit, and it's a sneaky one 🕵️

   createReceiver() calls receiver.start(), which schedules a real background poll every 5 s. Sometimes that poll sneaks in between the test's two manual poll() calls and adds an extra batch, so the exact count no longer matches. Same root cause as #414, I think.

   I reproduced it locally: wait 5.5 s between the two polls and testPoll2 fails every time (32 instead of 16).

   The fix is one line in createReceiver():

   connectorProperties.getPollConnectorProperties().setPollingFrequency(Integer.MAX_VALUE);

   In other words, we politely ask the scheduler to stop photobombing the test 📸

   Happy to open a PR if that works for you!
3. mgaffigan commented on Oct 5, 2026

   

   mgaffigan

   on Oct 5, 2026

   ContributorAuthor

   More actions

   @e210, thanks for the investigation! Please send a PR if you can. I agree it seems similar to 414.
4. MrigankKumawat commented on Oct 6, 2026

   

   MrigankKumawat

   on Oct 6, 2026

   More actions

   Hey @mgaffigan, I came across this while researching CI test stability. Dealing with tests that fail on the initial run and pass on retry masks whether the problem is in the test body, timing, or the CI runner environment. I'm building an open-source tool called Flaky-Repro that focuses on reproducing intermittent test failures by systematically varying execution conditions. If you're still working through this, happy to share how we approach reproducing these or see if varying execution conditions helps recreate it.
5. mgaffigan commented on Oct 6, 2026

   

   mgaffigan

   on Oct 6, 2026

   ContributorAuthor

   More actions

   @MrigankKumawat,

   Thanks for reaching out. This one reproduces easily by running in a loop, but I'll keep an eye out for the tool.
6. mgaffigan

   added a commit that references this issue on Oct 8, 2026

   Stop background polls from racing FileReceiverTest (#467)

   ...

   d9520f9
7. mgaffigan

   closed this as completedin #475on Oct 8, 2026
8. tonygermano

   added this to the Next Release milestone on Oct 8, 2026