<p>Torrey Searle has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/9379">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">t38_fast_reject: remove dependency on rtpdump<br><br>removed dependency on rtpdump, added executable permissions to<br>perl script validating response time<br><br>Change-Id: Ia73ef502f72bd75522d4b6656f6070807dbef349<br>---<br>M tests/fax/pjsip/t38_fast_reject/check_reinvite_rtt.pl<br>M tests/fax/pjsip/t38_fast_reject/run-test<br>M tests/fax/pjsip/t38_fast_reject/test-config.yaml<br>3 files changed, 1 insertion(+), 22 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/79/9379/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/fax/pjsip/t38_fast_reject/check_reinvite_rtt.pl b/tests/fax/pjsip/t38_fast_reject/check_reinvite_rtt.pl</span><br><span>old mode 100644</span><br><span>new mode 100755</span><br><span>diff --git a/tests/fax/pjsip/t38_fast_reject/run-test b/tests/fax/pjsip/t38_fast_reject/run-test</span><br><span>index 9d86e01..d20b6af 100755</span><br><span>--- a/tests/fax/pjsip/t38_fast_reject/run-test</span><br><span>+++ b/tests/fax/pjsip/t38_fast_reject/run-test</span><br><span>@@ -49,6 +49,7 @@</span><br><span>         'scenario' : 'A_PARTY.xml',</span><br><span>         '-i' : '127.0.0.1',</span><br><span>         '-p' : '5061',</span><br><span style="color: hsl(120, 100%, 40%);">+        '-m' : '1',</span><br><span>         '-s' : e164,</span><br><span>         '-message_file' : sippA_logfile,</span><br><span>         '-error_file' : sippA_errfile,</span><br><span>@@ -62,29 +63,8 @@</span><br><span>     test = SIPpTest(WORKING_DIR, TEST_DIR, SIPP_SCENARIOS)</span><br><span>     test.reactor_timeout = 100;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-    dump_A = WORKING_DIR + "/carrier_rtp.log"</span><br><span style="color: hsl(0, 100%, 40%);">-    dump_B = WORKING_DIR + "/customer_rtp.log"</span><br><span style="color: hsl(0, 100%, 40%);">-    rtpdump_A = subprocess.Popen(["rtpdump", "-t","5", "-F","ascii","-d","101","-o",dump_A, "127.0.0.1/9000"])</span><br><span style="color: hsl(0, 100%, 40%);">-    rtpdump_B = subprocess.Popen(["rtpdump", "-t","5", "-F","ascii","-d","101","-o",dump_B, "127.0.0.1/8000"])</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-    time.sleep(10) #Wait 10 seconds to ensure that all the sockets are open before running the test     </span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span>     reactor.run()</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-    # Kill the RTPDUMP, pass it the signal"</span><br><span style="color: hsl(0, 100%, 40%);">-    rtpdump_A.send_signal(signal.SIGINT)       </span><br><span style="color: hsl(0, 100%, 40%);">-    rtpdump_A.wait()</span><br><span style="color: hsl(0, 100%, 40%);">-    rtpdump_B.send_signal(signal.SIGINT)    </span><br><span style="color: hsl(0, 100%, 40%);">-    rtpdump_B.wait()</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-    #Verify that audio packets were routed in both directions after G711 fallback</span><br><span style="color: hsl(0, 100%, 40%);">-    if (os.path.getsize(dump_B) == 0):</span><br><span style="color: hsl(0, 100%, 40%);">-     logger.error("No RTP routed towards customer ...failing the test")</span><br><span style="color: hsl(0, 100%, 40%);">-    return 1</span><br><span style="color: hsl(0, 100%, 40%);">-    if (os.path.getsize(dump_A) == 0):</span><br><span style="color: hsl(0, 100%, 40%);">-          logger.error("No RTP routed towards carrier ...failing the test")</span><br><span style="color: hsl(0, 100%, 40%);">-     return 1</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span>     if not test.passed:</span><br><span>         return 1</span><br><span> </span><br><span>diff --git a/tests/fax/pjsip/t38_fast_reject/test-config.yaml b/tests/fax/pjsip/t38_fast_reject/test-config.yaml</span><br><span>index cf74af8..90eddc0 100644</span><br><span>--- a/tests/fax/pjsip/t38_fast_reject/test-config.yaml</span><br><span>+++ b/tests/fax/pjsip/t38_fast_reject/test-config.yaml</span><br><span>@@ -7,7 +7,6 @@</span><br><span>     dependencies:</span><br><span>         - python : 'twisted'</span><br><span>         - python : 'starpy'</span><br><span style="color: hsl(0, 100%, 40%);">-        - app    : 'rtpdump'</span><br><span> </span><br><span>     tags:</span><br><span>         - PJSIP</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/9379">change 9379</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/9379"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ia73ef502f72bd75522d4b6656f6070807dbef349 </div>
<div style="display:none"> Gerrit-Change-Number: 9379 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Torrey Searle <tsearle@gmail.com> </div>