vndb/mithril-vndb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix onunload test case

Browse source 
- the previous test only remove last item on array, but expect all
subcomponent's onunload to be called. This PR change it by clear that
array.
- uncomment previous test case that marked fail.
This commit is contained in:
Syaiful Bahri 2015-12-29 09:58:26 +07:00
parent 5003c0ac23
commit 795f098404
1 changed files with 8 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

19
test/mithril.mount.js
View file

@ -333,12 +333,11 @@ describe("m.mount()", function () {
})
}))
list.pop()
list = []
refresh(true)
// TODO: These fail.
// expect(spies[1]).to.have.been.called
// expect(spies[2]).to.have.been.called
expect(spies[1]).to.have.been.called
expect(spies[2]).to.have.been.called
expect(spies[3]).to.have.been.called
})
@ -374,17 +373,15 @@ describe("m.mount()", function () {
})
}))
list.pop()
list = []
refresh(true)
// TODO: These fail.
// expect(spies1[1]).to.have.been.called
// expect(spies1[2]).to.have.been.called
expect(spies1[1]).to.have.been.called
expect(spies1[2]).to.have.been.called
expect(spies1[3]).to.have.been.called
// TODO: These fail.
// expect(spies2[1]).to.have.been.called
// expect(spies2[2]).to.have.been.called
expect(spies2[1]).to.have.been.called
expect(spies2[2]).to.have.been.called
expect(spies2[3]).to.have.been.called
})