Page MenuHomePhabricator

[lib] Update `threadStore` with `threadStoreOps` for logOut.success/deleteAccount.success/setNewSession actions
ClosedPublic

Authored by atul on Jun 2 2022, 11:25 AM.
Tags
None
Referenced Files
F3393086: D4186.id13298.diff
Sat, Nov 30, 11:26 AM
F3388323: D4186.id.diff
Fri, Nov 29, 1:44 PM
Unknown Object (File)
Wed, Nov 27, 12:23 PM
Unknown Object (File)
Sat, Nov 23, 11:54 PM
Unknown Object (File)
Sat, Nov 23, 8:25 PM
Unknown Object (File)
Thu, Nov 7, 11:08 PM
Unknown Object (File)
Thu, Nov 7, 10:17 PM
Unknown Object (File)
Thu, Nov 7, 7:56 PM

Details

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 D4185

Test Plan

Hopefully the months of having assertThreadStoreThreadsAreEqual check in are sufficient.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage