[asterisk-commits] dlee: branch dlee/matts-ari-patch r4147 - /asterisk/team/dlee/matts-ari-patch/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Sep 10 13:40:06 CDT 2013


Author: dlee
Date: Tue Sep 10 13:40:02 2013
New Revision: 4147

URL: http://svnview.digium.com/svn/testsuite?view=rev&rev=4147
Log:
Give ARI its own test suite object

Testing ARI is a bit different than testing normal Asterisk tests. Typically, a
test in the Asterisk Test Suite uses the following model:

1. Create Asterisk Instances
2. Wait for instances to fully boot
3. Connect AMI
4. Wait for AMI connection
5. Spawn channels for test

Most of the test logic occurs in step number 5 - while it's possible to inject
logic into the first four steps without writing a new test object, it isn't
easy, particularly if what you need to do is defer the execution of one of
those four steps until something else happens. In particular, ARI tests need
to connect the websocket prior to step 4, as the AMI connection is what
typically causes channels to be created in the dialplan and test execution to
begin. If the websocket connection doesn't happen before that point, a channel
may enter into the Stasis application before the websocket connects and the
test will fail.

This patch gives ARI its own test object so that it can properly connect the
websocket prior to the AMI connection. This changes the startup routine to be:

1. Create Asterisk Instances
2. Wait for instances to fully boot
3. Connect ARI
4. Wait for ARI connection
5. Connection AMI
6. Wait for AMI connection
7. Spawn channels for test

Review: https://reviewboard.asterisk.org/r/2661/
........

Merged revisions 4065 from http://svn.asterisk.org/svn/testsuite/asterisk/trunk

Modified:
    asterisk/team/dlee/matts-ari-patch/   (props changed)

Propchange: asterisk/team/dlee/matts-ari-patch/
------------------------------------------------------------------------------
--- svnmerge-integrated (original)
+++ svnmerge-integrated Tue Sep 10 13:40:02 2013
@@ -1,1 +1,1 @@
-/asterisk/trunk:1-4064
+/asterisk/trunk:1-4065




More information about the asterisk-commits mailing list