Add support for onSnapshot about firestore-jest-mock HOT 5 CLOSED

Thanks for pointing this out. It's not an API i've used.

Digging into firestore's docs, it looks like it's a bit more complicated than the get call on a collection.
https://firebase.google.com/docs/reference/js/firebase.firestore.CollectionReference#onsnapshot

🤔 That seems a little inconsistent with some of their documented use-cases

db.collection("cities").doc("SF")
    .onSnapshot({
        // Listen for document metadata changes
        includeMetadataChanges: true
    }, (doc) => {
        // ...
    });

Or

db.collection("cities").where("state", "==", "CA")
    .onSnapshot((querySnapshot) => {
        var cities = [];
        querySnapshot.forEach((doc) => {
            cities.push(doc.data().name);
        });
        console.log("Current cities in CA: ", cities.join(", "));
    });

Sometimes they refer to the object returned as a doc but other times as a QuerySnapshot which is an instance of a QuerySnapShot object.

I'm assuming that it depends on whether you are fetching one versus multiple records. We should be able to get this to work with our current buildQuerySnapShot helper (props to @AverageHelper for building it 🙌 ).

This was mostly me thinking out loud, but all this is to say that I will try to build this out but it seems to be a bit more complicated than just returning the same results as get. I want to make sure we add support for the other things that a snapshot can do, in case someone else uses this and it doesn't work as expected.

Any tips are welcome

from firestore-jest-mock.

Thanks for taking a look!
Yeah I have not used the optional observer or options parameters of the onSnapshot method, so I don't have much insight into them and have no idea how widly used they are. I have only used the basic pattern:

onSnapshot ( onNext :  ( snapshot/doc :  QuerySnapshot/Document < T > ) => void)

Like the .get().then(...) pattern the onNext callback returns a single doc for DocumentReference.onSnapshot() and QuerySnapshot for the CollectionReference.onSnapshot()

from firestore-jest-mock.

@HauptmannEck Can you see if your tests pass with #65 in your repo?

from firestore-jest-mock.

@sbatson5 The tests I had built now work with that branch, and I get the expected output.

from firestore-jest-mock.

Versions 0.8.0 was released with this update!

from firestore-jest-mock.