Page MenuHomePhabricator
Differential D13702

[alias-server] Initialize serverState database collection with read and write helper methods
ClosedPublic
Actions

Authored by bytesofman on Apr 13 2023, 14:14.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC545ea3b814c5: [alias-server] Initialize serverState database collection with read and write…
Summary

T3060

Depends on D13701

Initialize serverState collection with read and write helper methods.

serverState will be used to determine how many tx from chronik tx history call have not yet been processed by the server.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
alias-server-add-server-state
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 23287
Build 46193: Build Diffalias-server-tests
Build 46192: arc lint + arc unit

Event Timeline

bytesofman created this revision.Apr 13 2023, 14:14
Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 13 2023, 14:14
bytesofman requested review of this revision.Apr 13 2023, 14:14
bytesofman updated this revision to Diff 39691.Apr 13 2023, 14:18

Add new unit test for app restart

Harbormaster completed remote builds in B23283: Diff 39690.Apr 13 2023, 14:20
Harbormaster completed remote builds in B23284: Diff 39691.
Fabien added a subscriber: Fabien.Apr 13 2023, 14:35
Fabien added inline comments.
apps/alias-server/src/db.js
60 ↗(On Diff #39691)

why do you need to convert to an array if there is a single element ?

bytesofman updated this revision to Diff 39698.Apr 13 2023, 16:00

Use .next() instead of .toArray()

bytesofman marked an inline comment as done.Apr 13 2023, 16:01
bytesofman added inline comments.
apps/alias-server/src/db.js
60 ↗(On Diff #39691)

Looks like there is a way to get just the doc and not the array. patched.

https://www.mongodb.com/docs/drivers/node/current/usage-examples/find/

Harbormaster completed remote builds in B23287: Diff 39698.Apr 13 2023, 16:01
Fabien accepted this revision.Apr 13 2023, 16:02
This revision is now accepted and ready to land.Apr 13 2023, 16:02
Closed by commit rABC545ea3b814c5: [alias-server] Initialize serverState database collection with read and write… (authored by bytesofman). · Explain WhyApr 13 2023, 16:06
This revision was automatically updated to reflect the committed changes.
bytesofman marked an inline comment as done.
bytesofman added a commit: rABC545ea3b814c5: [alias-server] Initialize serverState database collection with read and write….

Revision Contents
Changeset List

PathSize
apps/
alias-server/
1 line
src/
81 lines
test/
66 lines

Diff 39698

View Options

apps/alias-server/config.js

Loading...
View Options

apps/alias-server/src/db.js

Loading...
View Options

apps/alias-server/test/dbTests.js

Loading...