Skip to content

feat: wallet start tests #929

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
tuliomir wants to merge 21 commits into
base: master
Choose a base branch
from
Open

feat: wallet start tests #929

tuliomir wants to merge 21 commits into from

Conversation

@tuliomir
Copy link
Contributor

@tuliomir tuliomir commented Sep 11, 2025

Acceptance Criteria

  • The /start endpoint should be fully tested by the new integration suite

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@tuliomir
wip: only ws-daemon container doesnt run
a0eeb41
@tuliomir
wip: working container, broken sql
5df32a5
@tuliomir
feat: adds migration argument to build
a58d7c7
@tuliomir
fix: migrator now its own container
3e569a4
@tuliomir
fix: fullnode configs
c30c142
@tuliomir
feat: dynamic fullnode ids for daemon
5d16d5d
@tuliomir
feat: dynamic fullnode ids for daemon
eae48dc
@tuliomir
wip: placeholder env variables for the serverless
0013234
@tuliomir
fix: serverless build context
a93b9a1
@tuliomir
wip: initial tests
0804f95
@tuliomir
fix: removes hardcoded build context
df649c8 
This will be completely removed as soon as we have the actual images being built and published
@tuliomir
refactor: removes unnecessary debug code
5d7b811
@tuliomir
fix: fullnode port for ws-serverless
be12e71
@tuliomir
feat: improvements on the docker compose
5b9b1fe
@tuliomir
revert: ws-migrator is still needed
09f8a77
@tuliomir
docs: improves compose comments
c3aed28
@tuliomir
refactor: uses wallet service images instead of builds
421640a
@tuliomir
refactor: uses wallet service-service image
a860c76
@tuliomir
fix: serverless env configs
558dd4b
@tuliomir
chore: simplifying initial test suite
600d61e
@tuliomir
feat: wallet start tests
c9cc199
@tuliomir tuliomir self-assigned this Sep 11, 2025
@tuliomir tuliomir added the tests label Sep 11, 2025
@tuliomir tuliomir moved this from Todo to In Progress (Done) in Hathor Network Sep 11, 2025
Base automatically changed from test/wallet-service to master September 30, 2025 16:36
@tuliomir tuliomir moved this from In Progress (Done) to In Progress (WIP) in Hathor Network Oct 1, 2025
@tuliomir tuliomir moved this from In Progress (WIP) to Todo in Hathor Network Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andreabadesso andreabadesso Awaiting requested review from andreabadesso

@pedroferreira1 pedroferreira1 Awaiting requested review from pedroferreira1 pedroferreira1 is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@tuliomir tuliomir

Labels

tests

Projects

Hathor Network
Status: Todo

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tuliomir