Fix ReverseInterfaceTest.handle_program_state

[urfeex]

Attempting to fix thread synchronization. I'm not entirely sure that that's the problem as I cannot reprocude things locally and the output suggests that there's a client connecting twice, but it's something that could be fixed.

---

Note

Low Risk
Test-only synchronization changes; low product risk, with the main risk being potential remaining flakiness/timeouts in CI due to timing assumptions.

Overview
Fixes flakiness in ReverseIntefaceTest.handle_program_state by tightening thread synchronization around program-state notifications.

handleProgramState now marks when a new state is received, and waitForProgramState returns immediately if the expected state is already set; otherwise it loops with a predicate-based condition_variable wait and re-checks state until the timeout expires.

^{Written by Cursor Bugbot for commit 5237abb. This will update automatically on new commits. Configure here.}

[urfeex]

bugbot run

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.34%. Comparing base (68fe8be) to head (67fe8fa).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #430      +/-   ##
==========================================
- Coverage   74.35%   74.34%   -0.01%     
==========================================
  Files         103      103              
  Lines        5131     5130       -1     
  Branches      545      545              
==========================================
- Hits         3815     3814       -1     
  Misses       1034     1034              
  Partials      282      282              

Flag	Coverage Δ
start_ursim	83.33% <ø> (+1.78%)	⬆️
ur5-3.14.3	72.47% <ø> (+0.02%)	⬆️
ur5e-10.7.0	64.46% <ø> (-0.07%)	⬇️
ur5e-5.9.4	72.77% <ø> (-0.11%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cursor]

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

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on February 20

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.