DotNetAnalyzers / StyleCopAnalyzers

An implementation of StyleCop rules using the .NET Compiler Platform
MIT License
2.66k stars 508 forks source link

New Rule Proposal: SA1404 but for ExcludeFromCoverage #3877

Open mu88 opened 4 months ago

mu88 commented 4 months ago

I'd appreciate having a warning for cases when [ExcludeFromCoverage] is used without a justification, similar to SA1404.

For example, the following code would be a rule violation...

[ExcludeFromCoverage]
public bool Enable()
{
}

...and adding a justification would fix the violation:

[ExcludeFromCoverage(Justification = "Cannot be tested because...")]
public bool Enable()
{
}

If you consider this useful, I'd happily provide a PR.