warren/jasmine
1
0
Fork 0
Code Issues Pull Requests Actions 9 Packages Projects Releases Wiki Activity

Fix spy arity stuff to work in IE8

Browse Source
This commit is contained in:
Gregg Van Hove
2016-09-28 11:49:07 -07:00
parent 63cc6687ff
commit 0d1a9a72d9
3 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lib/jasmine-core/jasmine.js
View File

@@ -2058,7 +2058,8 @@ getJasmineRequireObj().Spy = function (j$) {
More explanation here (look at comments): http://www.bennadel.com/blog/1909-javascript-function-constructor-does-not-create-a-closure.htm
*/
/* jshint evil: true */
wrapper = eval('(function (' + args + ') { return spy.apply(this, Array.prototype.slice.call(arguments)); })'),
wrapper = eval('(0, function (' + args + ') { return spy.apply(this, Array.prototype.slice.call(arguments)); })'),
/* jshint evil: false */
spyStrategy = new j$.SpyStrategy({
name: name,
fn: originalFn,

4
spec/core/SpySpec.js
View File

@@ -69,8 +69,8 @@ describe('Spies', function () {
function senary (arg1, arg2, arg3, arg4, arg5, arg6) {}
];
functions.forEach(function (someFunction, arity) {
var spy = jasmineUnderTest.createSpy(someFunction.name, someFunction);
for (var arity = 0; arity < functions.length; arity++) {
var spy = jasmineUnderTest.createSpy(functions[arity].name, someFunction);
expect(spy.length).toEqual(arity);
});

3
src/core/Spy.js
View File

@@ -8,7 +8,8 @@ getJasmineRequireObj().Spy = function (j$) {
More explanation here (look at comments): http://www.bennadel.com/blog/1909-javascript-function-constructor-does-not-create-a-closure.htm
*/
/* jshint evil: true */
wrapper = eval('(function (' + args + ') { return spy.apply(this, Array.prototype.slice.call(arguments)); })'),
wrapper = eval('(0, function (' + args + ') { return spy.apply(this, Array.prototype.slice.call(arguments)); })'),
/* jshint evil: false */
spyStrategy = new j$.SpyStrategy({
name: name,
fn: originalFn,