On Mon, Oct 12, 2009 at 7:08 PM, Garrett Cooper <[email protected]> wrote:
> On Fri, Oct 9, 2009 at 3:00 AM, rohit verma <[email protected]> wrote:
>> Hi,
>>
>> Thanks Naresh for pointing out the issue in the HTML report. I had
>> also noticed the issue in my test reports.
>>
>> However, I wish to confirm that the issue being mentioned is not a
>> side-effect of the fix that I had posted for the pan driver.
>> If you compare the HTML reports from the previous releases and the
>> recent one, you can notice all the tags - FAIL, WARN, BROK, PASS,CONF
>> & RETR are now prefixed by ' T ' for reasons not known to me
>> (resulting in - TFAIL, TPASS etc).
>>
>> Now, if you check ltp-full-20090831/tools/genhtml.pl file you can see
>> all that old tags (FAIL, WARN, etc) are used instead of the new
>> ones(tags). As a result the numbers are not updated in the summary in
>> HTML report. Also FAIL/WARN/BROK test-cases are not uniquely colored
>> like in the old versions of LTP.
>>
>> Please check the same.
>> If possible I will generate a patch and submit the same.
>>
>> On Wed, Oct 7, 2009 at 4:43 PM, naresh kamboju <[email protected]>
>> wrote:
>>> Hi,
>>>
>>> I think you have noticed difference in the HTML file generation.
>>> HTML file is not reporting number of failure test cases
>>> in summary and failed column with different color.
>>> Some changes are made by Rohit recently.
>>>
>>> Rohit,
>>> Please investigate the changes made to HTML file generator.
>>>
>>> You can refer the results logs share in LTP release in the below link for
>>> X86
>>> (ltp-full-20090930_results_kernel-2.6.31_architecture-i386.tgz)
>>> Ia64
>>> (ltp-full-20090930_results_kernel-2.6.23.13_architecture-ia64.tgz)
>
> Try this.
>
> Fix genhtml.pl so that it functions with the new T-prefixing in the
> lib/tst_res.c result codes.
>
> Signed-off-by: Garrett Cooper <[email protected]>
>
> Index: contrib/ltp/src/tools/genhtml.pl
> ===================================================================
> --- src/tools/.CC/cache/genhtml.pl@@/main/ltp-upgrade/1 2009-10-12
> 19:08:28.000000000 -0700
> +++ src/tools/genhtml.pl 2009-10-12 19:08:18.000000000 -0700
> @@ -179,31 +179,31 @@
> }
> if ( $flag2 == 1 ) {
> $row_line = $row_line . "$line \n";
> - if ($line =~ /\ FAIL\ / ) {
> + if ($line =~ /\ TFAIL\ / ) {
> $detected_fail = 1;
> if ( $failed_test_counter_flag == 0 ) {
> $failed_test_counter++;
> $failed_test_counter_flag=1;
> }
> - } elsif ($line =~ /\ BROK\ / ) {
> + } elsif ($line =~ /\ TBROK\ / ) {
> $detected_brok = 1;
> if ( $brok_test_counter_flag == 0 ) {
> $brok_test_counter++;
> $brok_test_counter_flag=1;
> }
> - } elsif ($line =~ /\ WARN\ / ) {
> + } elsif ($line =~ /\ TWARN\ / ) {
> $detected_warn = 1;
> if ( $warn_test_counter_flag == 0 ) {
> $warn_test_counter++;
> $warn_test_counter_flag=1;
> }
> - } elsif ($line =~ /\ RETR\ / ) {
> + } elsif ($line =~ /\ TRETR\ / ) {
> $detected_retr = 1;
> if ( $retr_test_counter_flag == 0 ) {
> $retr_test_counter++;
> $retr_test_counter_flag=1;
> }
> - } elsif ($line =~ /\ CONF\ / ) {
> + } elsif ($line =~ /\ TCONF\ / ) {
> $detected_conf = 1;
> if ( $conf_test_counter_flag == 0 ) {
> $conf_test_counter++;
> @@ -251,3 +251,5 @@
> print "<tr><td><strong>Hostname</strong> </td> <td><strong>";
> $hostname=system("uname -n"); chop($hostname);
> print " $hostname </strong></td></tr></tbody></table></div></body></html>\n";
> +
> +# vim: syntax=perl
This was the checkin that changed the output format, BTW:
Cheers,
-Garrett
----------------------------------------------------------------------------------------------------------
Update of /cvsroot/ltp/ltp/lib
In directory ddv4jf1.ch3.sourceforge.com:/tmp/cvs-serv24617/lib
Modified Files:
tst_res.c
Log Message:
extend the test result to a bit field so we can extend the output further
Index: tst_res.c
===================================================================
RCS file: /cvsroot/ltp/ltp/lib/tst_res.c,v
retrieving revision 1.8
retrieving revision 1.9
diff -C2 -d -r1.8 -r1.9
*** tst_res.c 9 Jun 2009 16:01:20 -0000 1.8
--- tst_res.c 20 Jul 2009 02:42:32 -0000 1.9
***************
*** 116,119 ****
--- 116,120 ----
#include <unistd.h>
#include "test.h"
+ #include "usctest.h"
------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
