Mismatched "suiteEnd" event when a filter is applied

[step2yeung]

Tell us about your runtime:

• QUnit version: 2.9.4-pre
• What environment are you running QUnit in? (e.g., browser, Node): browser
• How are you running QUnit? (e.g., script, testem, Grunt): grunt

What are you trying to do?

Code that reproduces the problem:

1. git clone qunit & yarn dev
2. Add the following code to dist/qunit.js
   a. console.logs after function runLoggingCallbacks(key, args) {

function runLoggingCallbacks(key, args) {
    console.log('runLoggingCallbacks called for '+key)
    var callbacks = config.callbacks[key];

b. console.logs before if (module.testsRun === numberOfTests(module)) {

}).then(function () {
    console.log('module.testsRun === numberOfTests(module)');
    console.log(module.testsRun +' === ' +numberOfTests(module))
    if (module.testsRun === numberOfTests(module)) {
        var completedModules = [module];

3. Point a browser to http://localhost:8000/test/string-filter.html
4. Open up browser console

If you have any relevant configuration information, please include that here:

What did you expect to happen?

Expected moduleDone callback (from the suiteEnd event) to be called.

What actually happened?

moduleDone was not called because module.testsRun does not equal numberOfTests(module)
1 of the 3 test was filtered out and not ran. Leaving only 2 test to be ran.
So module.testsRun = 2 while numberOfTests(module) = 3

Regardless, I believe moduleDone should still be called.