Prevent early return (before assertion) in QUnit test

[platinumazure]

The purpose of this rule is to prevent unit tests from returning early (even conditionally) if there are still assertions further in the test code. If a test may need to return early, this is a sign that the test either is testing nondeterministically (in which case the test or code should be refactored to support deterministic testing), or that the entire test might need to be skipped in certain conditions.

The following example is a problem under this rule:

QUnit.test("test", function (assert) {
    if (true) {
        return;
    }
    assert.ok(false);
});

The following examples are not problems under this rule:

if (condition) {
    QUnit.test("test", function (assert) {
        assert.ok(false);
    });
}

QUnit[condition ? "test" : "skip"]("a test", function (assert) {
    assert.ok(false);
});

[platinumazure]

For now, we can report on any ReturnStatement in a test as long as it is not in a new function scope.