[test-results] [Bamboo] Asterisk Testing > Asterisk Trunk > #540 has FAILED (4 tests failed, no failures were new). Change made by Mark Michelson.

Bamboo bamboo at asterisk.org
Tue Jul 31 20:47:21 CDT 2012


-----------------------------------------------------------------------
Asterisk Testing > Asterisk Trunk > #540 failed.
-----------------------------------------------------------------------
Code has been updated by Mark Michelson.
4/281 tests failed, no failures were new.

http://bamboo.asterisk.org/browse/TESTING-ASTERISKTRUNK-540/


--------------
Failing Jobs
--------------
  - Asterisk CentOS 6 64-Bit (CentOS 6): 4 of 281 tests failed.



--------------
Code Changes
--------------
Mark Michelson (370644):

>Add "dialplan remove context" and modify "dialplan add include"
>
>From corruptor's review board posting:
>
>"I've noticed that we can remove particular extension from context with
>dialplan remove extension command but in order to remove all extensions
>in the context we should delete them on by one. I've created dialplan
>remove context command which uses ast_context_destroy to destroy the
>whole context with all extensions. I've created to functions for in
>pbx_config.c: handle_cli_dialplan_remove_context which actually removes
>context and complete_dialplan_remove_context which completes input.
>They are based on other similar functions and pretty trivial but I can be
>mistaken somewhere.
>
>"I've also modified dialplan add include <context2> into <context1>. I've
>made it similar dialplan add extension ... command. It creates <context1>
>if it doesn't exist and I've also modified complete_dialplan_add_include
>and removed check for existance of <context2> because we can include
>non-existent context into another one. (I usually include empty
>(non-existent) contexts in advance). Should we raise warning in this case
>as it's raised while reading extensions.conf?
>
>"I use those functions with AMI. I think manager commands should be created
>in addition to those CLI commands."
>
>I've addressed the latest comments on review board and have made some other
>coding guidelines-related cleanup. I also have modified the CHANGES file to
>mention these new commands.
>
>(closes issue ASTERISK-19292)
>reported by Andrey Solovyev
>
>Patches:
>	dialplan_add_include.patch
>    uploaded by Andrey Solovyev (license #5214)
>    dialplan_remove_context.patch
>    uploaded by Andrey Solovyev (license #5214)
>
>Review: https://reviewboard.asterisk.org/r/2042
>
>



--------------
Tests
--------------
Existing Test Failures (4)
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/multiple state change
   - AsteriskTestSuite: S/channels/ s i p/generic ccss
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/nominal state change
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/resubscribe

--
This message is automatically generated by Atlassian Bamboo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/test-results/attachments/20120731/8a25b804/attachment-0001.htm>


More information about the Test-results mailing list