[asterisk-dev] [Code Review] 3264: testsuite: manager/acl-login fails on some platforms

Corey Farrell reviewboard at asterisk.org
Mon Feb 24 23:09:16 CST 2014


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

Review request for Asterisk Developers and Matt Jordan.


Bugs: ASTERISK-23350
    https://issues.asterisk.org/jira/browse/ASTERISK-23350


Repository: testsuite


Description
-------

tests/manager/acl-login assumes that an AMI connection to 127.0.0.2 will be sent from 127.0.0.2.  On some systems all connections to 127.0.0.x are made from 127.0.0.1 by default.

This change includes a fallback of not passing the bindAddress, since this parameter was just added to AMIFactory.login.


Diffs
-----

  /asterisk/trunk/tests/manager/acl-login/run-test 4730 

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


Testing
-------

Tested with AMIFactory.login that doesn't accept bindAddress and with updated starpy that accepts it.  With old starpy the test failed due to all connections being from 127.0.0.1.  With updated starpy the test passed.


Thanks,

Corey Farrell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140225/006662a1/attachment-0001.html>


More information about the asterisk-dev mailing list