commit 7cd7c2d5c3af74616f77eff155a57b0897eefe35
author Michael Blow <mblow@apache.org> Fri May 27 21:01:50 2016 -0400
committer Michael Blow <michael.blow@couchbase.com> Fri May 27 21:11:34 2016 -0700
tree 5575d78cd43f5f009b8d1e0d9025c2f08890fd44
parent 930bc555cecd4ef874bf03b59f25e68519aa81a0

Fix Coverage Report

Fix the coverage report generated by jacoco plugin, broken by the
great merge, to again include hyracks classes / coverage.

Change-Id: I023e4293954aef291358190e20ef3c5225fe87a6
Reviewed-on: https://asterix-gerrit.ics.uci.edu/893
Reviewed-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: Till Westmann <tillw@apache.org>

asterixdb/asterix-coverage/pom.xml

diff --git a/asterixdb/asterix-coverage/pom.xml b/asterixdb/asterix-coverage/pom.xml
index 3692fb2..f71f3a2 100644
--- a/asterixdb/asterix-coverage/pom.xml
+++ b/asterixdb/asterix-coverage/pom.xml
@@ -52,35 +52,17 @@
                 <configuration>
                   <target if="coverage" unless="skipTests">
                     <echo message="Generating JaCoCo Reports" />
-                    <property name="asterix.dir" location="${basedir}/.."/>
-                    <!-- special case for jenkins -->
-                    <condition property="hyracks.dir" value="${asterix.dir}/hyracks">
-                      <available file="${asterix.dir}/hyracks/pom.xml"/>
-                    </condition>
-                    <pathconvert property="hyracks.dir">
-                      <first>
-                        <dirset dir="${asterix.dir}/..">
-                          <include name="*hyracks"/>
-                        </dirset>
-                      </first>
-                    </pathconvert>
+                    <property name="asterix.dir" location="${basedir}/../.."/>
+
                     <!-- gather all of the source dirs -->
                     <pathconvert property="source.dirs" pathsep=",">
                       <dirset dir="${asterix.dir}">
                         <include name="**/src/main/java"/>
-                        <!-- special case for jenkins -->
-                        <exclude name="hyracks/**"/>
                       </dirset>
                     </pathconvert>
-                    <pathconvert property="hyracks.source.dirs" pathsep=",">
-                      <dirset dir="${hyracks.dir}">
-                        <include name="**/src/main/java"/>
-                      </dirset>
-                    </pathconvert>
+
                     <echo message="asterix dir: ${asterix.dir}/"/>
-                    <echo message="hyracks dir: ${hyracks.dir}"/>
                     <echo message="asterix source dirs: ${source.dirs}/"/>
-                    <echo message="hyracks source dirs: ${hyracks.source.dirs}/"/>
 
                     <taskdef name="report" classname="org.jacoco.ant.ReportTask">
                       <classpath path="${basedir}/target/jacoco-jars/org.jacoco.ant.jar" />
@@ -97,8 +79,7 @@
                           <classfiles>
                             <fileset dir="${asterix.dir}">
                               <include name="**/target/classes/**/*.class"/>
-                              <!-- special case for jenkins -->
-                              <exclude name="hyracks/**"/>
+                              <exclude name="**/hyracks-hdfs/hyracks-hdfs-1.x/**"/>
                             </fileset>
                           </classfiles>
                           <sourcefiles encoding="UTF-8">
@@ -107,19 +88,6 @@
                             </multirootfileset>
                           </sourcefiles>
                         </group>
-                        <group name="Hyracks">
-                          <classfiles>
-                            <fileset dir="${hyracks.dir}">
-                              <include name="**/target/classes/**/*.class"/>
-                              <exclude name="hyracks/hyracks-hdfs/hyracks-hdfs-1.x/**"/>
-                            </fileset>
-                          </classfiles>
-                          <sourcefiles encoding="UTF-8">
-                            <multirootfileset basedirs="${hyracks.source.dirs}" type="file">
-                              <include name="**/*.java"/>
-                            </multirootfileset>
-                          </sourcefiles>
-                        </group>
                       </structure>
                       <html destdir="${basedir}/target/coverage-report/html" />
                       <xml destfile="${basedir}/target/coverage-report/coverage-report.xml" />