Skip to content

test: widen fee-payer server coverage#252

Open
brendanjryan wants to merge 1 commit intomainfrom
brendanryan/fee-payer-server-coverage
Open

test: widen fee-payer server coverage#252
brendanjryan wants to merge 1 commit intomainfrom
brendanryan/fee-payer-server-coverage

Conversation

@brendanjryan
Copy link
Copy Markdown
Collaborator

@brendanjryan brendanjryan commented Mar 27, 2026

Summary

  • widen fee-payer coverage in the Tempo session server suite across direct and fee-payer modes
  • teach the shared session test transaction helpers to build fee-payer open and top-up transactions
  • reduce middleware adapter coverage to one fee-payer smoke test per surface while keeping exhaustive fee-payer behavior in the core session and charge suites

Scope

This PR is now a single commit on top of main and only touches these related test files:

  • src/tempo/server/Session.test.ts
  • test/tempo/session.ts
  • src/middlewares/express.test.ts
  • src/middlewares/hono.test.ts
  • src/middlewares/elysia.test.ts
  • src/middlewares/nextjs.test.ts

Notes

  • adapter-level fee-payer coverage is intentionally kept to smoke tests
  • deeper fee-payer validation remains in the core suites, especially src/tempo/server/Session.test.ts, src/tempo/server/Charge.test.ts, and src/client/internal/Fetch.test.ts
  • no unrelated harness or setup changes remain in the PR

Testing

  • pnpm check:ci
  • pnpm check:types
  • pnpm test

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 27, 2026

Open in StackBlitz

npm i https://pkg.pr.new/mppx@252

commit: 0433ac6

@brendanjryan brendanjryan force-pushed the brendanryan/fee-payer-server-coverage branch from c20b6c8 to 001f97e Compare March 29, 2026 15:18
@brendanjryan brendanjryan force-pushed the brendanryan/fee-payer-server-coverage branch from 001f97e to 0433ac6 Compare March 29, 2026 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant