-
Notifications
You must be signed in to change notification settings - Fork 70
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
🤯 Refactor add new proposal tests #4473
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
await userEvent.click(await modal.findByText('Sign transaction and change mode')) | ||
|
||
expect(await modal.findByText('It failed 🙀')) | ||
// within(document.body).getByText('Transaction failed') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The TransactionStatus
popup doesn't appear during the tx.proposalsDiscussion.changeThreadMode
await completeForms(canvasElement, step) | ||
await userEvent.click(modal.getByText('Sign transaction and Bind Staking Account')) | ||
|
||
expect(await modal.findByText('It failed 🙀')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The transaction failure modal appears behind the add proposal modal in all flows.
As a result even though this element is in the document it's not actually visible.
3103f9e
to
faaf30d
Compare
faaf30d
to
687ffcc
Compare
Part of:
proposals
tests intoPages > Proposals
stories #3559Link to the stories: https://pioneer-2-storybook-git-feature-create-proposal-tests-joystream.vercel.app/?path=/story/pages-proposals-proposallist-current--default