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

Notification

Icon
Error

MetricsExclusions saved as absolute paths
Marqus
#1 Posted : Friday, May 25, 2012 10:04:54 AM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 2/22/2012(UTC)
Posts: 38
Location: Poland

Thanks: 2 times
Was thanked: 6 time(s) in 6 post(s)
In *.ncrunchsolution paths in MetricsExclusions are absolute - IMHO they should be relative. We keep this file under source control, developers has different paths to their local workspaces, so we are getting conflicts all the time, and this conflicts aren't resolvable.

Another suggestion: MetricsExclusions could have children nodes - one node for file - and this children could be alphabetically sorted. This way automatic confilct resolution will have easier task.
Remco
#2 Posted : Friday, May 25, 2012 11:14:05 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, this makes good sense. Thanks for letting me know about the merge issues ... I'll see what I can do.
Remco
#3 Posted : Tuesday, June 19, 2012 4:44:47 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)
For anyone interested, 1.40b has just been released including the change requested above. Metrics exclusions should now be using relative paths (as opposed to absolute), and are sorted alphabetically for easier merging.
Marqus
#4 Posted : Tuesday, June 19, 2012 6:40:05 AM(UTC)
Rank: Advanced Member

Groups: Registered
Joined: 2/22/2012(UTC)
Posts: 38
Location: Poland

Thanks: 2 times
Was thanked: 6 time(s) in 6 post(s)
Thanks

1.40b doesn't import MetricsExclusion from .ncrunchsolution saved in 1.39b format, so I have to set them manually again :(
I didn't noticed this before, but why this setting is in *.ncrunchsolution and not in *.ncrunchproject? If project is used in many solutions, it has to be configured many times.
Remco
#5 Posted : Tuesday, June 19, 2012 8:18:00 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)
Hi Marqus -

Yes, sorry, I needed to include it as a separate property to simplify the upgrade process (since the file references are now in relative rather than absolute form).

The data is stored at solution level more because of technical/design/ease-of-implementation reasons. NCrunch stores all code coverage information at solution level (i.e. in the .cache file), as the tests that cover a project may not be present in all solutions that use the project.

From a configuration perspective, storing the exclusions at project level does make more sense though. I'll have a think about it.


Cheers,

Remco
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.031 seconds.
Trial NCrunch
Take NCrunch for a spin
Do your fingers a favour and supercharge your testing workflow
Free Download