Commit b6eec92
Fix 'operation buffered during sync' test to actually test buffering
The test introduced in 6b1c2de was not actually exercising the
buffered-during-sync code path. It called counter.increment(5) and
immediately completed the sync, but the ACK consistently arrived
after sync completion, so the operation was applied directly rather
than being buffered. The test passed even with the buffering
implementation disabled.
Fix by using an ACK interceptor to deterministically control when
the ACK is delivered: intercept it, release it, then yield to the
event loop so publishAndApply enters the sync-wait before we
complete the sync.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>1 parent 4e2ddb0 commit b6eec92
1 file changed
+15
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8815 | 8815 | | |
8816 | 8816 | | |
8817 | 8817 | | |
| 8818 | + | |
| 8819 | + | |
| 8820 | + | |
8818 | 8821 | | |
8819 | | - | |
8820 | 8822 | | |
8821 | 8823 | | |
| 8824 | + | |
| 8825 | + | |
| 8826 | + | |
| 8827 | + | |
| 8828 | + | |
| 8829 | + | |
| 8830 | + | |
| 8831 | + | |
| 8832 | + | |
| 8833 | + | |
| 8834 | + | |
| 8835 | + | |
8822 | 8836 | | |
8823 | 8837 | | |
8824 | 8838 | | |
| |||
0 commit comments