Skipped child module hooks are leaked

[bendemboski]

Tell us about your runtime:

• QUnit version: 2.14, 2.16
• Which environment are you using? (e.g., browser, Node): both browser & Node
• How are you running QUnit? (e.g., QUnit CLI, Grunt, Karma, manually in browser): Ember tests & CLI

What are you trying to do?

Execute a test run that filters out some child modules that have hooks.

Code that reproduces the problem:

import { module, test } from 'qunit';

class Foo {}

module('parent', function(hooks) {
  let foo1;

  hooks.beforeEach(function() {
    foo1 = new Foo();
  });

  test('foo1 is ok', function(assert) {
    assert.ok(foo1);
  });

  module('child', function(hooks) {
    let foo2;

    hooks.beforeEach(function() {
      foo2 = new Foo();
    });

    test('foo2 is ok', function(assert) {
      assert.ok(foo2);
    });
  });
});

Run with filter=!child so the child module is not executed.

What did you expect to happen?

foo1 and foo2 are garbage collected after the test run because of the code that deletes module hooks out of the global config.

What actually happened?

They are not because the child module's tests never execute, so the child module never "completes" and its hooks are never deleted out of the global config.