-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3419/
-----------------------------------------------------------
Review request for Asterisk Developers.
Bugs: ASTERISK-23562
https://issues.asterisk.org/jira/browse/ASTERISK-23562
Repository: testsuite
Description
-------
This test verifies that when two PJSIP endpoints are in a call with video
enabled, fast picture updates can be received when one endpoint requests INFO.
The following code was added as well to create a TestEvent that spawns when the
INFO is created:
Index: channels/chan_pjsip.c
===================================================================
--- channels/chan_pjsip.c (revision 408267)
+++ channels/chan_pjsip.c (working copy)
@@ -57,6 +57,7 @@
#include "asterisk/stasis_endpoints.h"
#include "asterisk/stasis_channels.h"
#include "asterisk/indications.h"
+#include "asterisk/manager.h"
#include "asterisk/res_pjsip.h"
#include "asterisk/res_pjsip_session.h"
@@ -1112,8 +1113,14 @@
ao2_cleanup(channel->session);
}
}
+ manager_event(EVENT_FLAG_TEST, "TestEvent",
+ "Type: AST_CONTROL_VIDUPDATE\r\n"
+ "Result: Success");
} else {
res = -1;
+ manager_event(EVENT_FLAG_TEST, "TestEvent",
+ "Type: AST_CONTROL_VIDUPDATE\r\n"
+ "Result: Failure");
}
break;
case AST_CONTROL_CONNECTED_LINE:
Diffs
-----
./asterisk/trunk/tests/channels/pjsip/video_calls/tests.yaml PRE-CREATION
./asterisk/trunk/tests/channels/pjsip/video_calls/fast_picture_update/test-config.yaml
PRE-CREATION
./asterisk/trunk/tests/channels/pjsip/video_calls/fast_picture_update/sipp/start-call.xml
PRE-CREATION
./asterisk/trunk/tests/channels/pjsip/video_calls/fast_picture_update/sipp/receive-call.xml
PRE-CREATION
./asterisk/trunk/tests/channels/pjsip/video_calls/fast_picture_update/configs/ast1/pjsip.conf
PRE-CREATION
./asterisk/trunk/tests/channels/pjsip/video_calls/fast_picture_update/configs/ast1/extensions.conf
PRE-CREATION
./asterisk/trunk/tests/channels/pjsip/tests.yaml 4918
Diff: https://reviewboard.asterisk.org/r/3419/diff/
Testing
-------
Thanks,
Benjamin Keith Ford
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
