Is.Empty constraint has unclear failure message
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
NUnit Framework |
Fix Released
|
Low
|
Charlie Poole | ||
NUnit V2 |
Fix Released
|
Low
|
Charlie Poole |
Bug Description
Using v2.6 b2.
var emptyEnumerable = GetEmptyIEnumer
var emptyString = GetEmptyString();
Assert.
Assert.
If the methods under test in the above example return 'null' instead of the expected string or IEnumerable, the failure message states "System.
At least the test is failing correctly, so the problem's not too bad, but it can be slightly non-intuitive to fix the test, especially when the actual value being tested is a string.
I'd expect it either to say "The actual value must be an IEnumerable or a string", or it to give a more precise answer, something like "The actual value was not empty" / "not empty but null".
Changed in nunitv2: | |
status: | New → Triaged |
importance: | Undecided → Low |
Changed in nunitv2: | |
milestone: | none → 2.6.0 |
assignee: | nobody → Charlie Poole (charlie.poole) |
Changed in nunit-3.0: | |
status: | New → Triaged |
importance: | Undecided → Low |
Changed in nunitv2: | |
milestone: | 2.6.0b3 → 2.6.0 |
Changed in nunitv2: | |
status: | Triaged → Fix Committed |
Changed in nunit-3.0: | |
assignee: | nobody → Charlie Poole (charlie.poole) |
milestone: | none → 2.9.6 |
status: | Triaged → Fix Committed |
Changed in nunitv2: | |
status: | Fix Committed → Fix Released |
Changed in nunit-3.0: | |
status: | Fix Committed → Fix Released |