Fix NPE during report aggregation when multiple modules are specified using -pl arg (#104)

Co-authored-by: Eugene Platonov <[email protected]>

Author: sumeetgajjar

README.md

@@ -158,7 +158,8 @@ or
 There is no separate mojo for aggregated reports, there is `aggregate` parameter.
 To additionally generate aggregated SCoverage report for root module, when generating regular reports,
 set `aggregate` parameter value to `true`.
-It works only in multimodule projects.
+It works only in multimodule projects, the aggregated report will be generated in the current
+execution root.
 
 It can be configured by defining `aggregate` plugin configuration parameter or `scoverage.aggregate` project property.
 

src/it/test_report_for_some_submodules/invoker.properties

@@ -0,0 +1 @@
+invoker.goals=clean scoverage:report -e -ntp -pl module01 -pl module02/submodule02_02

src/it/test_report_for_some_submodules/module01/pom.xml

@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+
+    <parent>
+        <groupId>it.scoverage-maven-plugin</groupId>
+        <artifactId>test_report_for_some_submodules</artifactId>
+        <version>1.0-SNAPSHOT</version>
+        <relativePath>../pom.xml</relativePath>
+    </parent>
+
+    <modelVersion>4.0.0</modelVersion>
+    <artifactId>module01</artifactId>
+    <name>Test Scoverage report works for specified submodules: Module 1</name>
+
+</project>

src/it/test_report_for_some_submodules/module01/src/main/scala/pkg01/HelloService1.scala

@@ -0,0 +1,12 @@
+package pkg01
+
+class HelloService1
+{
+  def hello =
+  {
+    "Hello from module 1"
+  }
+
+}
+
+object HelloService1 extends HelloService1

src/it/test_report_for_some_submodules/module01/src/test/scala/pkg01/HelloServiceTest.scala

@@ -0,0 +1,14 @@
+package pkg01
+
+import org.junit.Test;
+import org.junit.Assert.assertEquals
+
+class HelloServiceTest
+{
+    @Test
+    def test1()
+    {
+        assertEquals("Hello from module 1", HelloService1.hello)
+    }
+
+}

src/it/test_report_for_some_submodules/module02/pom.xml

@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+
+    <parent>
+        <groupId>it.scoverage-maven-plugin</groupId>
+        <artifactId>test_report_for_some_submodules</artifactId>
+        <version>1.0-SNAPSHOT</version>
+        <relativePath>../pom.xml</relativePath>
+    </parent>
+
+    <modelVersion>4.0.0</modelVersion>
+    <artifactId>module02</artifactId>
+    <packaging>pom</packaging>
+    <name>Test Scoverage report works for specified submodules: Module 2</name>
+
+    <modules>
+        <module>submodule02_01</module>
+        <module>submodule02_02</module>
+    </modules>
+
+
+</project>

src/it/test_report_for_some_submodules/module02/submodule02_01/pom.xml

@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+
+    <parent>
+        <groupId>it.scoverage-maven-plugin</groupId>
+        <artifactId>module02</artifactId>
+        <version>1.0-SNAPSHOT</version>
+        <relativePath>../pom.xml</relativePath>
+    </parent>
+
+    <modelVersion>4.0.0</modelVersion>
+    <artifactId>submodule02_01</artifactId>
+    <name>Test Scoverage report works for specified submodules: SubModule 2_1</name>
+
+</project>

src/it/test_report_for_some_submodules/module02/submodule02_01/src/main/scala/pkg02_01/HelloService2.scala

@@ -0,0 +1,12 @@
+package pkg02_01
+
+class HelloService2
+{
+  def hello =
+  {
+    "Hello from submodule02_01"
+  }
+
+}
+
+object HelloService2 extends HelloService2

src/it/test_report_for_some_submodules/module02/submodule02_01/src/test/scala/pkg02_01/HelloServiceTest.scala

@@ -0,0 +1,14 @@
+package pkg02_01
+
+import org.junit.Test;
+import org.junit.Assert.assertEquals
+
+class HelloServiceTest
+{
+    @Test
+    def test2()
+    {
+        assertEquals("Hello from submodule02_01", HelloService2.hello)
+    }
+
+}

src/it/test_report_for_some_submodules/module02/submodule02_02/pom.xml

@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+
+    <parent>
+        <groupId>it.scoverage-maven-plugin</groupId>
+        <artifactId>module02</artifactId>
+        <version>1.0-SNAPSHOT</version>
+        <relativePath>../pom.xml</relativePath>
+    </parent>
+
+    <modelVersion>4.0.0</modelVersion>
+    <artifactId>submodule02_02</artifactId>
+    <name>Test Scoverage report works for specified submodules: SubModule 2_2</name>
+
+</project>