HomePhabricator
Diffusion Comm 7695313d1dab

[lib] Move `invariant` in `changeThreadSettings` above `callServerEndpoint`

Description

[lib] Move invariant in changeThreadSettings above callServerEndpoint

Summary: Unless I'm mistaken, it makes more sense to check this before we make a call to the keyserver?

Test Plan: Things continue to work as expected, we don't expect to see this invariant hit.

Reviewers: ashoat, ginsu

Reviewed By: ashoat

Subscribers: tomek

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

Details

Provenance
atulAuthored on Apr 5 2023, 7:08 AM
Reviewer
ashoat
Differential Revision
D7305: [lib] Move `invariant` in `changeThreadSettings` above `callServerEndpoint`
Parents
rCOMMeab4e254cf47: [lib] Introduce basic unit tests for `ThreadOpsUtils`
Branches
Unknown
Tags
Unknown