[asterisk-dev] [Code Review] Rework the leave_voicemail_contexts test

Matt Jordan reviewboard at asterisk.org
Tue May 1 14:21:24 CDT 2012


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

Review request for Asterisk Developers.


Summary
-------

This reworks the leave_voicemail_contexts test to:
1) Use the TEST_FRAMEWORK and test events to control the test, as opposed to a lot of Wait() statements
2) Actually test valid extensions that VoiceMail has a hope and a prayer of bouncing a user out to
3) Test the conditions that currently fail in Asterisk, namely the d(context) option failing if the extension the user attempts to go to does not exist in the current dialplan context.

This review being checked in is contingent on review 1892 being approved.


This addresses bug ASTERISK-18243.
    https://issues.asterisk.org/jira/browse/ASTERISK-18243


Diffs
-----

  /asterisk/trunk/tests/apps/voicemail/leave_voicemail_contexts/configs/ast1/extensions.conf 3212 
  /asterisk/trunk/tests/apps/voicemail/leave_voicemail_contexts/configs/ast1/voicemail.conf 3212 
  /asterisk/trunk/tests/apps/voicemail/leave_voicemail_contexts/configs/ast2/extensions.conf 3212 
  /asterisk/trunk/tests/apps/voicemail/leave_voicemail_contexts/run-test 3212 
  /asterisk/trunk/tests/apps/voicemail/leave_voicemail_contexts/test-config.yaml 3212 

Diff: https://reviewboard.asterisk.org/r/1893/diff


Testing
-------


Thanks,

Matt

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120501/112fdb2c/attachment-0001.htm>


More information about the asterisk-dev mailing list