Skip to content

[New Tx Flow] Update buildTransaction.test.ts#2005

Merged
jeesunikim merged 1 commit intofeature/new-tx-flowfrom
new-tx-flow-test-buildtransaction
Apr 3, 2026
Merged

[New Tx Flow] Update buildTransaction.test.ts#2005
jeesunikim merged 1 commit intofeature/new-tx-flowfrom
new-tx-flow-test-buildtransaction

Conversation

@jeesunikim
Copy link
Copy Markdown
Contributor

  • Reflect new names in tests (Clear all message, soroban one-op warning text) and new selector (Save transaction in footer)
  • Fix SorobanTransactionXdr reading isValid from the old store
  • Remove Prepare Transaction related test + simulate mock

… tests

SorobanTransactionXdr was reading isValid from the main store instead of
the flow store, so Soroban XDR never rendered on the build step. Updated
e2e tests to match new flow: save button selector, Clear all rename,
Soroban one-op warning text, and replaced Prepare button + simulate mock
with Next button assertions for extend/restore footprint operations.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-project-automation github-project-automation bot moved this to Backlog (Not Ready) in DevX Apr 3, 2026
@jeesunikim jeesunikim requested a review from quietbits April 3, 2026 00:35
@stellar-jenkins-ci
Copy link
Copy Markdown

@jeesunikim jeesunikim merged commit dd3d77e into feature/new-tx-flow Apr 3, 2026
4 checks passed
@jeesunikim jeesunikim deleted the new-tx-flow-test-buildtransaction branch April 3, 2026 16:17
@github-project-automation github-project-automation bot moved this from Backlog (Not Ready) to Done in DevX Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants