Skip to content

Commit

Permalink
Attempt to fix settings not clickable
Browse files Browse the repository at this point in the history
  • Loading branch information
DDDDDanica committed Feb 5, 2025
1 parent e003d9b commit 60cf105
Show file tree
Hide file tree
Showing 4 changed files with 34 additions and 5 deletions.
1 change: 1 addition & 0 deletions test/e2e/page-objects/pages/header-navbar.ts
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ class HeaderNavbar {
console.log('Open settings page');
await this.openThreeDotMenu();
await this.driver.clickElement(this.settingsButton);
await this.driver.delay(5000);
}

async clickSwitchNetworkDropDown(): Promise<void> {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ class SecurityAndPrivacySettings {
const headerNavbar = new HeaderNavbar(this.driver);
await headerNavbar.openSettingsPage();
const settingsPage = new SettingsPage(this.driver);
await this.driver.delay(5000);
await settingsPage.check_pageIsLoaded();
await settingsPage.goToPrivacySettings();
await this.check_pageIsLoaded();
Expand Down
1 change: 1 addition & 0 deletions test/e2e/page-objects/pages/settings/settings-page.ts
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ class SettingsPage {

async check_pageIsLoaded(): Promise<void> {
try {
await this.driver.delay(2000);
await this.driver.waitForSelector(this.settingsPageTitle);
} catch (e) {
console.log('Timeout while waiting for Settings page to be loaded', e);
Expand Down
36 changes: 31 additions & 5 deletions test/e2e/tests/metrics/segment-user-traits.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -166,12 +166,13 @@ describe('Segment User Traits', function () {
);
});

it('should stop sending identify events when user disables metrics in privacy settings after opting in during onboarding', async function () {
it.only('should stop sending identify events when user disables metrics in privacy settings after opting in during onboarding', async function () {
await withFixtures(
{
fixtures: new FixtureBuilder({ onboarding: true })
.withMetaMetricsController({
metaMetricsId: MOCK_META_METRICS_ID,
participateInMetaMetrics: true,
})
.build(),
title: this.test?.fullTitle(),
Expand All @@ -182,18 +183,43 @@ describe('Segment User Traits', function () {
await completeCreateNewWalletOnboardingFlow({
driver,
participateInMetaMetrics: true,
dataCollectionForMarketing: true,
});
events = await getEventPayloads(driver, mockedEndpoints);
assert.equal(events.length, 1);
const securityAndPrivacySettings = new SecurityAndPrivacySettings(
driver,
);
await securityAndPrivacySettings.navigateToPage();
await securityAndPrivacySettings.toggleParticipateInMetaMetrics();
events = await getEventPayloads(driver, mockedEndpoints);
assert.equal(events.length, 1);
},
);
// await withFixtures(
// {
// fixtures: new FixtureBuilder({ onboarding: true })
// .withMetaMetricsController({
// metaMetricsId: MOCK_META_METRICS_ID,
// participateInMetaMetrics: false,
// })
// .build(),
// title: this.test?.fullTitle(),
// testSpecificMock: mockSegment,
// },
// async ({ driver, mockedEndpoint: mockedEndpoints }) => {
// let events = [];
// await completeCreateNewWalletOnboardingFlow({
// driver,
// participateInMetaMetrics: false,
// dataCollectionForMarketing: true,
// });
// events = await getEventPayloads(driver, mockedEndpoints);
// assert.equal(events.length, 1);
// const securityAndPrivacySettings = new SecurityAndPrivacySettings(
// driver,
// );
// await securityAndPrivacySettings.navigateToPage();
// await securityAndPrivacySettings.toggleParticipateInMetaMetrics();
// events = await getEventPayloads(driver, mockedEndpoints);
// assert.equal(events.length, 1);
// },
// );
});
});

0 comments on commit 60cf105

Please sign in to comment.