Skip to content
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.

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

cardano-testnet: use new output of "transaction submit" #6003

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smelc
Copy link
Contributor

@smelc smelc commented Oct 4, 2024

Description

Test this new additional output of transaction submit: IntersectMBO/cardano-cli#925

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated.
  • Self-reviewed the diff

then do
H.note_ "Expected output of \"transaction submit\" not to be empty"
H.failure
else pure $ last $ lines stdout
Copy link
Contributor

@Jimbo4350 Jimbo4350 Oct 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a poor way of obtaining the txid. The txid can be calculated from the tx itself and you should query the mempool to confirm indeed if it was submitted or not.

@smelc smelc force-pushed the smelc/test-cli-transaction-submit-new-output branch from 82c73e4 to 71bac0b Compare November 7, 2024 11:06
@smelc smelc force-pushed the smelc/test-cli-transaction-submit-new-output branch from 71bac0b to 6311baf Compare November 14, 2024 13:28
@smelc smelc added need-update next-update Needs a dependency to be updated to be merged and removed need-update labels Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next-update Needs a dependency to be updated to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants