[asterisk-dev] [Code Review] 3419: Test for PJSIP fast picture update

Benjamin Keith Ford reviewboard at asterisk.org
Thu Apr 10 16:21:43 CDT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3419/
-----------------------------------------------------------

(Updated April 10, 2014, 9:21 p.m.)


Review request for Asterisk Developers.


Changes
-------

- Updated xml files to better address solution


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 (updated)
-----

  ./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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140410/a8638e1d/attachment.html>


More information about the asterisk-dev mailing list