Phantom support for Playwright

wallets/phantom/src/playwright/Phantom.ts

@@ -2,7 +2,7 @@ import type { BrowserContext, Page } from '@playwright/test'
 import type { GasSettings } from '../type/GasSettings'
 import type { Networks } from '../type/Networks'
 import { PhantomAbstract } from '../type/PhantomAbstract'
-import { CrashPage, HomePage, NotificationPage, OnboardingPage, UnlockPage } from './pages'
+import { HomePage, NotificationPage, OnboardingPage, UnlockPage } from './pages'
 
 const NO_EXTENSION_ID_ERROR = new Error('Phantom extensionId is not set')
 
@@ -16,14 +16,6 @@ const NO_EXTENSION_ID_ERROR = new Error('Phantom extensionId is not set')
  * @extends PhantomAbstract
  */
 export class Phantom extends PhantomAbstract {
-  /**
-   * This property can be used to access selectors for the crash page.
-   *
-   * @public
-   * @readonly
-   */
-  readonly crashPage: CrashPage
-
   /**
    * This property can be used to access selectors for the onboarding page.
    *
@@ -72,7 +64,6 @@ export class Phantom extends PhantomAbstract {
   ) {
     super(password, extensionId)
 
-    this.crashPage = new CrashPage()
     this.onboardingPage = new OnboardingPage(page)
     this.unlockPage = new UnlockPage(page)
     this.homePage = new HomePage(page)

wallets/phantom/src/playwright/fixtures/phantomFixtures.ts

@@ -1,5 +1,5 @@
 import path from 'node:path'
-import { type Page, chromium, expect } from '@playwright/test'
+import { type Page, chromium } from '@playwright/test'
 import { test as base } from '@playwright/test'
 import {
   CACHE_DIR_NAME,
@@ -19,8 +19,6 @@ type PhantomFixtures = {
   phantom: Phantom
   extensionId: string
   phantomPage: Page
-  aavePage: Page
-  solanaSandboxPage: Page
 }
 
 // If setup phantomPage in a fixture, browser does not handle it properly (even if ethereum.isConnected() is true, it's not reflected on the page).
@@ -107,47 +105,6 @@ export const phantomFixtures = (walletSetup: ReturnType<typeof defineWalletSetup
     page: async ({ page }, use) => {
       await page.goto('/')
 
-      await use(page)
-    },
-    aavePage: async ({ page, phantom }, use) => {
-      await page.goto('https://app.aave.com')
-
-      await phantom.toggleTestnetMode()
-
-      await page.locator('button#settings-button').click()
-      await page.locator('li:has-text("Testnet mode")').click()
-      await expect(page.getByRole('button', { name: 'TESTNET' })).toBeVisible()
-
-      await page.getByRole('button', { name: 'Connect wallet' }).first().click()
-      await page.getByRole('button', { name: 'Phantom' }).click()
-
-      await phantom.connectToDapp()
-      await phantom.page.waitForTimeout(1_000)
-      await phantom.closeUnsupportedNetworkWarning()
-
-      await expect(page.getByText('0xf3...2266'), '"0xf3...2266" should be visible').toBeVisible()
-
-      await use(page)
-    },
-    solanaSandboxPage: async ({ page, phantom }, use) => {
-      await phantom.page.waitForTimeout(1_000)
-      await phantom.importWalletFromPrivateKey(
-        'solana',
-        'XQaKFLLSKbzpVzmfJrj4yUjAyFy2Eu7JcNdbPdnLuod2Uw3yf3tjGd4ha1DBfFdjkZFX1PZg3knth2Tz2tvd8C4'
-      )
-
-      await phantom.toggleTestnetMode()
-
-      await page.goto('https://r3byv.csb.app/')
-      await page.locator('a:has-text("Yes, proceed to preview")').click()
-      await page.getByRole('button', { name: 'Connect to Phantom' }).click()
-
-      await phantom.connectToDapp()
-
-      await page.getByRole('button', { name: 'Clear Logs' }).click()
-
-      await expect(page.getByText('Click a button and watch magic happen...')).toBeVisible()
-
       await use(page)
     }
   })

wallets/phantom/src/playwright/pages/HomePage/actions/index.ts

@@ -1,4 +1,3 @@
-export * from './popups'
 export * from './lock'
 export * from './importWalletFromPrivateKey'
 export * from './switchAccount'

wallets/phantom/src/playwright/pages/NotificationPage/actions/approvePermission.ts

@@ -3,17 +3,6 @@ import Selectors from '../../../../selectors/pages/NotificationPage'
 import type { GasSettings } from '../../../../type/GasSettings'
 import { transaction } from './transaction'
 
-const editTokenPermission = async (notificationPage: Page, customSpendLimit: 'max' | number) => {
-  if (customSpendLimit === 'max') {
-    await notificationPage.locator(Selectors.PermissionPage.approve.maxButton).click()
-    return
-  }
-
-  await notificationPage
-    .locator(Selectors.PermissionPage.approve.customSpendingCapInput)
-    .fill(customSpendLimit.toString())
-}
-
 const approveTokenPermission = async (notificationPage: Page, gasSetting: GasSettings) => {
   // Approve flow is identical to the confirm transaction flow after we click the "Next" button.
   await transaction.confirm(notificationPage, gasSetting)
@@ -24,7 +13,6 @@ const rejectTokenPermission = async (notificationPage: Page) => {
 }
 
 export const approvePermission = {
-  editTokenPermission,
   approve: approveTokenPermission,
   reject: rejectTokenPermission
 }

wallets/phantom/src/playwright/pages/NotificationPage/actions/index.ts

@@ -3,5 +3,4 @@ export * from './closeUnsupportedNetworkWarning'
 export * from './connectToDapp'
 export * from './signSimpleMessage'
 export * from './signStructuredMessage'
-export * from './token'
 export * from './transaction'

wallets/phantom/src/playwright/pages/NotificationPage/page.ts

@@ -96,7 +96,7 @@ export class NotificationPage {
     await approvePermission.reject(notificationPage)
   }
 
-  async closeUnsupportedNetworkWarning(extensionId: string, account?: string) {
+  async closeUnsupportedNetworkWarning(extensionId: string) {
     const notificationPage = await getNotificationPageAndWaitForLoad(this.page.context(), extensionId)
 
     await closeUnsupportedNetworkWarning(notificationPage)

wallets/phantom/src/playwright/pages/index.ts

@@ -1,5 +1,4 @@
 export * from './OnboardingPage/page'
-export * from './CrashPage/page'
 export * from './UnlockPage/page'
 export * from './HomePage/page'
 export * from './NotificationPage/page'

wallets/phantom/src/selectors/index.ts

@@ -1,7 +1,6 @@
 export * from './loading'
 export * from './error'
 
-export { default as crashPage } from './pages/CrashPage'
 export { default as homePage } from './pages/HomePage'
 export { default as unlockPage } from './pages/UnlockPage'
 export { default as notificationPage } from './pages/NotificationPage'

wallets/phantom/src/selectors/pages/HomePage/index.ts

@@ -7,9 +7,7 @@ const addNewAccountMenu = {
 }
 
 const renameAccountMenu = {
-  saveButton: `button${createDataTestSelector('primary-button')}:has-text("Save")`,
-  confirmRenameButton: 'div.editable-label button.mm-button-icon',
-  renameInput: '.mm-text-field .mm-box--padding-right-4'
+  saveButton: `button${createDataTestSelector('primary-button')}:has-text("Save")`
 }
 
 const importAccountMenu = {
@@ -49,48 +47,6 @@ const accountMenu = {
 const manageAccountButton = (accountName: string) =>
   `[role="button"][data-testid="manage-accounts-sortable-${accountName}"]`
 
-const threeDotsMenu = {
-  threeDotsButton: createDataTestSelector('account-options-menu-button'),
-  settingsButton: createDataTestSelector('global-menu-settings'),
-  lockButton: createDataTestSelector('global-menu-lock'),
-  accountDetailsButton: createDataTestSelector('account-list-menu-details'),
-  accountDetailsCloseButton: '.mm-modal-content .mm-modal-header button.mm-button-icon.mm-button-icon--size-sm'
-}
-
-const popoverContainer = '.popover-container'
-const popover = {
-  closeButton: `${popoverContainer} ${createDataTestSelector('popover-close')}`
-}
-
-const recoveryPhraseReminder = {
-  gotItButton: '.recovery-phrase-reminder button.btn-primary'
-}
-
-const networkDropdownContainer = '.multichain-network-list-menu-content-wrapper'
-const networkDropdown = {
-  dropdownButton: createDataTestSelector('network-display'),
-  closeDropdownButton: `${networkDropdownContainer} > section > div:nth-child(1) button`,
-  networksList: `${networkDropdownContainer} .multichain-network-list-menu`,
-  networks: `${networkDropdownContainer} .multichain-network-list-item p`,
-  showTestNetworksToggle: `${networkDropdownContainer} > section > div > label.toggle-button`,
-  toggleOff: `${networkDropdownContainer} label.toggle-button.toggle-button--off`,
-  toggleOn: `${networkDropdownContainer} label.toggle-button.toggle-button--on`,
-  closeNetworkPopupButton:
-    '.mm-modal-header button.mm-button-icon.mm-box--color-icon-default.mm-box--background-color-transparent.mm-box--rounded-lg'
-}
-
-const tabContainer = '.tabs__content'
-const activityTab = {
-  activityTabButton: `${createDataTestSelector('home__activity-tab')}`,
-  transactionsList: `${tabContainer} .transaction-list__transactions`,
-  pendingQueuedTransactions: `${tabContainer} .transaction-list__pending-transactions .transaction-list-item .transaction-status-label--queued`,
-  pendingUnapprovedTransactions: `${tabContainer} .transaction-list__pending-transactions .transaction-list-item .transaction-status-label--unapproved`,
-  pendingApprovedTransactions: `${tabContainer} .transaction-list__pending-transactions .transaction-list-item .transaction-status-label--pending`,
-  completedTransactions: `${tabContainer} .transaction-list__completed-transactions .transaction-list-item`
-}
-
-const singleToken = '.multichain-token-list-item'
-
 export default {
   solanaWalletAddress: createDataTestSelector('account-header-chain-solana:101'),
   ethereumWalletAddress: createDataTestSelector('account-header-chain-eip155:1'),
@@ -100,16 +56,8 @@ export default {
   copyAccountAddressButton: createDataTestSelector('address-copy-button-text'),
   currentNetwork: `${createDataTestSelector('network-display')} span:nth-of-type(1)`,
   headerBackButton: createDataTestSelector('header--back'),
-  threeDotsMenu,
   settings,
-  activityTab,
-  networkDropdown,
   accountMenu,
   editAccountMenu,
-  recoveryPhraseReminder,
-  popover,
-  portfolio: {
-    singleToken
-  },
   manageAccountButton
 }

wallets/phantom/test/playwright/commonSteps/aaveSetup.ts

@@ -0,0 +1,36 @@
+import { type Page, expect } from '@playwright/test'
+import type { Phantom } from '../../../src/playwright'
+
+export const aaveSetup = async (page: Page, phantom: Phantom) => {
+  await page.goto('https://app.aave.com')
+
+  await phantom.toggleTestnetMode()
+
+  await page.locator('button#settings-button').click()
+  await page.locator('li:has-text("Testnet mode")').click()
+  await expect(page.getByRole('button', { name: 'TESTNET' })).toBeVisible()
+
+  await page.getByRole('button', { name: 'Connect wallet' }).first().click()
+  await page.getByRole('button', { name: 'Phantom' }).click()
+
+  await phantom.connectToDapp()
+  await phantom.page.waitForTimeout(1_000)
+  await phantom.closeUnsupportedNetworkWarning()
+
+  await expect(page.getByText('0xf3...2266'), '"0xf3...2266" should be visible').toBeVisible()
+
+  await page.goto(
+    'https://app.aave.com/reserve-overview/?underlyingAsset=0xff34b3d4aee8ddcd6f9afffb6fe49bd371b8a357&marketName=proto_sepolia_v3'
+  )
+
+  await page.getByRole('button', { name: 'Supply' }).click()
+  await page.locator('input[aria-label="amount input"]').fill('1')
+
+  await expect(
+    page.getByRole('button', {
+      name: 'Supply DAI'
+    })
+  ).toBeDisabled()
+
+  await page.locator('button:has-text("Approve DAI to continue")').click()
+}

wallets/phantom/test/playwright/commonSteps/solanaSandboxSetup.ts

@@ -0,0 +1,22 @@
+import { type Page, expect } from '@playwright/test'
+import type { Phantom } from '../../../src/playwright'
+
+export const solanaSandboxSetup = async (page: Page, phantom: Phantom) => {
+  await phantom.page.waitForTimeout(1_000)
+  await phantom.importWalletFromPrivateKey(
+    'solana',
+    'XQaKFLLSKbzpVzmfJrj4yUjAyFy2Eu7JcNdbPdnLuod2Uw3yf3tjGd4ha1DBfFdjkZFX1PZg3knth2Tz2tvd8C4'
+  )
+
+  await phantom.toggleTestnetMode()
+
+  await page.goto('https://r3byv.csb.app/')
+  await page.locator('a:has-text("Yes, proceed to preview")').click()
+  await page.getByRole('button', { name: 'Connect to Phantom' }).click()
+
+  await phantom.connectToDapp()
+
+  await page.getByRole('button', { name: 'Clear Logs' }).click()
+
+  await expect(page.getByText('Click a button and watch magic happen...')).toBeVisible()
+}