Skip to content

Commit

Permalink
#385: Added filtering by namespace for classfilters in DynamicCodeCov…
Browse files Browse the repository at this point in the history
…erageParser
  • Loading branch information
danielpalme committed Sep 8, 2020
1 parent cea803b commit 2290f09
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
1 change: 1 addition & 0 deletions src/Readme.txt
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ CHANGELOG
4.6.7.0

* Fix: #363: Improved handling of HTML generation (several HTML reports at once)
* Fix: #385: Added filtering by namespace for classfilters in DynamicCodeCoverageParser
* Fix: #388: Removed branch coverage columns if no branch coverage is available

4.6.6.0
Expand Down
4 changes: 1 addition & 3 deletions src/ReportGenerator.Core/Parser/DynamicCodeCoverageParser.cs
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ internal DynamicCodeCoverageParser(IFilter assemblyFilter, IFilter classFilter,
/// Parses the given XML report.
/// </summary>
/// <param name="report">The XML report.</param>
/// <param name="innerMaxDegreeOfParallism">The max degree of parallism for the class iteration foreach loop</param>
/// <returns>The parser result.</returns>
public ParserResult Parse(XContainer report)
{
Expand Down Expand Up @@ -81,7 +80,6 @@ public ParserResult Parse(XContainer report)
/// Processes the given assembly.
/// </summary>
/// <param name="module">The module.</param>
/// <param name="innerMaxDegreeOfParallism">The max degree of parallism for the class iteration foreach loop</param>
/// <returns>The <see cref="Assembly"/>.</returns>
private Assembly ProcessAssembly(XElement module)
{
Expand All @@ -106,7 +104,7 @@ private Assembly ProcessAssembly(XElement module)
return c;
})
.Distinct()
.Where(c => this.ClassFilter.IsElementIncludedInReport(c.ClassName))
.Where(c => this.ClassFilter.IsElementIncludedInReport(c.FullName))
.OrderBy(c => c.Namespace)
.ThenBy(c => c.ClassName)
.ToArray();
Expand Down

0 comments on commit 2290f09

Please sign in to comment.