Closed rhelmer closed 3 years ago
I think this is good enough to land; it has some basic unit tests and I built the integration test with it.
I shored up the mocks some more so we can actually test when onSnapshot
is registered - still need to go a little further and be able to subsequently call onSnapshot
(like we do with onAuthStateChanged
) but I want to investigate a bit more and see if there's a better way to do this with Jest, I suspect there is.
I'm testing this by embedding it in a test extension in https://github.com/mozilla-rally/rally-web-platform/pull/101 - once tests over there pass I think we'll be good to land this, and then I'll switch this PR to use the release version.
This adds support for Firebase, and for handling messaging between the website and the extension that is embedding the SDK.
Closes issue #5