Skip to content

ci: add infra setup for integ tests#211

Merged
pranishnepal merged 1 commit into
masterfrom
WAL-1507
May 29, 2026
Merged

ci: add infra setup for integ tests#211
pranishnepal merged 1 commit into
masterfrom
WAL-1507

Conversation

@pranishnepal
Copy link
Copy Markdown
Contributor

What

This PR adds the Docker foundation for integration testing advanced-wallets. Previously, all tests stubbed the AWM client entirely using Nock, meaning the MBE → AWM HTTP contract was never exercised automatically. This PR establishes the infrastructure to run integ tests where AWM and MBE start as actual HTTP servers and communicate over real TCP sockets.

This commit adds a validation test to verify the changes. Follow up PRs will add tests and expand on this PR.

Ticket: WAL-1504, WAL-1507

Testing

  • Without docker: npm run test:integration
  • With docker: npm run docker:test:integration

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 29, 2026

WAL-1507

Adds the Docker foundation for integration testing advanced-wallets.
Previously, all tests stubbed the AWM client entirely using Nock, meaning
the MBE → AWM HTTP contract was never exercised automatically. This PR
establishes the infrastructure to run integ tests where AWM and
MBE start as actual HTTP servers and communicate over real TCP sockets.

This commit adds a validation test to verify the changes.
Follow up PRs will add tests and expand on this PR.

Ticket: WAL-1504, WAL-1507
@pranishnepal pranishnepal marked this pull request as ready for review May 29, 2026 16:26
@pranishnepal pranishnepal requested a review from a team as a code owner May 29, 2026 16:26
@pranishnepal pranishnepal requested a review from mrdanish26 May 29, 2026 16:26
@pranishnepal pranishnepal merged commit f46a4c6 into master May 29, 2026
20 checks passed
@pranishnepal pranishnepal deleted the WAL-1507 branch May 29, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants