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

Notification

Icon
Error
A comma in the test name appears with a backslash in the test report
Options
Previous Topic Next Topic
abelb
#1 Posted : Monday, August 28, 2017 2:55:55 AM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 9/12/2014(UTC)
Posts: 155
Location: Netherlands

Thanks: 19 times
Was thanked: 11 time(s) in 11 post(s)
When you have a comma in a test module name it appears as an escaped (with backslash) comma in the test window. I.e.:

Quote:
Tests.QNameFunctions.fn:QName-against-empty\, for generic-comparison


Screenshot:



Not a big bug of any kind, just thought you'd know about it. Note that it does *not* happen in test names themselves (i.e., methods), but in modules.
Back to top
Remco
#2 Posted : Monday, August 28, 2017 2:59:31 AM(UTC)
Rank: NCrunch Developer

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

Thanks: 957 times
Was thanked: 1286 time(s) in 1193 post(s)
This is interesting. I wonder if this is being done by the test framework or something even further under the hood. Can you confirm which test framework and language you're using?
Back to top
abelb
#3 Posted : Monday, August 28, 2017 3:58:39 AM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 9/12/2014(UTC)
Posts: 155
Location: Netherlands

Thanks: 19 times
Was thanked: 11 time(s) in 11 post(s)
You were quick to reply!

I just ran it in the NUnit GUI runner, latest beta version. It shows the same backslash there, so it is possibly a bug with NUnit itself.
Back to top
abelb
#4 Posted : Monday, August 28, 2017 4:02:25 AM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 9/12/2014(UTC)
Posts: 155
Location: Netherlands

Thanks: 19 times
Was thanked: 11 time(s) in 11 post(s)
And just in case Microsoft's compiler added it, this is what Reflector shows of the module (static class in C#):
Back to top
Remco
#5 Posted : Monday, August 28, 2017 4:44:01 AM(UTC)
Rank: NCrunch Developer

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

Thanks: 957 times
Was thanked: 1286 time(s) in 1193 post(s)
Yes, that definitely looks like an NUnit one. I'm not sure if there's much I can do to help here. I don't think it would be wise for NCrunch to try and clean it up, in case other relevant data got lost.
Back to top
abelb
#6 Posted : Monday, August 28, 2017 4:47:25 AM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 9/12/2014(UTC)
Posts: 155
Location: Netherlands

Thanks: 19 times
Was thanked: 11 time(s) in 11 post(s)
I agree, because then you'd create a discrepancy between how NUnit reports tests and classes. I'll see if I can raise it there as a bug report.
Back to top
1 user thanked abelb for this useful post.
Remco on 8/28/2017(UTC)
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.034 seconds.
Trial NCrunch
Take NCrunch for a spin
Do your fingers a favour and supercharge your testing workflow
Free Download