Page MenuHomePhabricator

[lib] Remove `assertMessageStoreMessagesAreEqual(...)` from `message-reducer`
ClosedPublic

Authored by atul on Jun 22 2022, 3:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 4:52 PM
Unknown Object (File)
Sun, Dec 15, 4:52 PM
Unknown Object (File)
Sun, Dec 15, 4:52 PM
Unknown Object (File)
Sun, Dec 15, 4:52 PM
Unknown Object (File)
Sun, Dec 15, 4:52 PM
Unknown Object (File)
Fri, Dec 13, 11:26 PM
Unknown Object (File)
Nov 18 2024, 10:49 AM
Unknown Object (File)
Nov 8 2024, 2:46 PM
Subscribers

Details

Summary

We no longer need to assert that the "old approach" and the "ops approach" are equal... because we're removing the "old approach" altogether.

This diff

  • removes the last usage of assertMessageStoreMessagesAreEqual from mergeNewMessages
  • removes assertMessageStoreMessagesAreEqual
  • removes imports that assertMessageStoreMessagesAreEqual depended on
  • removes actionType argument from mergeNewMessages(..._) since it was only used by assertMessageStoreMessagesAreEqual from within mergeNewMessages

Depends on D4332

Test Plan

This should be a straightforward refactor given we've had assertMessageStoreMessagesAreEqual in place for some time now.

Diff Detail

Repository
rCOMM Comm
Branch
june21 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul requested review of this revision.Jun 22 2022, 3:07 PM
This revision is now accepted and ready to land.Jun 23 2022, 3:58 AM

rebase after cherrypicking and before landing