Skip to content

Commit d59ee66

Browse files
Merge pull request #21237 from wireapp/jl/refactor-dataShareConsent-modal
test: remove "dataShareConsent" modal [WPB-22420]
2 parents 01b5a9c + 17ff52f commit d59ee66

4 files changed

Lines changed: 2 additions & 37 deletions

File tree

apps/webapp/test/e2e_tests/pageManager/index.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ import {AcknowledgeModal} from './webapp/modals/acknowledge.modal';
3030
import {AppLockModal} from './webapp/modals/appLock.modal';
3131
import {ConfirmModal} from './webapp/modals/confirm.modal';
3232
import {ConfirmLogoutModal} from './webapp/modals/confirmLogout.modal';
33-
import {DataShareConsentModal} from './webapp/modals/dataShareConsent.modal';
3433
import {DetailViewModal} from './webapp/modals/detailView.modal';
3534
import {LeaveConversationModal} from './webapp/modals/leaveConversation.modal';
3635
import {OptionModal} from './webapp/modals/option.modal';
@@ -194,8 +193,6 @@ export class PageManager {
194193
conversationJoin: () => this.getOrCreate('webapp.pages.conversationJoin', () => ConversationJoinPage(this.page)),
195194
},
196195
modals: {
197-
dataShareConsent: () =>
198-
this.getOrCreate('webapp.modals.dataShareConsent', () => new DataShareConsentModal(this.page)),
199196
appLock: () => this.getOrCreate('webapp.modals.appLock', () => new AppLockModal(this.page)),
200197
userProfile: () => this.getOrCreate('webapp.modals.userProfile', () => new UserProfileModal(this.page)),
201198
confirmLogout: () => this.getOrCreate('webapp.modals.confirmLogout', () => new ConfirmLogoutModal(this.page)),

apps/webapp/test/e2e_tests/pageManager/webapp/modals/dataShareConsent.modal.ts

Lines changed: 0 additions & 32 deletions
This file was deleted.

apps/webapp/test/e2e_tests/specs/Authentication/authentication.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ test.describe('Authentication', () => {
5353
const pageManager = await PageManager.from(createPage(withLogin(createUser({disableTelemetry: false}))));
5454
const {modals} = pageManager.webapp;
5555

56-
await expect(modals.dataShareConsent().modalTitle).toBeVisible();
56+
await expect(modals.confirm().modalTitle).toBeVisible();
5757
},
5858
);
5959

apps/webapp/test/e2e_tests/specs/CriticalFlow/personalAccountLifecycle-TC-8638.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ test('Personal Account Lifecycle', {tag: ['@TC-8638', '@crit-flow-web']}, async
6868

6969
await test.step('Personal user A declines sending anonymous usage data', async () => {
7070
const {modals} = pageManagerA.webapp;
71-
await modals.dataShareConsent().clickDecline();
71+
await modals.confirm().cancelButton.click();
7272
});
7373

7474
await test.step('Personal user A checks that username was set correctly', async () => {

0 commit comments

Comments
 (0)