test: call - add call on-hold/resume test #990
Conversation
cspiel1
changed the title
can be used for unit tests
cspiel1
force-pushed
the
sipsess_ack_pending
branch
from
02034d2
to
770a3ab
Compare
|
Now I reverted (temporarily) the revert of @juha-h in order to reproduce the missing ACK. On my local dev host I was not able to reproduce this with this unit test baresip/baresip#2775 that now also checks if the ACK was received. Put this to draft. We have to clarify with @juha-h how we can reproduce the issue baresip/baresip#2772 first. Edit: Now I have a test that fails. But the |
cspiel1
force-pushed
the
sipsess_ack_pending
branch
from
770a3ab
to
71d7193
Compare
|
This PR and the related baresip/baresip#2775 is not able to reproduce the issue baresip/baresip#2772. The issue appears in combination with an outbound SIP proxy. I removed the commit fc588b9 from this PR. The reason for this PR is to add a function for checking if there is an ACK missing. This can be used in baresip/baresip#2775 for a baresip selftest in order to test call on-hold and resume and check also a final ACK for a re-INVITE. |
|
Thanks for this patch. Looks good to me. Should we merge it ? |
can be used for unit tests