HomePhabricator
Diffusion Comm 4ffcc9b0aa7f

[lib] Update `threadStore` with `threadStoreOps` for `saveMessages` action

Description

[lib] Update threadStore with threadStoreOps for saveMessages action

Summary:
Copied from D4185:

Previously we were directly updating the threadStore object with the necessary changes for each action... and also constructing a corresponding list of ops that were used for modifying the SQLite store (on native).

We've had an assertThreadStoreThreadsAreEqual check for each action to verify that the threadStore being constructed "directly" and via the "ops approach" were equivalent. At this point we're pretty confident that they are, and so can safely remove some of the redundant code.

In this diff we remove the assertThreadStoreThreadsAreEqual and return the "processed" threadStore as the "updated" threadStore.


Depends on D4188

Test Plan: Hopefully the months of having assertThreadStoreThreadsAreEqual check in are sufficient.

Reviewers: palys-swm, def-au1t

Reviewed By: palys-swm

Subscribers: ashoat, Adrian, yayabosh

Differential Revision: https://phab.comm.dev/D4189

Details