----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3832/#review12810 -----------------------------------------------------------
/asterisk/trunk/tests/cdr/sqlite3/test-config.yaml <https://reviewboard.asterisk.org/r/3832/#comment23142> This probably won't do what you want. A maxversion of '12' is expanded to be a version of '12.0.0'. That means the minversion and the maxversion are the same, both of which are equivalent to the first release of Asterisk 12. Instead, the maxversion here should be '13.0.0', that is, the first release of Asterisk 13 (which is less than any run of the 13 branch). The code that checks whether or not a pluggable module can be run makes sure that the version is between these two points: if (modminversion is not None and AsteriskVersion(ast_version) < AsteriskVersion(modminversion)): return False if (modmaxversion is not None and AsteriskVersion(ast_version) >= AsteriskVersion(modmaxversion)): return False /asterisk/trunk/tests/pbx/accountcode/dial_none/configs/ast1/extensions.conf <https://reviewboard.asterisk.org/r/3832/#comment23144> Having to wait 5 seconds for this test to end when it has essentially completed is usually something we avoid. There is another route you can take, however: (1) Drop the channel into Echo after answering it (2) Use the channel hangup module to hangup the channel on the Newexten event: modules: - config-section: hangup-channel typename: 'pluggable_modules.AMIChannelHangup' hangup-channel: id: '1' conditions: match: Event: 'Newexten' Channel: 'PJSIP/bob-.*' Application: Echo count: '1' This also supports a 'delay' parameter in case the hangup *does* need to wait a second or two. This finding would extend to the other tests that use the same pattern. /asterisk/trunk/tests/pbx/accountcode/dial_none/test-config.yaml <https://reviewboard.asterisk.org/r/3832/#comment23143> I don't think you need the dependency on func_channel for this test. - Matt Jordan On July 21, 2014, 4:49 p.m., rmudgett wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3832/ > ----------------------------------------------------------- > > (Updated July 21, 2014, 4:49 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: AFS-65 > https://issues.asterisk.org/jira/browse/AFS-65 > > > Repository: testsuite > > > Description > ------- > > New tests to check accountcode propagation with the new > accouncode/peeracccount interaction. > > * Made pluggable_modules.py Originator class and test_case.py > SimpleTestCase class call origination allow specifying the accountcode. > > * Fix tests/cdr/sqlite3 to work with the new accountcode propagation rules. > > * Add accountcode tag to existing tests doing something with accountcode. > > Testsuite tests to go with https://reviewboard.asterisk.org/r/3601/ > > > Diffs > ----- > > /asterisk/trunk/tests/pbx/tests.yaml 5288 > /asterisk/trunk/tests/pbx/accountcode/tests.yaml PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/queue_peer_preserve/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/queue_peer_preserve/configs/ast1/queues.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/queue_peer_preserve/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/queue_peer_preserve/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/queue_peer_none/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/queue_peer_none/configs/ast1/queues.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/queue_peer_none/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/queue_peer_none/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/queue_none/test-config.yaml > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/queue_none/configs/ast1/queues.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/queue_none/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/queue_none/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/local_originate/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/local_originate/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/local_crossover_back/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/local_crossover_back/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/local_crossover/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/local_crossover/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_straight_override/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_straight_override/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_straight_override/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/dial_straight_none/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_straight_none/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_straight_none/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/dial_predial/test-config.yaml > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/dial_predial/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_predial/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/dial_peer_override/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_peer_override/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_peer_override/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/dial_peer_none/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_peer_none/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_peer_none/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/dial_none/test-config.yaml > PRE-CREATION > /asterisk/trunk/tests/pbx/accountcode/dial_none/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/pbx/accountcode/dial_none/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/cdr/sqlite3/test-config.yaml 5288 > /asterisk/trunk/tests/cdr/console_dial_sip_transfer/test-config.yaml 5288 > /asterisk/trunk/tests/cdr/console_dial_sip_congestion/test-config.yaml 5288 > /asterisk/trunk/tests/cdr/console_dial_sip_busy/test-config.yaml 5288 > /asterisk/trunk/tests/cdr/console_dial_sip_answer/test-config.yaml 5288 > /asterisk/trunk/tests/cdr/cdr_unanswered_yes/test-config.yaml 5288 > /asterisk/trunk/tests/cdr/cdr_properties/cdr_accountcode/test-config.yaml > 5288 > > /asterisk/trunk/tests/cdr/cdr_properties/blind-transfer-accountcode/test-config.yaml > 5288 > /asterisk/trunk/tests/cdr/cdr_originate_sip_congestion_log/test-config.yaml > 5288 > > /asterisk/trunk/tests/cdr/cdr_manipulation/console_fork_before_dial/test-config.yaml > 5288 > > /asterisk/trunk/tests/cdr/cdr_manipulation/console_fork_after_busy_forward/test-config.yaml > 5288 > > /asterisk/trunk/tests/cdr/cdr_manipulation/cdr_fork_end_time/test-config.yaml > 5288 > /asterisk/trunk/lib/python/asterisk/test_case.py 5288 > /asterisk/trunk/lib/python/asterisk/pluggable_modules.py 5288 > > Diff: https://reviewboard.asterisk.org/r/3832/diff/ > > > Testing > ------- > > All accountcode tagged tests pass. > > > Thanks, > > rmudgett > >
-- _____________________________________________________________________ -- 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
