Skip to content

Commit

Permalink
fix: some fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
jake4take committed Dec 4, 2024
1 parent 60e7892 commit c4395de
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions packages/wallets/src/okx/okx.page.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import { WalletConfig } from '../wallets.constants';
import { WalletPage } from '../wallet.page';
import expect from 'expect';
import { test, BrowserContext, Page } from '@playwright/test';
import { test, BrowserContext, Page, expect } from '@playwright/test';
import {
AccountList,
HomePage,
Expand Down Expand Up @@ -162,7 +161,6 @@ export class OkxPage implements WalletPage {
});
}

// todo we have identical functions (switchNetwork() and changeNetwork())
async changeNetwork(networkName: string) {
await this.switchNetwork(await checkNetworkName(networkName));
}
Expand All @@ -189,6 +187,9 @@ export class OkxPage implements WalletPage {
await this.navigate();
await this.homePage.networkListButton.click();
const currentNetwork = await this.networkListPage.getWalletNetwork();
if (currentNetwork === 'All networks') {
await this.switchNetwork('Ethereum');
}
await this.goto();
// switch network for connected dApp
if (
Expand Down Expand Up @@ -252,7 +253,7 @@ export class OkxPage implements WalletPage {
});
}

// todo for mainnet
// need realize for mainnet
async openLastTxInEthplorer(txIndex = 0) {
console.error(
`OKX wallet does not display the transaction history for testnet (param ${txIndex})`,
Expand Down

0 comments on commit c4395de

Please sign in to comment.