Fix of the issue when last stub is not used.

cleishm / jsmockito

Javascript mocking framework inspired by the awesome mockito

http://jsmockito.org

Other

106 stars 19 forks source link

Fix of the issue when last stub is not used. #2

Closed volodymyr-mykhailyk closed 13 years ago

volodymyr-mykhailyk commented 13 years ago

Hi Chris,

I've just pushed changes that could fix issue posted by me: https://github.com/chrisleishman/jsmockito/issues#issue/1

Could you please review this changes and pull them to trunk.

Thanks --VM

cleishm commented 13 years ago

Hey - thanks VM. I'll take a look at this as later today. However, it would be very helpful if you could resubmit the pull request with changes only to the affected lines - right now the diff marks every line in the file as having changed (probably a line ending issue).

Thanks, Chris

volodymyr-mykhailyk commented 13 years ago

Ok. I will do this.

BTW I have 2 other improvements that are very usable in framework:

add thenRun stub - ability to execute custom callback.
integration to the JSUnit framework (http://www.jsunit.net/).

volodymyr-mykhailyk commented 13 years ago

closing this one