Skip to content

Commit d7848fb

Browse files
authored
test(extension): maintenance 8 apr fix duplicated dom selectors (#1044)
* test(extension): fix duplicated modals issue * test(extension): fix duplicated modals selectors
1 parent 320522c commit d7848fb

File tree

6 files changed

+24
-21
lines changed

6 files changed

+24
-21
lines changed

packages/e2e-tests/src/elements/NFTs/DeleteFolderModal.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
class DeleteFolderModal {
2-
private CONTAINER = '.ant-modal-content';
2+
private CONTAINER = '.ant-modal-wrap:not([style="display: none;"]) .ant-modal-content';
33
private TITLE = '[data-testid="create-folder-modal-title"]';
44
private DESCRIPTION = '[data-testid="create-folder-modal-description"]';
55
private CANCEL_BUTTON = '[data-testid="delete-folder-modal-cancel"]';
@@ -10,19 +10,19 @@ class DeleteFolderModal {
1010
}
1111

1212
get title() {
13-
return $(this.TITLE);
13+
return this.container.$(this.TITLE);
1414
}
1515

1616
get description() {
17-
return $(this.DESCRIPTION);
17+
return this.container.$(this.DESCRIPTION);
1818
}
1919

2020
get cancelButton() {
21-
return $(this.CANCEL_BUTTON);
21+
return this.container.$(this.CANCEL_BUTTON);
2222
}
2323

2424
get confirmButton() {
25-
return $(this.CONFIRM_BUTTON);
25+
return this.container.$(this.CONFIRM_BUTTON);
2626
}
2727

2828
async clickCancelButton() {

packages/e2e-tests/src/elements/NFTs/youllHaveToStartAgainModal.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
import { ChainablePromiseElement } from 'webdriverio';
33

44
class YoullHaveToStartAgainModal {
5-
private CONTAINER = '.ant-modal-content';
5+
private CONTAINER = '.ant-modal-wrap:not([style="display: none;"]) .ant-modal-content';
66
private TITLE = '[data-testid="create-folder-modal-title"]';
77
private DESCRIPTION = '[data-testid="create-folder-modal-description"]';
88
private CANCEL_BUTTON = '[data-testid="create-folder-modal-cancel"]';
@@ -13,19 +13,19 @@ class YoullHaveToStartAgainModal {
1313
}
1414

1515
get title(): ChainablePromiseElement<WebdriverIO.Element> {
16-
return $(this.TITLE);
16+
return this.container.$(this.TITLE);
1717
}
1818

1919
get description(): ChainablePromiseElement<WebdriverIO.Element> {
20-
return $(this.DESCRIPTION);
20+
return this.container.$(this.DESCRIPTION);
2121
}
2222

2323
get cancelButton(): ChainablePromiseElement<WebdriverIO.Element> {
24-
return $(this.CANCEL_BUTTON);
24+
return this.container.$(this.CANCEL_BUTTON);
2525
}
2626

2727
get agreeButton(): ChainablePromiseElement<WebdriverIO.Element> {
28-
return $(this.AGREE_BUTTON);
28+
return this.container.$(this.AGREE_BUTTON);
2929
}
3030
}
3131

packages/e2e-tests/src/elements/addressbook/DeleteAddressModal.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
class DeleteAddressModal {
2-
private CONTAINER = '.ant-modal-content';
2+
private CONTAINER = '.ant-modal-wrap:not([style="display: none;"]) .ant-modal-content';
33
private TITLE = '[data-testid="delete-address-modal-title"]';
44
private DESCRIPTION = '[data-testid="delete-address-modal-description"]';
55
private CANCEL_BUTTON = '[data-testid="delete-address-modal-cancel"]';
@@ -10,19 +10,19 @@ class DeleteAddressModal {
1010
}
1111

1212
get title() {
13-
return $(this.TITLE);
13+
return this.container.$(this.TITLE);
1414
}
1515

1616
get description() {
17-
return $(this.DESCRIPTION);
17+
return this.container.$(this.DESCRIPTION);
1818
}
1919

2020
get cancelButton() {
21-
return $(this.CANCEL_BUTTON);
21+
return this.container.$(this.CANCEL_BUTTON);
2222
}
2323

2424
get deleteAddressButton() {
25-
return $(this.DELETE_ADDRESS_BUTTON);
25+
return this.container.$(this.DELETE_ADDRESS_BUTTON);
2626
}
2727
}
2828

packages/e2e-tests/src/elements/modal.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
import { ChainablePromiseElement } from 'webdriverio';
33

44
class Modal {
5-
private CONTAINER = '.ant-modal-content';
5+
private CONTAINER = '.ant-modal-wrap:not([style="display: none;"]) .ant-modal-content';
66
private TITLE = '[data-testid="delete-address-modal-title"]';
77
private DESCRIPTION = '[data-testid="delete-address-modal-description"]';
88
private CANCEL_BUTTON = '[data-testid="delete-address-modal-cancel"]';
@@ -13,19 +13,19 @@ class Modal {
1313
}
1414

1515
get title(): ChainablePromiseElement<WebdriverIO.Element> {
16-
return $(this.TITLE);
16+
return this.container.$(this.TITLE);
1717
}
1818

1919
get description(): ChainablePromiseElement<WebdriverIO.Element> {
20-
return $(this.DESCRIPTION);
20+
return this.container.$(this.DESCRIPTION);
2121
}
2222

2323
get cancelButton(): ChainablePromiseElement<WebdriverIO.Element> {
24-
return $(this.CANCEL_BUTTON);
24+
return this.container.$(this.CANCEL_BUTTON);
2525
}
2626

2727
get confirmButton(): ChainablePromiseElement<WebdriverIO.Element> {
28-
return $(this.CONFIRM_BUTTON);
28+
return this.container.$(this.CONFIRM_BUTTON);
2929
}
3030

3131
buttonWithText(value: string): ChainablePromiseElement<WebdriverIO.Element> {

packages/e2e-tests/src/pageobject/settingsExtendedPageObject.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ class SettingsExtendedPageObject {
169169
}
170170

171171
async multiAddressModalConfirm() {
172-
if (await Modal.confirmButton.isDisplayed()) {
172+
if (await Modal.container.isDisplayed()) {
173173
expect(await Modal.confirmButton.getText()).to.equal(await t('modals.beta.button', 'staking'));
174174
await Modal.confirmButton.click();
175175
}

packages/e2e-tests/src/steps/commonSteps.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,15 +39,18 @@ import settingsExtendedPageObject from '../pageobject/settingsExtendedPageObject
3939
import consoleManager from '../utils/consoleManager';
4040
import consoleAssert from '../assert/consoleAssert';
4141
import { addAndActivateWalletInRepository, clearWalletRepository } from '../fixture/walletRepositoryInitializer';
42+
import MainLoader from '../elements/MainLoader';
4243

4344
Given(/^Lace is ready for test$/, async () => {
45+
await MainLoader.waitUntilLoaderDisappears();
4446
await settingsExtendedPageObject.waitUntilSyncingModalDisappears();
4547
await settingsExtendedPageObject.multiAddressModalConfirm();
4648
await tokensPageObject.waitUntilCardanoTokenLoaded();
4749
await settingsExtendedPageObject.closeWalletSyncedToast();
4850
});
4951

5052
Then(/^Lace is loaded properly$/, async () => {
53+
await MainLoader.waitUntilLoaderDisappears();
5154
await settingsExtendedPageObject.waitUntilSyncingModalDisappears();
5255
await tokensPageObject.waitUntilCardanoTokenLoaded();
5356
});

0 commit comments

Comments
 (0)