[InheritsTests]: opt-out instead of opt-in?

[kimpenhaus]

hey @thomhurst

thanks for tunit and keeping such pace in further development - very impressive 👍🏼

we were evaluating if we could switch from mstest to tunit. which is more or less possible by search and replace of some attributes. one of the problems we are facing is:

we have close to 18.000 unit tests which are also organized by some inheritance. the analyzer gives warnings that when having inheritance we should use [InheritsTests] (I guess this could be for the genrerators to kick in). you can imagine that we've got a lots of analyzer warnings about the [InheritsTests] - which when having TreatWarningsAsErrors - fails the build. Instead of always "opting in" - is there any option to have it as "default" to inherit tests ... and - when not wanted - to have an option to opt-out? (I can't think of any circumstance why someone wnats to ... but anyways 😄 )

thanks for your feedback.

[thomhurst]

Thanks 😄

Unfortunately source generators have to look up something in order to do their job.

They could look at just classes, but this means they're constantly firing on any change and leads to a slow and laggy IDE experience.

The recommended method is to make it look at specific attributes instead, and that's why you have to use [InheritsTests] so that the source generator can find it.

What I could do to help, is create a code fixer for that analyzer that adds the attribute for you. So you could run something like
dotnet format analyzers --diagnostics "TUnit####"

Would that be useful for you?

[kimpenhaus]

What I could do to help, is create a code fixer for that analyzer that adds the attribute for you.

if that would be possible - I think that would be amazing. as far as I know code fixes could be applied to all warnings solution-wide. that would be appreciated.

thanks!

[thomhurst]

Code fix should be in 0.16.1

Run it by doing dotnet format --severity info --diagnostics TUnit0030

[kimpenhaus]

man - you're machine - thanks a thousand times 🙏