Exclude empty modules/assemblies from reports

danielpalme · Jun 5, 2024 · 6c8a5e8 · 6c8a5e8
1 parent 794b34c
commit 6c8a5e8
Show file tree

Hide file tree

Showing 8 changed files with 35 additions and 7 deletions.
diff --git a/docs/main-PKRGEOQU.js b/docs/main-PKRGEOQU.js
diff --git a/src/Readme.txt b/src/Readme.txt
@@ -67,6 +67,10 @@ For further details take a look at LICENSE.txt.
 
 CHANGELOG
 
+5.3.7.0
+
+    * Fix: Exclude empty modules/assemblies from reports
+
 5.3.6.0
 
     * Fix: #676: Fixed issue with Cobertura files with empty class names

diff --git a/src/ReportGenerator.Core/Parser/DynamicCodeCoverageParser.cs b/src/ReportGenerator.Core/Parser/DynamicCodeCoverageParser.cs
@@ -70,7 +70,11 @@ public ParserResult Parse(XContainer report)
 
             foreach (var module in modules)
             {
-                assemblies.Add(this.ProcessAssembly(module));
+                var assembly = this.ProcessAssembly(module);
+                if (assembly.Classes.Any())
+                {
+                    assemblies.Add(assembly);
+                }
             }
 
             var result = new ParserResult(assemblies.OrderBy(a => a.Name).ToList(), false, this.ToString());

diff --git a/src/ReportGenerator.Core/Parser/JaCoCoParser.cs b/src/ReportGenerator.Core/Parser/JaCoCoParser.cs
@@ -65,7 +65,11 @@ public ParserResult Parse(XContainer report)
 
             foreach (var assemblyName in assemblyNames)
             {
-                assemblies.Add(this.ProcessAssembly(modules, assemblyName));
+                var assembly = this.ProcessAssembly(modules, assemblyName);
+                if (assembly.Classes.Any())
+                {
+                    assemblies.Add(assembly);
+                }
             }
 
             var result = new ParserResult(assemblies.OrderBy(a => a.Name).ToList(), true, this.ToString());

diff --git a/src/ReportGenerator.Core/Parser/MProfParser.cs b/src/ReportGenerator.Core/Parser/MProfParser.cs
@@ -64,7 +64,11 @@ public ParserResult Parse(XContainer report)
 
             foreach (var assemblyName in assemblyNames)
             {
-                assemblies.Add(this.ProcessAssembly(methods, assemblyName));
+                var assembly = this.ProcessAssembly(methods, assemblyName);
+                if (assembly.Classes.Any())
+                {
+                    assemblies.Add(assembly);
+                }
             }
 
             var result = new ParserResult(assemblies.OrderBy(a => a.Name).ToList(), false, this.ToString());

diff --git a/src/ReportGenerator.Core/Parser/NCoverParser.cs b/src/ReportGenerator.Core/Parser/NCoverParser.cs
@@ -64,7 +64,11 @@ public ParserResult Parse(XContainer report)
 
             foreach (var assemblyName in assemblyNames)
             {
-                assemblies.Add(this.ProcessAssembly(modules, assemblyName));
+                var assembly = this.ProcessAssembly(modules, assemblyName);
+                if (assembly.Classes.Any())
+                {
+                    assemblies.Add(assembly);
+                }
             }
 
             var result = new ParserResult(assemblies.OrderBy(a => a.Name).ToList(), false, this.ToString());

diff --git a/src/ReportGenerator.Core/Parser/OpenCoverParser.cs b/src/ReportGenerator.Core/Parser/OpenCoverParser.cs
@@ -125,7 +125,11 @@ public ParserResult Parse(XContainer report)
 
             foreach (var assemblyName in assemblyNames)
             {
-                assemblies.Add(this.ProcessAssembly(assemblyModules, files, trackedMethods, assemblyName));
+                var assembly = this.ProcessAssembly(assemblyModules, files, trackedMethods, assemblyName);
+                if (assembly.Classes.Any())
+                {
+                    assemblies.Add(assembly);
+                }
             }
 
             var result = new ParserResult(assemblies.OrderBy(a => a.Name).ToList(), true, this.ToString());

diff --git a/src/ReportGenerator.Core/Parser/VisualStudioParser.cs b/src/ReportGenerator.Core/Parser/VisualStudioParser.cs
@@ -80,7 +80,11 @@ public ParserResult Parse(XContainer report)
 
             foreach (var assemblyName in assemblyNames)
             {
-                assemblies.Add(this.ProcessAssembly(modules, files, assemblyName));
+                var assembly = this.ProcessAssembly(modules, files, assemblyName);
+                if (assembly.Classes.Any())
+                {
+                    assemblies.Add(assembly);
+                }
             }
 
             var result = new ParserResult(assemblies.OrderBy(a => a.Name).ToList(), false, this.ToString());