Grouping by class name in VS 2012 doesn't work

Added screenshot showing issue

I stumbled across this today after installing VS Update 2 to get Class-level test groupings to work. I decided to fix it according to the method outlined by Terje. It appears to work well for me. Attached is a patch of the diff.

Here's an image of how it looks:
http://i.imgur.com/VDms9N0.png

Bug is now fixed, but the fix causes some duplicate tests to be skipped. That's because UniqueName was making them unique. We'll deal with that problem separately and in a way that doesn't harm the ability to display tests by class.

@Tom: I used part of your patch. Some of the changes you made could lead to duplicate keys in the dictionary of NUnit tests so I didn't use them. However, it was useful to see that the code misled you a bit, since it didn't distinguish clearly between maps of NUnit tests and maps of VS TestCases. I changed some names to make that clearer.

Charlie

@Charlie: Happy to be useful. I actually did encounter some errors with the changes I made, and I see now from your comment why they were cropping up. One thought I had was perhaps altering UniqueName (or creating a new version of it) that put the uniqueness at the end, since that shouldn't affect group by class name, while still keeping the uniqueness. I don't know enough about the code base to know everything that could break, however. In any case, thanks for fixing this. It's a really great feature of the new VS and it's fantastic to be able to use it with NUnit.