-
Notifications
You must be signed in to change notification settings - Fork 4
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
Implement [e2e-d08]valWithdrawRewardsAsDelegateToAnother.feature #829
Conversation
…ther-address # Conflicts: # cypress/integration/[d07]-valWithdrawRewardsAsDelegate.feature
# TODO improve after getting more validator issues/789 | ||
* I see TGD balance "0.1" | ||
* I close wallet dialog modal | ||
# TODO improve after getting more validator issues/789 | ||
* I use "randomMnemonic07" to make query and check balance of this address "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.
I know this is not an accurate and not a good solution but we are going to improve it after I get more validator with Rewards
#789
# TODO improve after getting more validator issues/789 | ||
* I see TGD balance "0" | ||
* I close wallet dialog modal | ||
# TODO improve after getting more validator issues/789 |
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.
@@ -16,7 +16,7 @@ Given("Go to Trusted Circle page", () => { | |||
|
|||
When("I click on Add Trusted Circle button", () => { | |||
cy.url().should("not.include", "/trustedcircle/undefined"); //probably a bug of the App | |||
cy.findByText(/Add Trusted Circle/i).click(); | |||
cy.get('[data-cy="trusted-circle-page-add-trusted-circle-button"]').click(); |
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 hope it will help to prevent it
also did some related adjustments in this PR
FYI 30b30ef
Maybe we should wait to merge #832 before and then adapt this one to auto code ID |
i am fine, will just clone this branch for my next task |
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.
Good! We can merge as soon as the merge conflicts are solved
…ther-address # Conflicts: # cypress/steps-definition/TrustedCircles.ts
…ther-address # Conflicts: # src/__tests__/integration/happy-paths/trustedCircle.test.ts
…ther-address # Conflicts: # cypress/integration/[a01]-tgdBalance.feature # cypress/integration/[a04]-sendTgd.feature # cypress/integration/[c03]-withdrawRewardsAsDelegate.feature # cypress/integration/[d05]-valWithdrawMyRewards.feature # cypress/integration/[d06]-valWithdrawMyRewardsToAnother.feature # cypress/integration/[d07]-valWithdrawRewardsAsDelegate.feature # cypress/steps-definition/ConnectWalletModal.ts # cypress/steps-definition/MainNavigationMenu.ts
closes: #827