<div dir="ltr"><div class="gmail_default" style="color:#660000">I was worried that might be a problem.  What about using an environment variable as the trigger instead?  That would allow it to be easily inherited by any language, and easily ignored if not implemented.</div>
<div class="gmail_default" style="color:#660000"><br></div><div class="gmail_default" style="color:#660000">You could even set it externally to runtests.py at first for testing or automation:</div><div class="gmail_default" style="color:#660000">
<br></div><div class="gmail_default" style="color:#660000">export RUNTEST_VALGRIND='YES'</div><div class="gmail_default" style="color:#660000">./runtests.py</div><div class="gmail_default" style="color:#660000"><br>
</div><div class="gmail_default" style="color:#660000"><br></div><div class="gmail_default" style="color:#660000"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 23, 2013 at 8:19 AM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank">mjordan@digium.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"><br><div class="gmail_extra"><div class="gmail_quote"><div class="im">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><div>There's really two ways tests are executed:</div><div>

<br></div><div>(1) In the manner you just described, where a process is spawned that executes a run-test script</div><div>(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><br></div><div>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><br></div><div>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>Matt</div><div><br></div>-- <br><div dir="ltr"><div class="im"><div>Matthew Jordan<br></div><div>Digium, Inc. | Engineering Manager</div><div>445 Jan Davis Drive NW - Huntsville, AL 35806 - USA</div>

</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>
<br>--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">
<img alt="Digium logo" src="https://my.digium.com/images/graphics/digium_RGB_signature.gif" width="288" height="50" style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px"><br style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">
<strong style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">Scott Griepentrog</strong><br style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px"><span style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">Digium, Inc · Software Developer</span><br style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">
<span style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">445 Jan Davis Drive NW · Huntsville, AL 35806 · US</span><br style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">
<span style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">direct/fax: +1 256 428 6239 · mobile: +1 317 507 4029</span><br style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">
<span style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px">Check us out at: </span><a href="http://www.digium.com" style="font-family:Arial,Helvetica,sans-serif;font-size:12px" target="_blank">http://digium.com</a><span style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px"> · </span><a href="http://www.asterisk.org" style="font-family:Arial,Helvetica,sans-serif;font-size:12px" target="_blank">http://asterisk.org</a><br>
</div>
</div>