OpenCoverUI / OpenCover.UI

Visual Studio Integration to OpenCover
MIT License
77 stars 46 forks source link

Trait grouping not working for class-level categories #118

Open MelGrubb opened 8 years ago

MelGrubb commented 8 years ago

First of all, let me say that this project is great, and I'm excited to see community support for coverage tooling. I only just started looking at it today, and it's already 95% of what I need. I got red/green highlighting and a usable report. The installation process is a little... weird, but I figured it out.

I have decorated my test classes with Category attributes, which works just fine in Visual Studio's built-in test explorer. The are not, however, picked up by OpenCover's test explorer. If I put the category attributes on the individual tests, then they are properly categorized.

If someone looks into this, I would also suggest that you make sure to look at inherited attributes. I normally add my category attributes to an abstract class that applies to an entire area. For instance, I may have an abstract FooControllerTests base class, from which I would inherit a When_Getting_the_Index_view class, which would have a Then_the_blah_blah_blah method. This way, the category applies to a whole group of tests automatically, and I can re-categorize them easily if I wanted to.

Of course, project/namespace hierarchy grouping would obviate the need for trait grouping altogether in my case. I see that's already been requested. I second that request.

pver commented 8 years ago

Good suggestion indeed, thanks! :+1: I'll add it as planned for the next release

lorenh commented 8 years ago

This would be a great addition we could really use. Is there any sort of timeline for pulling together another release with all of the fixes since 0.8.1?