Welcome Guest! To enable all features please Login or Register.

Notification

Icon
Error
NCrunch incorrectly reporting tests as having the same name
Options
Previous Topic Next Topic
bartj
#1 Posted : Monday, August 6, 2018 11:46:38 PM(UTC)
Rank: Member

Groups: Registered
Joined: 12/4/2013(UTC)
Posts: 26
Location: New Zealand

Thanks: 2 times
Was thanked: 3 time(s) in 3 post(s)
NCrunch is reporting that two distinct NUnit tests have the same name, even though nunit-console.exe and the ReSharper test runner correctly report them as two different tests.

This test code is enough to trigger the incorrect warning from NCrunch:

Code:
[TestCase("abc")]
[TestCase("\x0007bc")]
public void NotDuplicates(string input)
{
}
Back to top
Remco
#2 Posted : Tuesday, August 7, 2018 12:30:20 AM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 7,144

Thanks: 959 times
Was thanked: 1290 time(s) in 1196 post(s)
Hi Bart,

Thanks for sharing this issue. I've reproduced it as described. We're talking about this internally and will get back to you soon.
Back to top
michaelkroes
#3 Posted : Tuesday, August 7, 2018 11:45:51 AM(UTC)
Rank: NCrunch Developer

Groups: Registered
Joined: 9/22/2017(UTC)
Posts: 306
Location: Netherlands

Thanks: 138 times
Was thanked: 73 time(s) in 69 post(s)
Hi,

We've had a look at this. This was introduced in version 3.17.0.2 to improve the rendering of NUnit tests that have a TestCase attribute that contains escape characters or unicode. NUnit adds slashes to the testname that it reports back. We've tried improving on that rendering, but since it causes these side effects the change is rolled back. This rollback will be in the next release.

Again, thanks for sharing this with us!
Back to top
bartj
#4 Posted : Monday, August 27, 2018 5:43:48 PM(UTC)
Rank: Member

Groups: Registered
Joined: 12/4/2013(UTC)
Posts: 26
Location: New Zealand

Thanks: 2 times
Was thanked: 3 time(s) in 3 post(s)
Hi,

Will you be attempting to fix the original issue, or will the bug resurface in the next release?

Thanks,
Bart
Back to top
michaelkroes
#5 Posted : Tuesday, August 28, 2018 6:54:38 AM(UTC)
Rank: NCrunch Developer

Groups: Registered
Joined: 9/22/2017(UTC)
Posts: 306
Location: Netherlands

Thanks: 138 times
Was thanked: 73 time(s) in 69 post(s)
For now we will not fix the original issue. Fixing it would mean deviating from how other test runners display the test name and we haven't thought of a better way to display the tests without introducing duplicate test names. For the time being we will show the test name as it is reported by nunit without any modification.
Back to top
Users browsing this topic
Guest
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

YAF | YAF © 2003-2011, Yet Another Forum.NET
This page was generated in 0.029 seconds.
Trial NCrunch
Take NCrunch for a spin
Do your fingers a favour and supercharge your testing workflow
Free Download