Page MenuHomePhabricator

[SQLite] implement C++ user store ops
ClosedPublic

Authored by kamil on Nov 16 2023, 6:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 21, 8:12 PM
Unknown Object (File)
Sat, Apr 19, 12:05 PM
Unknown Object (File)
Sat, Apr 19, 3:48 AM
Unknown Object (File)
Fri, Apr 18, 7:45 AM
Unknown Object (File)
Fri, Apr 18, 3:12 AM
Unknown Object (File)
Thu, Apr 17, 7:51 PM
Unknown Object (File)
Sun, Apr 6, 6:24 AM
Unknown Object (File)
Mar 17 2025, 8:07 PM
Subscribers

Details

Summary

Code for native ops

Depends on D9909

Test Plan

Code for testing:

await commCoreModule.processUserStoreOperations([
  {
    type: 'replace_user',
    payload: {
      id: '1',
      userInfo: 'fdfd',
    },
  },
  {
    type: 'replace_user',
    payload: {
      id: '2',
      userInfo: 'fdfd',
    },
  },
]);

await commCoreModule.processUserStoreOperations([
  {
    type: 'replace_user',
    payload: {
      id: '2',
      userInfo: 'fdfd',
    },
  },
]);

await commCoreModule.processUserStoreOperations([
  {
    type: 'remove_users',
    payload: {
      ids: ['1'],
    },
  },
]);

await commCoreModule.processUserStoreOperations([
  {
    type: 'remove_all_users',
  },
]);

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Nov 16 2023, 9:12 AM
tomek added inline comments.
native/cpp/CommonCpp/NativeModules/PersistentStorageUtilities/DataStores/UserStore.cpp
19–20

Is it intentional to return an empty array? Is it implemented later in the stack?

This revision is now accepted and ready to land.Nov 17 2023, 7:46 AM
native/cpp/CommonCpp/NativeModules/PersistentStorageUtilities/DataStores/UserStore.cpp
19–20

it's implemented in next diff: D9911

This revision was automatically updated to reflect the committed changes.