Skip to content

Commit

Permalink
fix breaking test
Browse files Browse the repository at this point in the history
  • Loading branch information
rvagg committed Nov 23, 2013
1 parent 0ba8188 commit d2b939a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions test.js
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -475,7 +475,7 @@ tap.test('test clear() extensibility', function (t) {


tap.test('test iterator() extensibility', function (t) { tap.test('test iterator() extensibility', function (t) {
var spy = sinon.spy() var spy = sinon.spy()
, expectedOptions = { options: 1 } , expectedOptions = { options: 1, reverse: false } // reverse now explicitly set
, test , test


function Test (location) { function Test (location) {
Expand All @@ -487,12 +487,12 @@ tap.test('test iterator() extensibility', function (t) {
Test.prototype._iterator = spy Test.prototype._iterator = spy


test = new Test('foobar') test = new Test('foobar')
test.iterator(expectedOptions) test.iterator({ options: 1 })


t.equal(spy.callCount, 1, 'got _close() call') t.equal(spy.callCount, 1, 'got _close() call')
t.equal(spy.getCall(0).thisValue, test, '`this` on _close() was correct') t.equal(spy.getCall(0).thisValue, test, '`this` on _close() was correct')
t.equal(spy.getCall(0).args.length, 1, 'got one arguments') t.equal(spy.getCall(0).args.length, 1, 'got one arguments')
t.equal(spy.getCall(0).args[0], expectedOptions, 'got expected options argument') t.deepEqual(spy.getCall(0).args[0], expectedOptions, 'got expected options argument')
t.end() t.end()
}) })


Expand Down Expand Up @@ -556,4 +556,4 @@ tap.test('test end() extensibility', function (t) {
t.equal(spy.getCall(0).args.length, 1, 'got one arguments') t.equal(spy.getCall(0).args.length, 1, 'got one arguments')
t.equal(spy.getCall(0).args[0], expectedCb, 'got expected cb argument') t.equal(spy.getCall(0).args[0], expectedCb, 'got expected cb argument')
t.end() t.end()
}) })

0 comments on commit d2b939a

Please sign in to comment.