Mercurial > hg > release > thermostat-0.5

changeset 869:cf634ed45556

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Fix coverage reports Coverage report generation relies on using the argLine property. This property normally contains ${coverageAgent} which sets up the jacoco agent. A few poms specify their own argLine but do not use ${coverageAgent} which results in the jacoco agent not being set up properly and no coverage results. Reviewed-by: neugens Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2012-December/004848.html
author Omair Majid <omajid@redhat.com>
date Mon, 17 Dec 2012 15:26:45 -0500
parents 4ebc8c2c19fb
children b7125452697f
files agent/core/pom.xml keyring/pom.xml system-backend/pom.xml
diffstat 3 files changed, 3 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/agent/core/pom.xml	Mon Dec 17 14:05:26 2012 -0500
+++ b/agent/core/pom.xml	Mon Dec 17 15:26:45 2012 -0500
@@ -114,7 +114,7 @@
         <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-surefire-plugin</artifactId>
           <configuration>
-            <argLine>-Djava.library.path=${project.build.directory}/</argLine>
+            <argLine>-Djava.library.path=${project.build.directory}/ ${coverageAgent}</argLine>
           </configuration>
       </plugin>
       
--- a/keyring/pom.xml	Mon Dec 17 14:05:26 2012 -0500
+++ b/keyring/pom.xml	Mon Dec 17 15:26:45 2012 -0500
@@ -37,7 +37,7 @@
             <systemPropertyVariables>
               <com.redhat.thermostat.utils.keyring.provider>com.redhat.thermostat.utils.keyring.MemoryKeyring</com.redhat.thermostat.utils.keyring.provider>
             </systemPropertyVariables>
-            <argLine>-Djava.library.path=${project.build.directory}/</argLine>
+            <argLine>-Djava.library.path=${project.build.directory}/ ${coverageAgent}</argLine>
           </configuration>
       </plugin>
       
--- a/system-backend/pom.xml	Mon Dec 17 14:05:26 2012 -0500
+++ b/system-backend/pom.xml	Mon Dec 17 15:26:45 2012 -0500
@@ -113,7 +113,7 @@
           <artifactId>maven-surefire-plugin</artifactId>
           <configuration>
             <!-- FIXME: the test need to link to agent-core native libraries -->
-            <argLine>-Djava.library.path=${project.build.directory}/../../agent/core/target/</argLine>
+            <argLine>-Djava.library.path=${project.build.directory}/../../agent/core/target/ ${coverageAgent}</argLine>
           </configuration>
       </plugin>