Skip to content

Commit

Permalink
address comments
Browse files Browse the repository at this point in the history
  • Loading branch information
chloeYue committed Sep 25, 2024
1 parent 1d0e40d commit 0867c16
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions test/e2e/page-objects/pages/account-list-page.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@ class AccountListPage {

private closeAccountModalButton: string;

private closeEditLabelButton: string;

private createAccountButton: string;

private editableLabelButton: string;
Expand Down Expand Up @@ -55,7 +53,6 @@ class AccountListPage {
tag: 'button',
};
this.closeAccountModalButton = 'button[aria-label="Close"]';
this.closeEditLabelButton = 'button[aria-label="Close"]';
this.createAccountButton =
'[data-testid="multichain-account-menu-popover-action-button"]';
this.editableLabelButton = '[data-testid="editable-label-button"]';
Expand Down Expand Up @@ -108,11 +105,11 @@ class AccountListPage {
await this.driver.clickElement(this.editableLabelButton);
await this.driver.fill(this.editableLabelInput, newLabel);
await this.driver.clickElement(this.saveAccountLabelButton);
await this.driver.clickElement(this.closeEditLabelButton);
await this.driver.clickElement(this.closeAccountModalButton);
}

async closeAccountModal(): Promise<void> {
console.log(`Open add account modal in account list`);
console.log(`Close account modal in account list`);
await this.driver.clickElementAndWaitToDisappear(
this.closeAccountModalButton,
);
Expand Down

0 comments on commit 0867c16

Please sign in to comment.