Skip to content

NUnit2018

Mikkel Nylander Bundgaard edited this page Apr 25, 2020 · 2 revisions

NUnit2018

Consider using Assert.That(expr, Is.Not.Null) instead of Assert.NotNull(expr).

Topic Value
Id NUnit2018
Severity Info
Enabled True
Category Assertion
Code ClassicModelAssertUsageAnalyzer

Description

Consider using the constraint model, Assert.That(expr, Is.Not.Null), instead of the classic model, Assert.NotNull(expr).

Motivation

The classic Assert model contains less flexibility than the constraint model, so this analyzer marks usages of Assert.NotNull from the classic Assert model.

[Test]
public void Test()
{
    object obj = null;
    Assert.NotNull(obj);
}

How to fix violations

The analyzer comes with a code fix that will replace Assert.NotNull(expression) with Assert.That(expression, Is.Not.Null). So the code block above will be changed into.

[Test]
public void Test()
{
    object obj = null;
    Assert.That(obj, Is.Not.Null);
}

Configure severity

Via ruleset file.

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

Via #pragma directive.

#pragma warning disable NUnit2018 // Consider using Assert.That(expr, Is.Not.Null) instead of Assert.NotNull(expr).
Code violating the rule here
#pragma warning restore NUnit2018 // Consider using Assert.That(expr, Is.Not.Null) instead of Assert.NotNull(expr).

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

#pragma warning disable NUnit2018 // Consider using Assert.That(expr, Is.Not.Null) instead of Assert.NotNull(expr).

Via attribute [SuppressMessage].

[System.Diagnostics.CodeAnalysis.SuppressMessage("Assertion", 
    "NUnit2018:Consider using Assert.That(expr, Is.Not.Null) instead of Assert.NotNull(expr).",
    Justification = "Reason...")]
Clone this wiki locally