<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Wed, Oct 23, 2013 at 3:40 AM, nitesh bansal <span dir="ltr"><<a href="mailto:nitesh.bansal@gmail.com" target="_blank">nitesh.bansal@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Scott,<div><br></div><div>Command line option is passed to the master script runtests.py and asterisk is spawned from the child script run-test, presently there isn't anyway of passing the argument from master script to</div>

<div>child script. I'll try and figure out the best way to share data between these two scripts?</div><div><br></div></div></blockquote><div><br></div><div style>There's really two ways tests are executed:</div><div style>
<br></div><div style>(1) In the manner you just described, where a process is spawned that executes a run-test script</div><div style>(2) Using the pluggable module framework. This will execute TestRunner, which will - based on the YAML - construct the objects in the test and run them</div>
<div style><br></div><div style>For tests that use the second approach, passing a new command line parameter is trivial - the only thing that gets updated is TestRunner. However, the first approach is problematic. As Paul's commit that removed valgrind noted, the problem with command line parameters is that they impact every run-test in the testsuite - which can also cross language boundaries as well (there are still some lua tests in the testsuite).</div>
<div style><br></div><div style>I'd recommend against command line parameters for that reason. If you make the usage of valgrind configured via the YAML, then - for the most part - all you'll have to do is update TestConfig to parse the option; TestCase to make use of the option and pass it to Asterisk (the library); and Asterisk (library) to spawn valgrind with Asterisk as opposed to Asterisk directly. That will capture a good 80% of the tests in the TestSuite, without having to modify any of them.</div>
<div><br></div></div><div style>Matt</div><div style><br></div>-- <br><div dir="ltr"><div>Matthew Jordan<br></div><div>Digium, Inc. | Engineering Manager</div><div>445 Jan Davis Drive NW - Huntsville, AL 35806 - USA</div>
<div>Check us out at: <a href="http://digium.com" target="_blank">http://digium.com</a> & <a href="http://asterisk.org" target="_blank">http://asterisk.org</a></div></div>
</div></div>