Exploring tests with filter returns all tests

[ivanli]

NUnit.Engine.ITestRunner.Explore(...) does not obey the filter that is passed as argument to it and returns a list of all the tests within the assembly.

This behaviour is inconsistent with calling NUnit.Engine.ITestRunner.Run(...), which obeys the filter as expected and only executes the tests which pass the filter.

[CharliePoole]

@ChrisMaddock I'm reviewing each issue in the light of the upcoming release.

This one IMO should get in. One of us may need to complete it if @ivanli doesn't respond on the PR. It sat idle for a long time, so he may be working on other things by now.

Additionally, the PR would need to be re-reviewed in light of the changes made by PR #30.

In any case, I think this is important enough that it should get into the release.

[CharliePoole]

This was confirmed once but it needs to be re-confirmed because the underlying code has changed quite a bit since the issue was filed.

UPDATE: Confirmed!

[CharliePoole]

Keeping this as a v2 framework driver bug. Since nunitv2 actually has no Explore function the driver will have to build it on top of the framework by loading the entire assembly and subsequently applying the filter.

In fact, this adds a new feature. The rationale here is

1. Not having the feature creates a bug because the engine expects it to be present.

2. Adding the feature in nunitv2 itself makes no sense since it would only ever be used by the adapter.

[CharliePoole]

@ivanli Can you try one of the packages at https://ci.appveyor.com/project/CharliePoole/nunit-v2-framework-driver/builds/31644230/artifacts to see if it works for you?

[ivanli]

Hi Charlie, it's been a while since this issue came up and we've moved to nunit v3. I can dig up the old project and give it a go sometime this week. Will update.

[CharliePoole]

@ivanli Yes, it has taken a while. I only now took over this project again, so I'm cleaning up old stuff. If you are able to check it out, that will be handy but if you can't do it easily just let me know.