diff options
| author | Felix Lange <fjl@users.noreply.github.com> | 2018-10-09 22:34:24 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-10-09 22:34:24 +0800 |
| commit | 4e474c74dc2ac1d26b339c32064d0bac98775e77 (patch) | |
| tree | 0d0d40fd543b039852252f1193c0eaa22a695a8e /internal | |
| parent | da290e9707d6b3074b571722bbb557815b03ad48 (diff) | |
| download | dexon-4e474c74dc2ac1d26b339c32064d0bac98775e77.tar dexon-4e474c74dc2ac1d26b339c32064d0bac98775e77.tar.gz dexon-4e474c74dc2ac1d26b339c32064d0bac98775e77.tar.bz2 dexon-4e474c74dc2ac1d26b339c32064d0bac98775e77.tar.lz dexon-4e474c74dc2ac1d26b339c32064d0bac98775e77.tar.xz dexon-4e474c74dc2ac1d26b339c32064d0bac98775e77.tar.zst dexon-4e474c74dc2ac1d26b339c32064d0bac98775e77.zip | |
rpc: fix subscription corner case and speed up tests (#17874)
Notifier tracks whether subscription are 'active'. A subscription
becomes active when the subscription ID has been sent to the client. If
the client sends notifications in the request handler before the
subscription becomes active they are dropped. The tests tried to work
around this problem by always waiting 5s before sending the first
notification.
Fix it by buffering notifications until the subscription becomes active.
This speeds up all subscription tests.
Also fix TestSubscriptionMultipleNamespaces to wait for three messages
per subscription instead of six. The test now finishes just after all
notifications have been received and doesn't hit the 30s timeout anymore.
Diffstat (limited to 'internal')
0 files changed, 0 insertions, 0 deletions