Election-Tech-Initiative / Election-Tech-Initiative/electionguard-core2

🐞 Dotnet Unit Tests Use Consistent Assertions

Open
#188 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
C#
Stars
14
Forks
12
PR merge metrics
No merged PRs in 30d

Description

### Is there an existing issue for this?

- [X] I have searched the existing issues

### Current Behavior

some use:

`Assert.That(result, Is.Not.Null);`

instead of `Assert.IsNotNull(result);`

### Expected Behavior

use a consistent format throughout and configure intellisense to highlight it.

https://docs.nunit.org/articles/nunit-analyzers/NUnit2019.html#motivation

### Steps To Reproduce

_No response_

### Environment

```markdown
- OS:
```

### Anything else?

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.