Added additional checks to the TestUpdateCollection

[segfault99]

Hello. Current TestUpdateCollection test does not detect if the update operation modifies other fields in the document, so my tests were passing albeit update function was deleting all other fields those are not in updateMap. I forgot to add this on my previous pull request.

[ostafen]

Thank you, this is a good idea. While we are at it, I would make the check not only for a single document, but for each document matching the criteria.
That is to say, before calling Update, you call:

docs, err := db.Query("todos").Where(criteria).FindAll()

Then, after doing the Update, you retrieve each updated document by repeatedly performing following query (iterating on the docs slice):

updatedDoc, err := db.Query("todos").Where(c.Field("id").Eq(doc.Get("id"))).FindFirst()

What do you think? This should be even more robust :=)

[segfault99]

Yes it will be even better.

[ostafen]

@segfault99, thank you. If you have any suggestion about features you would like to see in clover, join our gitter channel, or simply open an issue :=)