-
Notifications
You must be signed in to change notification settings - Fork 322
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
[QAA-168] Adding new send test cases #7938
base: develop
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 5 Skipped Deployments
|
Wiz Scan Summary
To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension. |
@@ -39,9 +40,18 @@ export class SendModal extends Modal { | |||
await this.recipientInput.fill(recipient); | |||
} | |||
|
|||
@step("Enter recipient as $0") |
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.
"$0" will be displayed as Object object in report because transaction is not a string
} | ||
|
||
for (const transaction of transactionsAddressInvalid) { | ||
test.describe("Send funds setp 1 (Recipient) - negative cases (Button disabled)", () => { |
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.
typo
@@ -314,3 +417,63 @@ test.describe("Verify send max user flow", () => { | |||
}, | |||
); | |||
}); | |||
|
|||
for (const transaction of transactionAddressValid) { | |||
test.describe("Send funds step 1 (Recipient) - positive cases (Button enabled)", () => { |
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.
I think this should be a part of full e2e scenario where we actually send funds (even if without broadcasting) because creating test just for one check that button is enabled is an overkill
✅ Checklist
npx changeset
was attached.📝 Description
Adding new send test scénarios:
CI Run: https://github.com/LedgerHQ/ledger-live/actions/runs/11069828999 (fail = getAppAndVersion)
❓ Context
🧐 Checklist for the PR Reviewers