phuccaoca123 / mb-unit

Automatically exported from code.google.com/p/mb-unit
0 stars 0 forks source link

Multiple FixtureCategory Tree Nodes - Bug or Feature? #81

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

1. Create a TestFixture with multiple FixtureCategory attributes in it.
2. In each FixtureCategory put a period in the name.  Example: 
FixtureCategory("MyApp.DataAccess"), FixtureCategory("MyApp.BusinessLogic")

What is the expected output? What do you see instead?

In the MbUnit GUI, the class shows up in both categories, and each category
name is split by period to create a branch (Parent->Child)  So it becomes

- MyApp
    - DataAccess
         mytestclass
    - BusinessLogic
         mytestclass

What version of the product are you using? On what operating system?
2.4.197 / XP Pro

Please provide any additional information below.

I'm not sure if this is a bug or a feature.  If it is a feature, great!  It
would really help with what I'm doing, but then I have a bug with it.  If
you have 2 FixtureCategory attributes within the same TestFixture and one
ends with the same word as the other, it will only show up in 1 of the
places on the tree.  For example:
[TestFixture, FixtureCategory("MyApp.BusinessLogic"),
FixtureCategory("BusinessLogic")] 
public class mytestclass(){}

mytestclass will only show up under 1 of the categories above, not both.

Original issue reported on code.google.com by hecktarz...@gmail.com on 10 Oct 2007 at 2:43

GoogleCodeExporter commented 9 years ago
It's definitely a feature! (patch contains fix & regression test)

Original comment by grahamr...@gmail.com on 15 Oct 2007 at 3:08

Attachments:

GoogleCodeExporter commented 9 years ago
When do you anticipate the next release will be with this patch in it?

Original comment by hecktarz...@gmail.com on 16 Oct 2007 at 2:00

GoogleCodeExporter commented 9 years ago

Original comment by jeff.br...@gmail.com on 30 Oct 2007 at 2:07

GoogleCodeExporter commented 9 years ago
Applied patch & added test.

Original comment by grahamr...@gmail.com on 3 Dec 2007 at 2:59