Web worker Tests

[wmluke]

Hi @tofumatt,

Here are some unit tests to flush out worker support mentioned in #144.

I ran most of my testing manually in Chrome (v34.0.1847.131), Safari (v7.0.3), and Firefox (v29.0). Then I followed up with the casperjs/slimerjs tests.

Test Results:

• Chrome:
  • asyncStorage: pass
  • webSQLStorage: pass
  • localStorageWrapper: fails but this is expected as localstorage is not available within a worker
• Safari:
  • asyncStorage: fail but this is expected as Safari doesn't yet support IndexDB
  • webSQLStorage: pass
  • localStorageWrapper: fails but this is expected as localstorage is not available within a worker
• Firefox/Casper:
  • asyncStorage: fail
  • webSQLStorage: fail
  • localStorageWrapper: fails but this is expected as localstorage is not available within a worker

I'm not exactly sure why but it seems that in Firefox localForage forces the driver to localStorageWrapper within a worker even when explicitly set to asyncStorage or webSQLStorage.

Also, I added two new grunt tasks: grunt serve and grunt test-webworker. The former just keeps the connect servers running for manual testing and the later just runs test/test.webworker.coffee.

Anyway, let me know if there is anything more to do.

Thanks again,
Luke

[tofumatt]

Interesting, so inside a web worker, localStorage backend is broken... I guess web workers are a bit of a special case... do any localStorage-only browsers support web workers? I'd think not.

[tofumatt]

Oh? ^_^

[wmluke]

Oops, my auto code formatter can be a too aggressive sometimes...thought I had disabled.

[tofumatt]

No prob :-) I’ve added the Gruntfile to the jshint checker, though it doesn’t even seem to complain about that one, so maybe I need to make it more aggressive! ^_^

[tofumatt]

This is great; I'm going to check this out locally and go through this. Will try to merge today. 👍

[tofumatt]

Seems there's some bug with slimerjs and web workers, you're right. I can't seem to fix it easily, so I'm going to skip the tests in slimer (we do this already; it's a bit buggy). At some point I'd like to move to a more reliable (and also cross-platform) testing platform, so maybe we'll fix this then ( #148).

I've tweaked some of your code locally; I'll push it and merge it in soon.

[tofumatt]

Merged b23cc8b and cb39ee2.

See the release here: https://github.com/mozilla/localForage/releases/tag/0.8.0