Page MenuHomePhabricator

[SQLite] implement C++ keyserver store ops
ClosedPublic

Authored by kamil on Jan 22 2024, 4:32 AM.
Tags
None
Referenced Files
F3157081: D10776.diff
Tue, Nov 5, 7:16 PM
Unknown Object (File)
Thu, Oct 24, 1:43 AM
Unknown Object (File)
Wed, Oct 16, 5:10 PM
Unknown Object (File)
Mon, Oct 14, 7:05 PM
Unknown Object (File)
Mon, Oct 14, 7:04 PM
Unknown Object (File)
Mon, Oct 14, 7:04 PM
Unknown Object (File)
Mon, Oct 14, 7:04 PM
Unknown Object (File)
Mon, Oct 14, 7:04 PM
Subscribers

Details

Summary

Code for native ops

Depends on D10775

Test Plan

Call this code:

await commCoreModule.processKeyserverStoreOperations([
    {
      type: 'replace_keyserver',
      payload: {
        id: '1',
        keyserverInfo: 'fdfd',
      },
    },
    {
      type: 'replace_keyserver',
      payload: {
        id: '2',
        keyserverInfo: 'fdfd',
      },
    },
  ]);

  await commCoreModule.processKeyserverStoreOperations([
    {
      type: 'replace_keyserver',
      payload: {
        id: '2',
        keyserverInfo: 'fdfd45454545',
      },
    },
  ]);

  await commCoreModule.processKeyserverStoreOperations([
    {
      type: 'remove_keyserver',
      payload: {
        id: '1',
      },
    },
  ]);

  await commCoreModule.processKeyserverStoreOperations([
    {
      type: 'remove_all_keyservers',
    },
  ]);

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.Jan 22 2024, 6:39 AM
ashoat added a subscriber: will.

Might be good for @will to review this stack in preparation for his projects next month

This revision is now accepted and ready to land.Jan 24 2024, 9:25 AM

update to allow removing array of keyservers

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp
1340 ↗(On Diff #36219)

this formatting is a rebase issue

This revision was automatically updated to reflect the committed changes.