]> xenbits.xensource.com Git - libvirt.git/commitdiff
Switch to using lcov for code coverage repots
authorDaniel P. Berrange <berrange@redhat.com>
Thu, 29 May 2008 19:27:04 +0000 (19:27 +0000)
committerDaniel P. Berrange <berrange@redhat.com>
Thu, 29 May 2008 19:27:04 +0000 (19:27 +0000)
ChangeLog
Makefile.am
configure.in
scripts/.cvsignore [deleted file]
scripts/Makefile.am [deleted file]
scripts/README [deleted file]
scripts/coverage-report-entry.pl [deleted file]
scripts/coverage-report.pl [deleted file]
scripts/coverage-report.xsl [deleted file]
src/Makefile.am

index b36217f9fc0ea51841078e9ea555462404bff8e7..530af890cefea6a64974b96a0b67e9b4e6fd0515 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+Thu May 29 15:25:00 EST 2008 Daniel P. Berrange <berrange@redhat.com>
+
+       * configure.in, scripts/*: Remove all coverage helper scripts
+       in favour of using lcov
+       * src/Makefile.am, Makefile.am: Switch to use lcov for pretty
+       code coverage reports
+
 Thu May 29 15:17:00 EST 2008 Daniel P. Berrange <berrange@redhat.com>
 
        * src/libvirt.c: Don't register storage API if daemon is
index 73552a48756f6a9219913938aef7aefb9a534ffb..e0aba2daa78806841ce697430e6e29e1ade8b16e 100644 (file)
@@ -1,7 +1,10 @@
 ## Process this file with automake to produce Makefile.in
 
+LCOV = lcov
+GENHTML = genhtml
+
 SUBDIRS = gnulib/lib include src qemud proxy docs gnulib/tests \
-  python tests po scripts
+  python tests po
 
 ACLOCAL_AMFLAGS = -I m4 -I gnulib/m4
 
@@ -28,24 +31,15 @@ tests:
        @(if [ "$(pythondir)" != "" ] ; then cd python ; \
          $(MAKE) MAKEFLAGS+=--silent tests ; fi)
 
-cov: cov-recursive cov-am
+cov: clean-cov
+       mkdir $(top_builddir)/coverage
+       $(LCOV) -c -o $(top_builddir)/coverage/libvirt.info.tmp -d $(top_srcdir)/src  -d $(top_srcdir)/qemud -d $(top_srcdir)/tests
+       $(LCOV) -r $(top_builddir)/coverage/libvirt.info.tmp -o $(top_builddir)/coverage/libvirt.info *usr*
+       rm $(top_builddir)/coverage/libvirt.info.tmp
+       $(GENHTML) -s -t "libvirt" -o $(top_builddir)/coverage --legend $(top_builddir)/coverage/libvirt.info
 
 clean-cov:
        rm -rf $(top_builddir)/coverage
-       cd src && $(MAKE) $(AM_MAKEFLAGS) clean-cov
-
-cov-recursive:
-       cd src && $(MAKE) $(AM_MAKEFLAGS) cov
-
-cov-am:
-       rm -rf $(top_builddir)/coverage
-       mkdir $(top_builddir)/coverage
-       perl $(srcdir)/scripts/coverage-report.pl src/*.cov > $(top_builddir)/coverage/index.xml
-       xsltproc $(srcdir)/scripts/coverage-report.xsl \
-         $(top_builddir)/coverage/index.xml \
-         > $(top_builddir)/coverage/index.html
-       for i in $(top_builddir)/src/*.gcov ; do o=`echo $$i | sed -e 's,$(top_builddir)/src,coverage,'` ; \
-         perl $(srcdir)/scripts/coverage-report-entry.pl $$i > $$o.html ; done
 
 # disable this check
 distuninstallcheck:
index 24d8d8629da2d18ebe145fef0219a9d3b8ffb5a4..60233557074fafba68def34b292660ef14c5af1c 100644 (file)
@@ -1001,7 +1001,7 @@ AC_OUTPUT(Makefile src/Makefile include/Makefile docs/Makefile \
          gnulib/lib/Makefile \
          gnulib/tests/Makefile \
           libvirt.pc libvirt.spec \
-          po/Makefile.in scripts/Makefile \
+          po/Makefile.in \
          include/libvirt/Makefile include/libvirt/libvirt.h \
          python/Makefile python/tests/Makefile \
           qemud/Makefile \
diff --git a/scripts/.cvsignore b/scripts/.cvsignore
deleted file mode 100644 (file)
index 282522d..0000000
+++ /dev/null
@@ -1,2 +0,0 @@
-Makefile
-Makefile.in
diff --git a/scripts/Makefile.am b/scripts/Makefile.am
deleted file mode 100644 (file)
index 7078fe8..0000000
+++ /dev/null
@@ -1,4 +0,0 @@
-
-EXTRA_DIST = coverage-report.pl \
-             coverage-report-entry.pl \
-             coverage-report.xsl
diff --git a/scripts/README b/scripts/README
deleted file mode 100644 (file)
index fdbf957..0000000
+++ /dev/null
@@ -1,3 +0,0 @@
-This directory provides a collection of tools used in the
-build / test process. They are not installed / used after
-deployment.
diff --git a/scripts/coverage-report-entry.pl b/scripts/coverage-report-entry.pl
deleted file mode 100644 (file)
index c3f0255..0000000
+++ /dev/null
@@ -1,77 +0,0 @@
-#!/usr/bin/perl
-#
-# Copyright (C) 2006-2007 Daniel P. Berrange
-#
-# This library is free software; you can redistribute it and/or
-# modify it under the terms of the GNU Lesser General Public
-# License as published by the Free Software Foundation; either
-# version 2.1 of the License, or (at your option) any later version.
-#
-# This library is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-# Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU Lesser General Public
-# License along with this library; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307  USA
-#
-# Author: Daniel P. Berrange <berrange@redhat.com>
-#
-# coverage-report-entry.pl: convert gcov annotated source into HTML
-#
-# This script takes a gcov annotated source code files on STDIN
-# converts it to HTML, coloured according to coverage, and sends
-# it to STDOUT
-
-print <<EOF;
-<html>
-<head>
-<title>Coverage report for $ARGV[0]</title>
-<style type="text/css">
-          span.perfect {
-            background: rgb(0,255,0);
-          }
-          span.terrible {
-            background: rgb(255,0,0);
-          }
-</style>
-</head>
-<body>
-<h1>Coverage report for $ARGV[0]</h1>
-
-<pre>
-EOF
-
-
-while (<>) {
-    s/&/&amp;/g;
-    s/</&lt;/g;
-    s/>/&gt;/g;
-
-    if (/^\s*function (\S+) called (\d+) returned \d+% blocks executed \d+%/) {
-       my $class = $2 > 0 ? "perfect" : "terrible";
-       $_ = "<span class=\"$class\" id=\"" . $1 . "\">$_</span>";
-    } elsif (/^\s*branch\s+\d+\s+taken\s+(\d+)%\s+.*$/) {
-       my $class = $1 > 0 ? "perfect" : "terrible";
-       $_ = "<span class=\"$class\">$_</span>";
-    } elsif (/^\s*branch\s+\d+\s+never executed.*$/) {
-       my $class = "terrible";
-       $_ = "<span class=\"$class\">$_</span>";
-    } elsif (/^\s*call\s+\d+\s+never executed.*$/) {
-       my $class = "terrible";
-       $_ = "<span class=\"$class\">$_</span>";
-    } elsif (/^\s*call\s+\d+\s+returned\s+(\d+)%.*$/) {
-       my $class = $1 > 0 ? "perfect" : "terrible";
-       $_ = "<span class=\"$class\">$_</span>";
-    }
-
-
-    print;
-}
-
-print <<EOF;
-</pre>
-</body>
-</html>
-EOF
diff --git a/scripts/coverage-report.pl b/scripts/coverage-report.pl
deleted file mode 100644 (file)
index 1001aa4..0000000
+++ /dev/null
@@ -1,164 +0,0 @@
-#!/usr/bin/perl
-#
-# Copyright (C) 2006-2007 Daniel P. Berrange
-#
-# This library is free software; you can redistribute it and/or
-# modify it under the terms of the GNU Lesser General Public
-# License as published by the Free Software Foundation; either
-# version 2.1 of the License, or (at your option) any later version.
-#
-# This library is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-# Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU Lesser General Public
-# License along with this library; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307  USA
-#
-# Author: Daniel P. Berrange <berrange@redhat.com>
-#
-# coverage-report.pl: generate XML coverage summary report
-#
-# This script takes a listof gcov .cov files as args, and generates
-# an XML document summarizing the coverage per function and per
-# source file.
-
-use warnings;
-use strict;
-
-my %coverage = ( function => {}, file => {} );
-
-my @functionBlackList = (
-                        "__memcpy",
-                        "__memmove",
-                        "__memset",
-                        "__strcat",
-                        "__strcpy",
-                        "__strncpy",
-                        "__strsep",
-                        "__strtok"
-                     );
-
-my %filemap;
-
-my $type;
-my $name;
-
-my @functions;
-
-while (<>) {
-    if (/^Function '(.*)'\s*$/) {
-       $type = "function";
-       $name = $1;
-       $coverage{$type}->{$name} = {};
-       push @functions, $name;
-    } elsif (/^File '(.*?)'\s*$/) {
-       $type = "file";
-       $name = $1;
-       $coverage{$type}->{$name} = {};
-
-       foreach my $func (@functions) {
-           $coverage{"function"}->{$func}->{file} = $name;
-       }
-       @functions = ();
-    } elsif (/^Lines executed:(.*)%\s*of\s*(\d+)\s*$/) {
-       $coverage{$type}->{$name}->{lines} = $2;
-       $coverage{$type}->{$name}->{linesCoverage} = $1;
-    } elsif (/^Branches executed:(.*)%\s*of\s*(\d+)\s*$/) {
-       $coverage{$type}->{$name}->{branches} = $2;
-       $coverage{$type}->{$name}->{branchesCoverage} = $1;
-    } elsif (/^Taken at least once:(.*)%\s*of\s*(\d+)\s*$/) {
-       $coverage{$type}->{$name}->{conds} = $2;
-       $coverage{$type}->{$name}->{condsCoverage} = $1;
-    } elsif (/^Calls executed:(.*)%\s*of\s*(\d+)\s*$/) {
-       $coverage{$type}->{$name}->{calls} = $2;
-       $coverage{$type}->{$name}->{callsCoverage} = $1;
-    } elsif (/^No branches$/) {
-       $coverage{$type}->{$name}->{branches} = 0;
-       $coverage{$type}->{$name}->{branchesCoverage} = "100.00";
-       $coverage{$type}->{$name}->{conds} = 0;
-       $coverage{$type}->{$name}->{condsCoverage} = "100.00";
-    } elsif (/^No calls$/) {
-       $coverage{$type}->{$name}->{calls} = 0;
-       $coverage{$type}->{$name}->{callsCoverage} = "100.00";
-    } elsif (/^\s*(.*):creating '(.*)'\s*$/) {
-       $filemap{$1} = $2;
-    } elsif (/^\s*$/) {
-       # nada
-    } else {
-       warn "unexpected input [$_]\n";
-    }
-}
-
-my %summary;
-foreach my $type ("function", "file") {
-    $summary{$type} = {};
-    foreach my $m ("lines", "branches", "conds", "calls") {
-       my $totalGot = 0;
-       my $totalMiss = 0;
-       my $count = 0;
-       foreach my $func (keys %{$coverage{function}}) {
-           my $blacklisted = 0;
-           foreach my $blackName (@functionBlackList) {
-               $blacklisted = 1 if $func =~ /^$blackName/;
-           }
-           next if $blacklisted;
-
-           $count++;
-           my $got = $coverage{function}->{$func}->{$m};
-           $totalGot += $got;
-           my $miss = $got * $coverage{function}->{$func}->{$m ."Coverage"} / 100;
-           $totalMiss += $miss;
-       }
-       $summary{$type}->{$m} = sprintf("%d", $totalGot);
-       if ($totalGot == 0) {
-           $summary{$type}->{$m . "Coverage"} = "100.00";
-       } else {
-           $summary{$type}->{$m . "Coverage"} = sprintf("%.2f", $totalMiss / $totalGot * 100);
-       }
-    }
-}
-
-
-
-print "<coverage>\n";
-
-foreach my $type ("function", "file") {
-    printf "<%ss>\n", $type;
-    foreach my $name (sort { $a cmp $b } keys %{$coverage{$type}}) {
-       if ($type eq "file") {
-           next if $name =~ m,^/usr,;
-       } else {
-           my $blacklisted = 0;
-           foreach my $blackName (@functionBlackList) {
-               $blacklisted = 1 if $name =~ /^$blackName/;
-           }
-           next if $blacklisted;
-       }
-
-       my $rec = $coverage{$type}->{$name};
-       printf "  <entry name=\"%s\" details=\"%s\">\n", $name, ($type eq "file" ? $filemap{$name} : $filemap{$rec->{file}});
-       printf "    <lines count=\"%s\" coverage=\"%s\"/>\n", $rec->{lines}, $rec->{linesCoverage};
-       if (exists $rec->{branches}) {
-           printf "    <branches count=\"%s\" coverage=\"%s\"/>\n", $rec->{branches}, $rec->{branchesCoverage};
-       }
-       if (exists $rec->{conds}) {
-           printf "    <conditions count=\"%s\" coverage=\"%s\"/>\n", $rec->{conds}, $rec->{condsCoverage};
-       }
-       if (exists $rec->{calls}) {
-           printf "    <calls count=\"%s\" coverage=\"%s\"/>\n", $rec->{calls}, $rec->{callsCoverage};
-       }
-       print  "  </entry>\n";
-    }
-
-    printf "  <summary>\n";
-    printf "    <lines count=\"%s\" coverage=\"%s\"/>\n", $summary{$type}->{lines}, $summary{$type}->{linesCoverage};
-    printf "    <branches count=\"%s\" coverage=\"%s\"/>\n", $summary{$type}->{branches}, $summary{$type}->{branchesCoverage};
-    printf "    <conditions count=\"%s\" coverage=\"%s\"/>\n", $summary{$type}->{conds}, $summary{$type}->{condsCoverage};
-    printf "    <calls count=\"%s\" coverage=\"%s\"/>\n", $summary{$type}->{calls}, $summary{$type}->{callsCoverage};
-    printf  "  </summary>\n";
-    printf "</%ss>\n", $type;
-}
-
-print "</coverage>\n";
diff --git a/scripts/coverage-report.xsl b/scripts/coverage-report.xsl
deleted file mode 100644 (file)
index 6775035..0000000
+++ /dev/null
@@ -1,217 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>\r
-<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform"\r
-                version="1.0">\r
-\r
-  <xsl:output method="html"/>\r
-\r
-  <xsl:template match="coverage">\r
-    <html>\r
-      <head>\r
-        <title>Coverage report</title>\r
-        <style type="text/css">\r
-          tbody tr.odd td.label {\r
-            border-top: 1px solid rgb(128,128,128);\r
-            border-bottom: 1px solid rgb(128,128,128);\r
-          }\r
-          tbody tr.odd td.label {\r
-            background: rgb(200,200,200);\r
-          }\r
-          \r
-          thead, tfoot {\r
-            background: rgb(60,60,60);\r
-            color: white;\r
-            font-weight: bold;\r
-          }\r
-\r
-          tr td.perfect {\r
-            background: rgb(0,255,0);\r
-            color: black;\r
-          }\r
-          tr td.excellant {\r
-            background: rgb(140,255,140);\r
-            color: black;\r
-          }\r
-          tr td.good {\r
-            background: rgb(160,255,0);\r
-            color: black;\r
-          }\r
-          tr td.poor {\r
-            background: rgb(255,160,0);\r
-            color: black;\r
-          }\r
-          tr td.bad {\r
-            background: rgb(255,140,140);\r
-            color: black;\r
-          }\r
-          tr td.terrible {\r
-            background: rgb(255,0,0);\r
-            color: black;\r
-          }\r
-        </style>\r
-      </head>\r
-      <body>\r
-        <h1>Coverage report</h1>\r
-        <xsl:apply-templates/>\r
-      </body>\r
-    </html>\r
-  </xsl:template>\r
-\r
-  <xsl:template match="functions">\r
-    <h2>Function coverage</h2>\r
-    <xsl:call-template name="content">\r
-      <xsl:with-param name="type" select="'function'"/>\r
-    </xsl:call-template>\r
-  </xsl:template>\r
-  \r
-\r
-  <xsl:template match="files">\r
-    <h2>File coverage</h2>\r
-    <xsl:call-template name="content">\r
-      <xsl:with-param name="type" select="'file'"/>\r
-    </xsl:call-template>\r
-  </xsl:template>\r
-\r
-  <xsl:template name="content">\r
-    <xsl:param name="type"/>\r
-    <table>\r
-      <thead>\r
-        <tr>\r
-          <th>Name</th>\r
-          <th>Lines</th>\r
-          <th>Branches</th>\r
-          <th>Conditions</th>\r
-          <th>Calls</th>\r
-        </tr>\r
-      </thead>\r
-      <tbody>\r
-        <xsl:for-each select="entry">\r
-          <xsl:call-template name="entry">\r
-            <xsl:with-param name="type" select="$type"/>\r
-            <xsl:with-param name="class">\r
-              <xsl:choose>\r
-                <xsl:when test="position() mod 2">\r
-                  <xsl:text>odd</xsl:text>\r
-                </xsl:when>\r
-                <xsl:otherwise>\r
-                  <xsl:text>even</xsl:text>\r
-                </xsl:otherwise>\r
-              </xsl:choose>\r
-            </xsl:with-param>\r
-          </xsl:call-template>\r
-        </xsl:for-each>\r
-      </tbody>\r
-      <tfoot>\r
-        <xsl:for-each select="summary">\r
-          <xsl:call-template name="entry">\r
-            <xsl:with-param name="type" select="'summary'"/>\r
-            <xsl:with-param name="class">\r
-              <xsl:choose>\r
-                <xsl:when test="position() mod 2">\r
-                  <xsl:text>odd</xsl:text>\r
-                </xsl:when>\r
-                <xsl:otherwise>\r
-                  <xsl:text>even</xsl:text>\r
-                </xsl:otherwise>\r
-              </xsl:choose>\r
-            </xsl:with-param>\r
-          </xsl:call-template>\r
-        </xsl:for-each>\r
-      </tfoot>\r
-    </table>\r
-  </xsl:template>\r
-  \r
-  <xsl:template name="entry">\r
-    <xsl:param name="type"/>\r
-    <xsl:param name="class"/>\r
-    <tr class="{$class}">\r
-      <xsl:choose>\r
-        <xsl:when test="$type = 'function'">\r
-          <td class="label"><a href="{@details}.html#{@name}"><xsl:value-of select="@name"/></a></td>\r
-        </xsl:when>\r
-        <xsl:when test="$type = 'file'">\r
-          <td class="label"><a href="{@details}.html"><xsl:value-of select="@name"/></a></td>\r
-        </xsl:when>\r
-        <xsl:otherwise>\r
-          <td class="label">Summary</td>\r
-        </xsl:otherwise>\r
-      </xsl:choose>\r
-\r
-      <xsl:if test="count(lines)">\r
-        <xsl:apply-templates select="lines"/>\r
-      </xsl:if>\r
-      <xsl:if test="not(count(lines))">\r
-        <xsl:call-template name="missing"/>\r
-      </xsl:if>\r
-\r
-      <xsl:if test="count(branches)">\r
-        <xsl:apply-templates select="branches"/>\r
-      </xsl:if>\r
-      <xsl:if test="not(count(branches))">\r
-        <xsl:call-template name="missing"/>\r
-      </xsl:if>\r
-\r
-      <xsl:if test="count(conditions)">\r
-        <xsl:apply-templates select="conditions"/>\r
-      </xsl:if>\r
-      <xsl:if test="not(count(conditions))">\r
-        <xsl:call-template name="missing"/>\r
-      </xsl:if>\r
-\r
-      <xsl:if test="count(calls)">\r
-        <xsl:apply-templates select="calls"/>\r
-      </xsl:if>\r
-      <xsl:if test="not(count(calls))">\r
-        <xsl:call-template name="missing"/>\r
-      </xsl:if>\r
-\r
-    </tr>\r
-  </xsl:template>\r
-  \r
-  <xsl:template match="lines">\r
-    <xsl:call-template name="row"/>\r
-  </xsl:template>\r
-\r
-  <xsl:template match="branches">\r
-    <xsl:call-template name="row"/>\r
-  </xsl:template>\r
-\r
-  <xsl:template match="conditions">\r
-    <xsl:call-template name="row"/>\r
-  </xsl:template>\r
-\r
-  <xsl:template match="calls">\r
-    <xsl:call-template name="row"/>\r
-  </xsl:template>\r
-\r
-  <xsl:template name="missing">\r
-    <td></td>\r
-  </xsl:template>\r
-\r
-  <xsl:template name="row">\r
-    <xsl:variable name="quality">\r
-      <xsl:choose>\r
-        <xsl:when test="@coverage = 100">\r
-          <xsl:text>perfect</xsl:text>\r
-        </xsl:when>\r
-        <xsl:when test="@coverage >= 80.0">\r
-          <xsl:text>excellant</xsl:text>\r
-        </xsl:when>\r
-        <xsl:when test="@coverage >= 60.0">\r
-          <xsl:text>good</xsl:text>\r
-        </xsl:when>\r
-        <xsl:when test="@coverage >= 40.0">\r
-          <xsl:text>poor</xsl:text>\r
-        </xsl:when>\r
-        <xsl:when test="@coverage >= 20.0">\r
-          <xsl:text>bad</xsl:text>\r
-        </xsl:when>\r
-        <xsl:otherwise>\r
-          <xsl:text>terrible</xsl:text>\r
-        </xsl:otherwise>\r
-      </xsl:choose>\r
-    </xsl:variable>\r
-    \r
-    <td class="{$quality}"><xsl:value-of select="@coverage"/>% of <xsl:value-of select="@count"/></td>\r
-  </xsl:template>\r
-\r
-</xsl:stylesheet>\r
index 286e10ed0b925dbcea5e1feded356d86f1e306bb..0549bfda97ffa6e03f3ee9d1d31e049d51118611 100644 (file)
@@ -149,22 +149,4 @@ else
 EXTRA_DIST += parthelper.c
 endif
 
-cov: clean-cov
-       for i in $(CLIENT_SOURCES); do                          \
-         case $$i in *.c) ;; *) continue;; esac;               \
-         b=$$(basename $$i .c);                                \
-         o_files=;                                             \
-         for i in '' _test; do                                 \
-           g="$(LV_LIBTOOL_OBJDIR)/libvirt$${i}_la-$$b.gcda";  \
-           o="$(LV_LIBTOOL_OBJDIR)/libvirt$${i}_la-$$b.o";     \
-           test -f "$$o" -a -f "$$g"                           \
-             && o_files="$$o_files $$o";                       \
-         done;                                                 \
-         test -n "$$o_files"                                   \
-              && gcov -o $(LV_LIBTOOL_OBJDIR) -b -f $$o_files > $$b.cov;       \
-       done
-
-clean-cov:
-       rm -f *.cov *.gcov
-
-CLEANFILES = *.cov *.gcov .libs/*.gcda .libs/*.gcno *.gcno *.gcda
+CLEANFILES = *.gcov .libs/*.gcda .libs/*.gcno *.gcno *.gcda