Feature: assertThrows should not succeed when a "marked to fail" method was called on mock

[realistschuckle]

From Feature: assertThrows should not succeed when a "marked to fail" method was called on mock (Issue #14) from @jigargosar:

I really am missing this feature.

I have marked a method to fail. And need to assert the fact in test that that method was never called.

In this particular case, the mock throws an error, but the SUT catches it, since its running a loop, with try/catch in the loop.

Thus my tests are all passing. :(

Ideally if assertThrows would throw an exception when any method was called on mock that was marked for failure, this wouldn't let my test pass.

Is it possible to easily integrate this feature?

thanks.

this.assert = function() {      
    var success = true;
    for(var method in entries) {
        var entriesForMethod = entries[method];
        entriesForMethod.forEach(function(entry) {
            if(!entry.shouldFail && entry.executed == false) {
                success = false;
                console.error(
                        "method call for: '" + method + "()' with params: " + getParamString(entry.args) + " was not executed!\n"
                );
            }
            if(entry.shouldFail && entry.executed){
                success = false;
                console.error(
                        "method call for: '" + method + "()' with params: " + getParamString(entry.args) + " should not be executed!\n"
                );
            }
        });
    }

    return success;
};

[realistschuckle]

In other mocking frameworks, the fail method would have a name like raiseError or throws. The assertThrows is a method that raises an Error when asserting that the test has met the expectations of the mock unlike the assert method that just writes stuff to console.error. We will fix this confusion, I think, in the next major release.

[realistschuckle]

I have just created an interim milestone that will act as a bridge between versions 0.3 and 1.0. I will address this issue in that release.