Skip to content

test: add frontend smoke coverage#35

Open
Sikkra wants to merge 1 commit into
BitgesellOfficial:mainfrom
Sikkra:codex/frontend-smoke-test
Open

test: add frontend smoke coverage#35
Sikkra wants to merge 1 commit into
BitgesellOfficial:mainfrom
Sikkra:codex/frontend-smoke-test

Conversation

@Sikkra
Copy link
Copy Markdown

@Sikkra Sikkra commented May 19, 2026

Summary

Adds a focused React Testing Library smoke test for the frontend app so npm test -- --watchAll=false runs a real test suite instead of exiting with No tests found.

This covers the bridge shell, both conversion tabs, and the wallet connection guidance rendered through the existing MetaMask hook flow.

Verification

  • npm install in app/
  • CI=true npm test -- --watchAll=false in app/ (1 passed)
  • git diff --check

Build note: npm run build on Node.js v24 still requires the legacy OpenSSL provider for this older react-scripts@4 stack; node --openssl-legacy-provider ./node_modules/react-scripts/scripts/build.js succeeds with existing warnings.

Related: #7
Program reference: BitgesellOfficial/bitgesell#81

USDT-TRC20 payout address if accepted: TPwPFww7zxXFQ7zugo22gktQhckWVarRqi

@MyTH-zyxeon
Copy link
Copy Markdown

Maintainer-facing review assist for the #81 bounty queue:

This is a narrow, useful testing PR: it turns the frontend from "No tests found" into a real React Testing Library smoke check that covers the bridge shell, both conversion tabs, and the unavailable-wallet guidance path.

Acceptance checks I would use before merge:

No payment action or claim from me here; just a focused review pass to help maintainers evaluate the PR.

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