Review Board 1.7.16


testsuite: fix a number of tests where Asterisk does not shutdown gracefully

Review Request #4411 - Created Feb. 9, 2015 and submitted

Corey Farrell
Reviewers
asterisk-dev
testsuite
* Add Hangup() to priority after Dial() where needed.  This prevents auto-fallthrough from playing 10 seconds of BUSY or CONGESTION tone.
* Decrease Wait(10) to Wait(5) in tests/channels/SIP/info_dtmf.
* Maintain list of AGI connections where needed so they can all be agi.finish().
* Replace calls to reactor.stop() with self.stop_reactor(), remove test.start_asterisk()/test.stop_asterisk() from main().
* Delay self.stop_reactor() in tests/channels/SIP/sip_tls_call by 2 seconds.  This gives the calls enough time to end and avoid shutdown timeout.
Ran all effected tests against Asterisk 11 with REF_DEBUG.  Prior to these fixes graceful shutdown of Asterisk timed out, causing reference leaks to be reported.  These tests now shutdown gracefully and have no reference leaks.
Review request changed
Updated (Feb. 11, 2015, 11:29 a.m.)
  • changed from pending to submitted
Committed in revision 6388

https://reviewboard.asterisk.org/ runs on a server provided by Digium, Inc. and uses bandwidth donated to the open source Asterisk community by API Digital Communications in Huntsville, AL USA.
Please report problems with this site to asteriskteam@digium.com.