superfeedr / indexeddb-backbonejs-adapter

An indexedDB adapter for Backbonejs
http://blog.superfeedr.com/indexeddb-backbonejs-adapter/
MIT License
248 stars 61 forks source link

Failed test with Firefox 33 (global failure) #61

Closed mcepl closed 9 years ago

mcepl commented 10 years ago
Error: assertion outside test context, was tests</<.success/<.success/<.success@http://localhost:8000/tests/test.js:797:33 .save/i.success@http://localhost:8000/lib/backbone-min.js:1:5387 sync/options.success@http://localhost:8000/backbone-indexeddb.js:590:26 Driver.prototype.update/writeTransaction.oncomplete@http://localhost:8000/backbone-indexeddb.js:293:17
Source:     

http://localhost:8000/lib/qunit.js:431

With Mozilla/5.0 (X11; Linux x86_64; rv:33.0) Gecko/20100101 Firefox/33.0

julien51 commented 10 years ago

You're more than welcome to submit fixes :)

mcepl commented 10 years ago

I know, I have not expected anything else from you ;)

julien51 commented 10 years ago

Closing now... please re-open with a fix and tests!

mcepl commented 10 years ago

Could you please reopen? I have just reproduced it with the current master (839a7bd) and Firefox Nightly (Mozilla/5.0 (X11; Linux x86_64; rv:35.0) Gecko/20100101 Firefox/35.0):

global failure (1, 0, 1)Rerun1 ms

    Error: assertion outside test context, was tests</<.success/<.success/<.success@http://localhost:8000/tests/test.js:893:33 .save/i.success@http://localhost:8000/lib/backbone-min.js:1:5387 sync/options.success@http://localhost:8000/backbone-indexeddb.js:587:26 Driver.prototype.update/writeTransaction.oncomplete@http://localhost:8000/backbone-indexeddb.js:263:17
    Source:     

    http://localhost:8000/lib/qunit.js:431
julien51 commented 10 years ago

Done! A fix would be nice!

julien51 commented 9 years ago

Ok, closing this time until we have a fix.