<div dir="ltr">Hi Cyril,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 18, 2016 at 12:39 AM, Cyril Hrubis <span dir="ltr"><<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
<div><div class="h5">...<br>
<br>
</div></div>Adding a script that runs all testcases is frowned upon in LTP. What you<br>
should do instead is to make it possible to run individual testcases and<br>
add the tests one test per line to a runtest file.<br>
<br></blockquote><div><br></div><div>Seems like these regression cases are petty and simple. I fear that would be too dispersed to read. How about taking use of the cgroup tests code-style to organise them?<br><br>which will be like:<br>---------------------<br>$ cat ftrace_regression_test.sh <br>#! /bin/sh<br>...<br><br>export TCID="ftrace-regression-test"<br>export TST_TOTAL=N ( N depends on how many test_$cur)<br>export TST_COUNT=1<br><br>. ftrace_lib.sh<br><br>test_success=false<br><br>test_check()<br>{<br>    if $test_success; then<br>        tst_resm TPASS "finished running the test."<br>    else<br>        tst_resm TFAIL "running the test failed, please check log message."<br>    fi<br><br>    test_success=false<br>}<br><br>#comments the test_1 info blablabla....<br>test_1()<br>{<br>   ...<br>   set test_success 'true' or 'false' base on the test result<br>   ...<br>   test_check;<br>}<br><br>test_2()<br>{<br>   ...<br>   test_check;<br>}<br><br>test_3()<br>{<br>   ...<br>   test_check;<br>}<br><br>test_regression()<br>{<br>    for cur in $(seq $TST_TOTAL); do<br>        test_$cur<br>    done<br>}<br><br>#----------------------------------<br>echo "Ftrace Regression Test Begin"<br><br>save_old_setting<br><br>test_regression<br><br>clean_up<br><br>echo "Ftrace Regression Test End"<br clear="all"><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Also newly added test should use test.sh library.<br></blockquote><div><br>sure, this no problem! <br></div></div><br>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Regards,<br></div>Li Wang<br></div><div>Email: <a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a><br></div></div></div></div></div></div>
</div></div></div>