NUnit2014

Use SomeItemsConstraint for better assertion messages in case of failure

Topic	Value
Id	NUnit2014
Severity	Info
Enabled	True
Category	Assertion
Code	SomeItemsConstraintUsageAnalyzer

Description

Using SomeItemsConstraint will lead to better assertion messages in case of failure.

Motivation

Using Does.Contain (or Does.Not.Contain) constraint will lead to better assertion messages in case of failure, so this analyzer marks all usages of string Contains method where it is possible to replace with Does.Contain constraint.

[Test]
public void Test()
{
    var actual = new List<int> {1,2,3};
    int expected = 1;
    Assert.True(actual.Contains(expected));
}

How to fix violations

The analyzer comes with a code fix that will replace Assert.True(actual.Contains(expected)) with Assert.That(actual, Does.Contain(expected)). So the code block above will be changed into

[Test]
public void Test()
{
    var actual = new List<int> {1,2,3};
    int expected = 1;
    Assert.That(actual, Does.Contain(expected));
}

Configure severity

Via ruleset file

Configure the severity per project, for more info see MSDN.

Via .editorconfig file

# NUnit2014: Use SomeItemsConstraint for better assertion messages in case of failure
dotnet_diagnostic.NUnit2014.severity = chosenSeverity

where chosenSeverity can be one of none, silent, suggestion, warning, or error.

Via #pragma directive

#pragma warning disable NUnit2014 // Use SomeItemsConstraint for better assertion messages in case of failure
Code violating the rule here
#pragma warning restore NUnit2014 // Use SomeItemsConstraint for better assertion messages in case of failure

Or put this at the top of the file to disable all instances.

#pragma warning disable NUnit2014 // Use SomeItemsConstraint for better assertion messages in case of failure

Via attribute [SuppressMessage]

[System.Diagnostics.CodeAnalysis.SuppressMessage("Assertion",
    "NUnit2014:Use SomeItemsConstraint for better assertion messages in case of failure",
    Justification = "Reason...")]