Page MenuHomePhabricator

[keyserver][lib] Update freshMessageStore to return currentAsOf for all keyservers
ClosedPublic

Authored by inka on Nov 14 2023, 2:18 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 2:30 PM
Unknown Object (File)
Sat, Dec 14, 4:12 PM
Unknown Object (File)
Fri, Dec 6, 7:19 AM
Unknown Object (File)
Tue, Nov 26, 8:54 AM
Unknown Object (File)
Tue, Nov 26, 1:16 AM
Unknown Object (File)
Sun, Nov 24, 4:35 PM
Unknown Object (File)
Oct 28 2024, 12:14 AM
Unknown Object (File)
Oct 15 2024, 5:16 PM
Subscribers

Details

Summary

issue: https://linear.app/comm/issue/ENG-5722/update-freshmessagestore-function
freshMessageStore should return currentAsOf for all keyservers

Test Plan

Tested that after the setInitialReduxState action the currentAsOf field is correct - a "map" from keyserver id to number.
This tests this function, because freshMessageStore is called by the keyserver in getInitialReduxStateResponder which returns the initial state. The states from all keyservers are then merged in getInitialReduxState, and the returnd value is dispatched with setInitialReduxState action.
I also logged to the console the value returned from freshMessageStore in getInitialReduxStateResponder and checked that it is correct.
I also logged in and checked that the currentAsOf in messageStore is correct after logInActionTypes.success

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka requested review of this revision.Nov 14 2023, 2:38 AM
This revision is now accepted and ready to land.Nov 14 2023, 7:04 AM
keyserver/src/responders/redux-state-responders.js
171

For future reference, the new Flow requires an annotation here

keyserver/src/responders/redux-state-responders.js
171

Actually, hmmm... I guess this isn't always required, as I'm not seeing any errors after rebasing. Sorry for the distraction!