-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ReportGenerator producing duplicate class rows in summary (cobertura format) #124
Comments
Can you please use latest version of reportgenerator and try again? This has been fixed in reportgenerator => danielpalme/ReportGenerator@edb7491#diff-dc56cf0bce5eabc34a9b64648410d7f2edf7673cfb44bb447bf0903387d245a4 |
I used latest released reportgenerator version 5.3.6 |
I would recommend to share these cases with them. Hopefully they will fix these cases as well. Here is the existing issue. |
I will have a look in the next days. |
I took a look at those class names. With ReportGenerator version 5.3.6 the part ".d__0" is removed from the classes. This results in the following names:
If I would also remove the generic type But there are other cases where the generic type at the end is related to the class itself. Example: Corresponding class: public class GenericAsyncClass<T>
{
public async Task MyAsyncMethod()
{
}
} The only way to distinguish between these two variants is @standsed proposal:
It would be perfect if Microsoft CodeCoverage could use With coverlet the class |
We keep the names as it is when generated.
|
In scenario below I am getting duplicate rows (for the same class) in ReportGenerator summary output
For following code
Generated report looks as follows
Full project where this is reproduced https://github.com/standsed/CodeCoverageRepro
Issue seems arise from class name in produced cobertura xml file.
1st row is from codecoverage file 2nd from coverlet
Looks like dot after Class1 breaks some aggregation in ReportGenerator. If I replace this dot with / as in coverlet then report correctly displays just one row for Class1.
coverlet.txt
ms_codecoverage.txt
Since dot is a namespace delimiter this could be something which might need to be adjusted in codecoverage, but let me know if you see this otherwise.
The text was updated successfully, but these errors were encountered: