Enable running tests by location

[bclothier]

In addition to category and selection, it should be possible (and hopefully trivial) to run tests based on their location (e.g. module they are in).

Originally posted by @bclothier in #4698 (comment)

[Vogel612]

This is somewhat abstractable to running tests based on their current grouping in the grouping grid. As such we'd need a "Run Tests in Selected Group", where group would be any single of:

• Location
• Outcome
• Category
• ...?

[Irubataru]

I think that is what I suggested when I talked about it that it would be more versatile if you could run issues in the selected group. In that case it could also replace "Run selected category" (and invalidate #4703) as that functionality would be there if you chose to group by category.

[comintern]

The problem inherent in this UI is that it kind of conflicts with "Run selected tests" if we're allowing multi-selection in the test grid (see #4701). If the user Ctrl selects multiple row in different groupings, then what exactly is the "Selected Group"? Both of them?

The menu on the toolbar currently gives the following options:

Compare this to the options available in the VS "Run..." menu:

The main difference is that VS also has context menus in its TreeView for running a selected node, whether or not it has sub-nodes. If it does have sub-nodes, it runs all of them.

The compromise here might be to just remove the "Selected Category" option from the toolbar completely (ref #4703) and change the "Selected Test" option to "Selected Tests" (plural). We could then add context menus on the grid expanders more fine grained control. Maybe something like:

Run Group
---------
Collapse
Expand

Individual tests could have context menus also, but I'm not sure what to put in them other than "Run Test" (although VS has an "Add to playlist option..." 😀 )

Thoughts?

[bclothier]

FWIW as long we gain a right-click menu to run a test in a group (regardless of what the grouping is set), I think losing the Selected Category and having Run selected tests makes more sense and as a bonus simplify the UI. I'm for it.

[comintern]

Closed via #4769.