|
|
|
const assert = require('assert')
|
|
|
|
const webdriver = require('selenium-webdriver')
|
|
|
|
|
|
|
|
const { By, Key, until } = webdriver
|
|
|
|
const {
|
|
|
|
tinyDelayMs,
|
|
|
|
regularDelayMs,
|
|
|
|
largeDelayMs,
|
|
|
|
} = require('./helpers')
|
|
|
|
const { buildWebDriver } = require('./webdriver')
|
|
|
|
const Ganache = require('./ganache')
|
|
|
|
const enLocaleMessages = require('../../app/_locales/en/messages.json')
|
|
|
|
|
|
|
|
const ganacheServer = new Ganache()
|
|
|
|
|
|
|
|
describe('MetaMask', function () {
|
|
|
|
let driver
|
|
|
|
let tokenAddress
|
|
|
|
|
|
|
|
const testSeedPhrase = 'phrase upgrade clock rough situate wedding elder clever doctor stamp excess tent'
|
|
|
|
|
|
|
|
this.timeout(0)
|
|
|
|
this.bail(true)
|
|
|
|
|
|
|
|
before(async function () {
|
|
|
|
await ganacheServer.start()
|
|
|
|
const result = await buildWebDriver()
|
|
|
|
driver = result.driver
|
|
|
|
})
|
|
|
|
|
|
|
|
afterEach(async function () {
|
|
|
|
if (process.env.SELENIUM_BROWSER === 'chrome') {
|
|
|
|
const errors = await driver.checkBrowserForConsoleErrors(driver)
|
|
|
|
if (errors.length) {
|
|
|
|
const errorReports = errors.map((err) => err.message)
|
|
|
|
const errorMessage = `Errors found in browser console:\n${errorReports.join('\n')}`
|
|
|
|
console.error(new Error(errorMessage))
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (this.currentTest.state === 'failed') {
|
|
|
|
await driver.verboseReportOnFailure(this.currentTest.title)
|
|
|
|
}
|
|
|
|
})
|
|
|
|
|
|
|
|
after(async function () {
|
|
|
|
await ganacheServer.quit()
|
|
|
|
await driver.quit()
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Going through the first time flow', function () {
|
|
|
|
it('clicks the continue button on the welcome screen', async function () {
|
|
|
|
await driver.findElement(By.css('.welcome-page__header'))
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), '${enLocaleMessages.getStarted.message}')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('clicks the "Create New Wallet" option', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Create a Wallet')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('clicks the "No thanks" option on the metametrics opt-in screen', async function () {
|
|
|
|
await driver.clickElement(By.css('.btn-default'))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('accepts a secure password', async function () {
|
|
|
|
const passwordBox = await driver.findElement(By.css('.first-time-flow__form #create-password'))
|
|
|
|
const passwordBoxConfirm = await driver.findElement(By.css('.first-time-flow__form #confirm-password'))
|
|
|
|
|
|
|
|
await passwordBox.sendKeys('correct horse battery staple')
|
|
|
|
await passwordBoxConfirm.sendKeys('correct horse battery staple')
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.first-time-flow__checkbox'))
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.first-time-flow__form button'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
let seedPhrase
|
|
|
|
|
|
|
|
it('reveals the seed phrase', async function () {
|
|
|
|
const byRevealButton = By.css('.reveal-seed-phrase__secret-blocker .reveal-seed-phrase__reveal-button')
|
|
|
|
await driver.findElement(byRevealButton)
|
|
|
|
await driver.clickElement(byRevealButton)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const revealedSeedPhrase = await driver.findElement(By.css('.reveal-seed-phrase__secret-words'))
|
|
|
|
seedPhrase = await revealedSeedPhrase.getText()
|
|
|
|
assert.equal(seedPhrase.split(' ').length, 12)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), '${enLocaleMessages.next.message}')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
async function clickWordAndWait (word) {
|
|
|
|
await driver.clickElement(By.css(`[data-testid="seed-phrase-sorted"] [data-testid="draggable-seed-${word}"]`))
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
}
|
|
|
|
|
|
|
|
it('can retype the seed phrase', async function () {
|
|
|
|
const words = seedPhrase.split(' ')
|
|
|
|
|
|
|
|
for (const word of words) {
|
|
|
|
await clickWordAndWait(word)
|
|
|
|
}
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('clicks through the success screen', async function () {
|
|
|
|
await driver.findElement(By.xpath(`//div[contains(text(), 'Congratulations')]`))
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), '${enLocaleMessages.endOfFlowMessage10.message}')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Show account information', function () {
|
|
|
|
it('shows the QR code for the account', async function () {
|
|
|
|
await driver.clickElement(By.css('[data-testid="account-options-menu-button"]'))
|
|
|
|
await driver.clickElement(By.css('[data-testid="account-options-menu__account-details"]'))
|
|
|
|
await driver.findVisibleElement(By.css('.qr-code__wrapper'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const accountModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
await driver.clickElement(By.css('.account-modal-close'))
|
|
|
|
|
|
|
|
await driver.wait(until.stalenessOf(accountModal))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Lock an unlock', function () {
|
|
|
|
it('logs out of the account', async function () {
|
|
|
|
await driver.clickElement(By.css('.account-menu__icon'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const lockButton = await driver.findClickableElement(By.css('.account-menu__lock-button'))
|
|
|
|
assert.equal(await lockButton.getText(), 'Lock')
|
|
|
|
await lockButton.click()
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('accepts the account password after lock', async function () {
|
|
|
|
const passwordField = await driver.findElement(By.id('password'))
|
|
|
|
await passwordField.sendKeys('correct horse battery staple')
|
|
|
|
await passwordField.sendKeys(Key.ENTER)
|
|
|
|
await driver.delay(largeDelayMs * 4)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Add account', function () {
|
|
|
|
it('choose Create Account from the account menu', async function () {
|
|
|
|
await driver.clickElement(By.css('.account-menu__icon'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//div[contains(text(), 'Create Account')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('set account name', async function () {
|
|
|
|
const accountName = await driver.findElement(By.css('.new-account-create-form input'))
|
|
|
|
await accountName.sendKeys('2nd account')
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Create')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('should display correct account name', async function () {
|
|
|
|
const accountName = await driver.findElement(By.css('.selected-account__name'))
|
|
|
|
assert.equal(await accountName.getText(), '2nd account')
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Import seed phrase', function () {
|
|
|
|
it('logs out of the vault', async function () {
|
|
|
|
await driver.clickElement(By.css('.account-menu__icon'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const lockButton = await driver.findClickableElement(By.css('.account-menu__lock-button'))
|
|
|
|
assert.equal(await lockButton.getText(), 'Lock')
|
|
|
|
await lockButton.click()
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('imports seed phrase', async function () {
|
|
|
|
const restoreSeedLink = await driver.findClickableElement(By.css('.unlock-page__link--import'))
|
|
|
|
assert.equal(await restoreSeedLink.getText(), 'Import using account seed phrase')
|
|
|
|
await restoreSeedLink.click()
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const seedTextArea = await driver.findElement(By.css('textarea'))
|
|
|
|
await seedTextArea.sendKeys(testSeedPhrase)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const passwordInputs = await driver.findElements(By.css('input'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await passwordInputs[0].sendKeys('correct horse battery staple')
|
|
|
|
await passwordInputs[1].sendKeys('correct horse battery staple')
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), '${enLocaleMessages.restore.message}')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('balance renders', async function () {
|
|
|
|
const balance = await driver.findElement(By.css('[data-testid="wallet-balance"] .list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(balance, /100\s*ETH/))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Send ETH from inside MetaMask using default gas', function () {
|
|
|
|
it('starts a send transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Send')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const inputAddress = await driver.findElement(By.css('input[placeholder="Search, public address (0x), or ENS"]'))
|
|
|
|
await inputAddress.sendKeys('0x2f318C334780961FB129D2a6c30D0763d9a5C970')
|
|
|
|
|
|
|
|
const inputAmount = await driver.findElement(By.css('.unit-input__input'))
|
|
|
|
await inputAmount.sendKeys('1000')
|
|
|
|
|
|
|
|
const errorAmount = await driver.findElement(By.css('.send-v2__error-amount'))
|
|
|
|
assert.equal(await errorAmount.getText(), 'Insufficient funds.', 'send screen should render an insufficient fund error message')
|
|
|
|
|
|
|
|
await inputAmount.sendKeys(Key.BACK_SPACE)
|
|
|
|
await driver.delay(50)
|
|
|
|
await inputAmount.sendKeys(Key.BACK_SPACE)
|
|
|
|
await driver.delay(50)
|
|
|
|
await inputAmount.sendKeys(Key.BACK_SPACE)
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
|
|
|
|
await driver.assertElementNotPresent(By.css('.send-v2__error-amount'))
|
|
|
|
|
|
|
|
const amountMax = await driver.findClickableElement(By.css('.send-v2__amount-max'))
|
|
|
|
await amountMax.click()
|
|
|
|
|
|
|
|
assert.equal(await inputAmount.isEnabled(), false)
|
|
|
|
|
|
|
|
let inputValue = await inputAmount.getAttribute('value')
|
|
|
|
|
|
|
|
assert(Number(inputValue) > 99)
|
|
|
|
|
|
|
|
await amountMax.click()
|
|
|
|
|
|
|
|
assert.equal(await inputAmount.isEnabled(), true)
|
|
|
|
|
|
|
|
await inputAmount.sendKeys('1')
|
|
|
|
|
|
|
|
inputValue = await inputAmount.getAttribute('value')
|
|
|
|
assert.equal(inputValue, '1')
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
// Continue to next screen
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Next')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('confirms the transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.clickElement(By.css('[data-testid="home__activity-tab"]'))
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 1
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValues = await driver.findElement(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txValues, /-1\s*ETH/), 10000)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Send ETH from inside MetaMask using fast gas option', function () {
|
|
|
|
it('starts a send transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Send')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const inputAddress = await driver.findElement(By.css('input[placeholder="Search, public address (0x), or ENS"]'))
|
|
|
|
await inputAddress.sendKeys('0x2f318C334780961FB129D2a6c30D0763d9a5C970')
|
|
|
|
|
|
|
|
const inputAmount = await driver.findElement(By.css('.unit-input__input'))
|
|
|
|
await inputAmount.sendKeys('1')
|
|
|
|
|
|
|
|
const inputValue = await inputAmount.getAttribute('value')
|
|
|
|
assert.equal(inputValue, '1')
|
|
|
|
|
|
|
|
// Set the gas price
|
|
|
|
await driver.clickElement(By.xpath(`//button/div/div[contains(text(), "Fast")]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
// Continue to next screen
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Next')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('confirms the transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 2
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValues = await driver.findElement(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txValues, /-1\s*ETH/), 10000)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Send ETH from inside MetaMask using advanced gas modal', function () {
|
|
|
|
it('starts a send transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Send')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const inputAddress = await driver.findElement(By.css('input[placeholder="Search, public address (0x), or ENS"]'))
|
|
|
|
await inputAddress.sendKeys('0x2f318C334780961FB129D2a6c30D0763d9a5C970')
|
|
|
|
|
|
|
|
const inputAmount = await driver.findElement(By.css('.unit-input__input'))
|
|
|
|
await inputAmount.sendKeys('1')
|
|
|
|
|
|
|
|
const inputValue = await inputAmount.getAttribute('value')
|
|
|
|
assert.equal(inputValue, '1')
|
|
|
|
|
|
|
|
// Set the gas limit
|
|
|
|
await driver.clickElement(By.css('.advanced-gas-options-btn'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const gasModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Save')]`))
|
|
|
|
await driver.wait(until.stalenessOf(gasModal))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
// Continue to next screen
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Next')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('confirms the transaction', async function () {
|
|
|
|
const transactionAmounts = await driver.findElements(By.css('.currency-display-component__text'))
|
|
|
|
const transactionAmount = transactionAmounts[0]
|
|
|
|
assert.equal(await transactionAmount.getText(), '1')
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 3
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValues = await driver.findElement(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txValues, /-1\s*ETH/), 10000)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Send ETH from dapp using advanced gas controls', function () {
|
|
|
|
let windowHandles
|
|
|
|
let extension
|
|
|
|
let popup
|
|
|
|
let dapp
|
|
|
|
|
|
|
|
it('goes to the settings screen', async function () {
|
|
|
|
await driver.clickElement(By.css('.account-menu__icon'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//div[contains(text(), 'Settings')]`))
|
|
|
|
|
|
|
|
// await driver.findElement(By.css('.tab-bar'))
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//div[contains(text(), 'Advanced')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="advanced-setting-show-testnet-conversion"] .settings-page__content-item-col > div > div'))
|
|
|
|
|
|
|
|
const advancedGasTitle = await driver.findElement(By.xpath(`//span[contains(text(), 'Advanced gas controls')]`))
|
|
|
|
await driver.scrollToElement(advancedGasTitle)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="advanced-setting-advanced-gas-inline"] .settings-page__content-item-col > div > div'))
|
|
|
|
windowHandles = await driver.getAllWindowHandles()
|
|
|
|
extension = windowHandles[0]
|
|
|
|
await driver.closeAllWindowHandlesExcept([extension])
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.app-header__logo-container'))
|
|
|
|
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('connects the dapp', async function () {
|
|
|
|
await driver.openNewPage('http://127.0.0.1:8080/')
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Connect')]`))
|
|
|
|
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.waitUntilXWindowHandles(3)
|
|
|
|
windowHandles = await driver.getAllWindowHandles()
|
|
|
|
|
|
|
|
extension = windowHandles[0]
|
|
|
|
dapp = await driver.switchToWindowWithTitle('E2E Test Dapp', windowHandles)
|
|
|
|
popup = windowHandles.find((handle) => handle !== extension && handle !== dapp)
|
Connect distinct accounts per site (#7004)
* add PermissionsController
remove provider approval controller
integrate rpc-cap
create PermissionsController
move provider approval functionality to permissions controller
add permissions approval ui, settings page
add permissions activity and history
move some functionality to metamask-inpage-provider
rename siteMetadata -> domainMetadata
add accountsChange notification to inpage provider
move functionality to inpage provider
update inpage provider
Remove 'Connections' settings page (#7369)
add hooks for exposing accounts in settings
rename unused messages in non-English locales
Add external extension id to metadata (#7396)
update inpage provider, rpc-cap
add eth_requestAccounts handling to background
prevent notifying connections if extension is locked
update inpage provider
Fix lint errors
add migration
review fixes
transaction controller review updates
removed unused messages
* Login Per Site UI (#7368)
* LoginPerSite original UI changes to keep
* First commit
* Get necessary connected tab info for redirect and icon display for permissioned sites
* Fix up designs and add missing features
* Some lint fixes
* More lint fixes
* Ensures the tx controller + tx-state-manager orders transactions in the order they are received
* Code cleanup for LoginPerSite-ui
* Update e2e tests to use new connection flow
* Fix display of connect screen and app header after login when connect request present
* Update metamask-responsive-ui.spec for new item in accounts dropdown
* Fix approve container by replacing approvedOrigins with domainMetaData
* Adds test/e2e/permissions.spec.js
* Correctly handle cancellation of a permissions request
* Redirect to home after disconnecting all sites / cancelling all permissions
* Fix display of site icons in menu
* Fix height of permissions page container
* Remove unused locale messages
* Set default values for openExternalTabs and tabIdOrigins in account-menu.container
* More code cleanup for LoginPerSite-ui
* Use extensions api to close tab in permissions-connect
* Remove unnecessary change in domIsReady() in contentscript
* Remove unnecessary private function markers and class methods (for background tab info) in metamask-controller.
* Adds getOriginOfCurrentTab selector
* Adds IconWithFallback component and substitutes for appropriate cases
* Add and utilize font mixins
* Remove unused method in disconnect-all.container.js
* Simplify buttonSizeLarge code in page-container-footer.component.js
* Add and utilize getAccountsWithLabels selector
* Remove console.log in ui/app/store/actions.js
* Change last connected time format to yyyy-M-d
* Fix css associated with IconWithFallback change
* Ensure tracked openNonMetamaskTabsIDs are correctly set to inactive on tab changes
* Code cleanup for LoginPerSite-ui
* Use reusable function for modifying openNonMetamaskTabsIDs in background.js
* Enables automatic switching to connected account when connected domain is open
* Prevent exploit of tabIdOriginMap in background.js
* Remove unneeded code from contentscript.js
* Simplify current tab origin and window opener logic using remotePort listener tabs.queryTabs
* Design and styling fixes for LoginPerSite-ui
* Fix permissionHistory and permission logging for eth_requestAccounts and eth_accounts
* Front end changes to support display of lastConnected time in connected and permissions screens
* Fix lint errors
* Refactor structure of permissionsHistory
* Fix default values and object modifications for domain and permissionsHistory related data
* Fix connecting to new accounts from modal
* Replace retweet.svg with connect-white.svg
* Fix signature-request.spec
* Update metamask-inpage-provider version
* Fix permissions e2e tests
* Remove unneeded delay from test/e2e/signature-request.spec.js
* Add delay before attempting to retrieve network id in dapp in ethereum-on=.spec
* Use requestAccountTabIds strategy for determining tab id that opened a given window
* Improve default values for permissions requests
* Add some message descriptions to app/_locales/en/messages.json
* Code clean up in permission controller
* Stopped deep cloning object in mapObjectValues
* Bump metamask-inpage-provider version
* Add missing description in app/_locales/en/messages.json
* Return promises from queryTabs and switchToTab of extension.js
* Remove unused getAllPermissions function
* Use default props in icon-with-fallback.component.js
* Stop passing to permissions controller
* Delete no longer used clear-approved-origins modal code
* Remove duplicate imports in ui/app/components/app/index.scss
* Use URL instead of regex in getOriginFromUrl()
* Add runtime error checking to platform, promise based extension.tab methods
* Support permission requests from external extensions
* Improve font size and colour of the domain origin on the permission confirmation screen
* Add support for toggling permissions
* Ensure getRenderablePermissionsDomains only returns domains with exposedAccount caveat permissions
* Remove unused code from LoginPerSite-ui branch
* Ensure modal closes on Enter press for new-account-modal.component.js
* Lint fix
* fixup! Login Per Site UI (#7368)
* Some code cleanup for LoginPerSite
* Adds UX for connecting to dapps via the connected sites screen (#7593)
* Adds UX for connecting to dapps via the connected sites screen
* Use openMetaMaskTabIds from background.js to determine if current active tab is MetaMask
* Delete unused permissions controller methods
* Fixes two small bugs in the LoginPerSite ui (#7595)
* Restore `providerRequest` message translations (#7600)
This message was removed, but it was replaced with a very similar
message called `likeToConnect`. The only difference is that the new
message has "MetaMask" in it. Preserving these messages without
"MetaMask" is probably better than deleting them, so these messages
have all been restored and renamed to `likeToConnect`.
* Login per site no sitemetadata fix (#7610)
* Support connected sites for which we have no site metadata.
* Change property containing subtitle info often populated by origin to a more accurate of purpose name
* Lint fix
* Improve disconnection modal messages (#7612)
* Improve disconnectAccountModalDescription and disconnectAllModalDescription messages
* Update disconnectAccountModalDescription app/_locales/en/messages.json
Co-Authored-By: Mark Stacey <markjstacey@gmail.com>
* Improve disconnectAccount modal message clarity
* Adds cancel button to the account selection screen of the permissions request flow (#7613)
* Fix eth_accounts permission language & selectability (#7614)
* fix eth_accounts language & selectability
* fix MetaMask capitalization in all messages
* Close sidebar when opening connected sites (#7611)
The 'Connected Sites' button in the accounts details now closes the
sidebar, if it is open. This was accomplished by pulling the click
handler for that button up to the wallet view component, where another
button already followed a similar pattern of closing the sidebar.
It seemed confusing to me that one handler was in the `AccountsDetails`
container component, and one was handed down from above, so I added
PropTypes to the container component.
I'm not sure that the WalletView component is the best place for this
logic, but I've put it there for now to be consistent with the add
token button.
* Reject permissions request upon tab close (#7618)
Permissions requests are now rejected when the page is closed. This
only applies to the full-screen view, as that is the view permission
requests should be handled in. The case where the user deals with the
request through a different view is handled in #7617
* Handle tab update failure (#7619)
`extension.tabs.update` can sometimes fail if the user interacts with
the tabs directly around the same time. The redirect flow has been
updated to ensure that the permissions tab is still closed in that
case. The user is on their own to find the dapp tab again in that case.
* Login per site tab popup fixes (#7617)
* Handle redirect in response to state update in permissions-connect
* Ensure origin is available to permissions-connect subcomponents during redirect
* Hide app bar whenever on redirect route
* Improvements to handling of redirects in permissions-connect
* Ensure permission request id change handling only happens when page is not null
* Lint fix
* Decouple confirm transaction screen from the selected address (#7622)
* Avoid race condtion that could prevent contextual account switching (#7623)
There was a race condition in the logic responsible for switching the
selected account based upon the active tab. It was asynchronously
querying the active tab, then assuming it had been retrieved later.
The active tab info itself was already in the redux store in another
spot, one that is guaranteed to be set before the UI renders. The
race condition was avoided by deleting the duplicate state, and using
the other active tab state.
* Only redirect back to dapp if current tab is active (#7621)
The "redirect back to dapp" behaviour can be disruptive when the
permissions connect tab is not active. The purpose of the redirect was
to maintain context between the dapp and the permissions request, but
if the user has already moved to another tab, that no longer applies.
* Fix JSX style lint errors
* Remove unused state
5 years ago
|
|
|
|
|
|
|
await driver.switchToWindow(popup)
|
|
|
|
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Next')]`))
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Connect')]`))
|
Connect distinct accounts per site (#7004)
* add PermissionsController
remove provider approval controller
integrate rpc-cap
create PermissionsController
move provider approval functionality to permissions controller
add permissions approval ui, settings page
add permissions activity and history
move some functionality to metamask-inpage-provider
rename siteMetadata -> domainMetadata
add accountsChange notification to inpage provider
move functionality to inpage provider
update inpage provider
Remove 'Connections' settings page (#7369)
add hooks for exposing accounts in settings
rename unused messages in non-English locales
Add external extension id to metadata (#7396)
update inpage provider, rpc-cap
add eth_requestAccounts handling to background
prevent notifying connections if extension is locked
update inpage provider
Fix lint errors
add migration
review fixes
transaction controller review updates
removed unused messages
* Login Per Site UI (#7368)
* LoginPerSite original UI changes to keep
* First commit
* Get necessary connected tab info for redirect and icon display for permissioned sites
* Fix up designs and add missing features
* Some lint fixes
* More lint fixes
* Ensures the tx controller + tx-state-manager orders transactions in the order they are received
* Code cleanup for LoginPerSite-ui
* Update e2e tests to use new connection flow
* Fix display of connect screen and app header after login when connect request present
* Update metamask-responsive-ui.spec for new item in accounts dropdown
* Fix approve container by replacing approvedOrigins with domainMetaData
* Adds test/e2e/permissions.spec.js
* Correctly handle cancellation of a permissions request
* Redirect to home after disconnecting all sites / cancelling all permissions
* Fix display of site icons in menu
* Fix height of permissions page container
* Remove unused locale messages
* Set default values for openExternalTabs and tabIdOrigins in account-menu.container
* More code cleanup for LoginPerSite-ui
* Use extensions api to close tab in permissions-connect
* Remove unnecessary change in domIsReady() in contentscript
* Remove unnecessary private function markers and class methods (for background tab info) in metamask-controller.
* Adds getOriginOfCurrentTab selector
* Adds IconWithFallback component and substitutes for appropriate cases
* Add and utilize font mixins
* Remove unused method in disconnect-all.container.js
* Simplify buttonSizeLarge code in page-container-footer.component.js
* Add and utilize getAccountsWithLabels selector
* Remove console.log in ui/app/store/actions.js
* Change last connected time format to yyyy-M-d
* Fix css associated with IconWithFallback change
* Ensure tracked openNonMetamaskTabsIDs are correctly set to inactive on tab changes
* Code cleanup for LoginPerSite-ui
* Use reusable function for modifying openNonMetamaskTabsIDs in background.js
* Enables automatic switching to connected account when connected domain is open
* Prevent exploit of tabIdOriginMap in background.js
* Remove unneeded code from contentscript.js
* Simplify current tab origin and window opener logic using remotePort listener tabs.queryTabs
* Design and styling fixes for LoginPerSite-ui
* Fix permissionHistory and permission logging for eth_requestAccounts and eth_accounts
* Front end changes to support display of lastConnected time in connected and permissions screens
* Fix lint errors
* Refactor structure of permissionsHistory
* Fix default values and object modifications for domain and permissionsHistory related data
* Fix connecting to new accounts from modal
* Replace retweet.svg with connect-white.svg
* Fix signature-request.spec
* Update metamask-inpage-provider version
* Fix permissions e2e tests
* Remove unneeded delay from test/e2e/signature-request.spec.js
* Add delay before attempting to retrieve network id in dapp in ethereum-on=.spec
* Use requestAccountTabIds strategy for determining tab id that opened a given window
* Improve default values for permissions requests
* Add some message descriptions to app/_locales/en/messages.json
* Code clean up in permission controller
* Stopped deep cloning object in mapObjectValues
* Bump metamask-inpage-provider version
* Add missing description in app/_locales/en/messages.json
* Return promises from queryTabs and switchToTab of extension.js
* Remove unused getAllPermissions function
* Use default props in icon-with-fallback.component.js
* Stop passing to permissions controller
* Delete no longer used clear-approved-origins modal code
* Remove duplicate imports in ui/app/components/app/index.scss
* Use URL instead of regex in getOriginFromUrl()
* Add runtime error checking to platform, promise based extension.tab methods
* Support permission requests from external extensions
* Improve font size and colour of the domain origin on the permission confirmation screen
* Add support for toggling permissions
* Ensure getRenderablePermissionsDomains only returns domains with exposedAccount caveat permissions
* Remove unused code from LoginPerSite-ui branch
* Ensure modal closes on Enter press for new-account-modal.component.js
* Lint fix
* fixup! Login Per Site UI (#7368)
* Some code cleanup for LoginPerSite
* Adds UX for connecting to dapps via the connected sites screen (#7593)
* Adds UX for connecting to dapps via the connected sites screen
* Use openMetaMaskTabIds from background.js to determine if current active tab is MetaMask
* Delete unused permissions controller methods
* Fixes two small bugs in the LoginPerSite ui (#7595)
* Restore `providerRequest` message translations (#7600)
This message was removed, but it was replaced with a very similar
message called `likeToConnect`. The only difference is that the new
message has "MetaMask" in it. Preserving these messages without
"MetaMask" is probably better than deleting them, so these messages
have all been restored and renamed to `likeToConnect`.
* Login per site no sitemetadata fix (#7610)
* Support connected sites for which we have no site metadata.
* Change property containing subtitle info often populated by origin to a more accurate of purpose name
* Lint fix
* Improve disconnection modal messages (#7612)
* Improve disconnectAccountModalDescription and disconnectAllModalDescription messages
* Update disconnectAccountModalDescription app/_locales/en/messages.json
Co-Authored-By: Mark Stacey <markjstacey@gmail.com>
* Improve disconnectAccount modal message clarity
* Adds cancel button to the account selection screen of the permissions request flow (#7613)
* Fix eth_accounts permission language & selectability (#7614)
* fix eth_accounts language & selectability
* fix MetaMask capitalization in all messages
* Close sidebar when opening connected sites (#7611)
The 'Connected Sites' button in the accounts details now closes the
sidebar, if it is open. This was accomplished by pulling the click
handler for that button up to the wallet view component, where another
button already followed a similar pattern of closing the sidebar.
It seemed confusing to me that one handler was in the `AccountsDetails`
container component, and one was handed down from above, so I added
PropTypes to the container component.
I'm not sure that the WalletView component is the best place for this
logic, but I've put it there for now to be consistent with the add
token button.
* Reject permissions request upon tab close (#7618)
Permissions requests are now rejected when the page is closed. This
only applies to the full-screen view, as that is the view permission
requests should be handled in. The case where the user deals with the
request through a different view is handled in #7617
* Handle tab update failure (#7619)
`extension.tabs.update` can sometimes fail if the user interacts with
the tabs directly around the same time. The redirect flow has been
updated to ensure that the permissions tab is still closed in that
case. The user is on their own to find the dapp tab again in that case.
* Login per site tab popup fixes (#7617)
* Handle redirect in response to state update in permissions-connect
* Ensure origin is available to permissions-connect subcomponents during redirect
* Hide app bar whenever on redirect route
* Improvements to handling of redirects in permissions-connect
* Ensure permission request id change handling only happens when page is not null
* Lint fix
* Decouple confirm transaction screen from the selected address (#7622)
* Avoid race condtion that could prevent contextual account switching (#7623)
There was a race condition in the logic responsible for switching the
selected account based upon the active tab. It was asynchronously
querying the active tab, then assuming it had been retrieved later.
The active tab info itself was already in the redux store in another
spot, one that is guaranteed to be set before the UI renders. The
race condition was avoided by deleting the duplicate state, and using
the other active tab state.
* Only redirect back to dapp if current tab is active (#7621)
The "redirect back to dapp" behaviour can be disruptive when the
permissions connect tab is not active. The purpose of the redirect was
to maintain context between the dapp and the permissions request, but
if the user has already moved to another tab, that no longer applies.
* Fix JSX style lint errors
* Remove unused state
5 years ago
|
|
|
|
|
|
|
await driver.waitUntilXWindowHandles(2)
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('initiates a send from the dapp', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Send')]`), 10000)
|
|
|
|
await driver.delay(2000)
|
|
|
|
|
|
|
|
windowHandles = await driver.getAllWindowHandles()
|
|
|
|
await driver.switchToWindowWithTitle('MetaMask Notification', windowHandles)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.assertElementNotPresent(By.xpath(`//li[contains(text(), 'Data')]`))
|
|
|
|
|
|
|
|
const [gasPriceInput, gasLimitInput] = await driver.findElements(By.css('.advanced-gas-inputs__gas-edit-row__input'))
|
|
|
|
|
|
|
|
await gasPriceInput.clear()
|
|
|
|
await driver.delay(50)
|
|
|
|
await gasPriceInput.sendKeys('10')
|
|
|
|
await driver.delay(50)
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
await driver.delay(50)
|
|
|
|
|
|
|
|
await gasLimitInput.clear()
|
|
|
|
await driver.delay(50)
|
|
|
|
await gasLimitInput.sendKeys('25000')
|
Use `AdvancedGasInputs` in `AdvancedTabContent` (#7186)
* Use `AdvancedGasInputs` in `AdvancedTabContent`
The `AdvancedGasInputs` component was originally extracted from the
`AdvancedTabContent` component, duplicating much of the rendering
logic. They have since evolved separately, with bugs being fixed in one
place but not the other.
The inputs and outputs expected weren't exactly the same, as the
`AdvancedGasInputs` component converts the input custom gas price and
limit, and it converts them both in the setter methods as well.
The `GasModalPageContainer` had to be adjusted to avoid converting
these values multiple times.
Both components dealt with input debouncing separately, both in less
than ideal ways. `AdvancedTabContent` didn't debounce either field, but
it did debounce the check for whether the gas limit field was below the
minimum value. So if a less-than-minimum value was set, it would be
propogated upwards and could be saved if the user clicked 'Save'
quickly enough. After a second delay it would snap back to the minimum
value. The `AdvancedGasInputs` component debounced both fields, but
it would replace any gas limit below the minimum with the minimum
value. This led to a problem where a brief pause during typing would
reset the field to 21000.
The `AdvancedGasInputs` approach was chosen, except that it was
updated to no longer change the gas limit if it was below the minimum.
Instead it displays an error. Parent components were checked to ensure
they would detect the error case of the gas limit being set too low,
and prevent the form submission in those cases. Of the three parents,
one had already dealt with it correctly, one needed to convert the
gas limit from hex first, and another needed the gas limit check added.
Closes #6872
* Cleanup send components
Empty README files have been removed, and a mistake in the index file
for the send page has been corrected. The Gas Slider component class
name was updated as well; it looks like it was originally created from
`AdvancedTabContent`, so it still had that class name.
5 years ago
|
|
|
|
|
|
|
await driver.delay(1000)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`), 10000)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.waitUntilXWindowHandles(2)
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 4
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValue = await driver.findClickableElement(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txValue, /-3\s*ETH/), 10000)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('the transaction has the expected gas price', async function () {
|
|
|
|
const txValue = await driver.findClickableElement(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await txValue.click()
|
|
|
|
const popoverCloseButton = await driver.findClickableElement(By.css('.popover-header__button'))
|
|
|
|
const txGasPrice = await driver.findElement(By.css('[data-testid="transaction-breakdown__gas-price"]'))
|
|
|
|
await driver.wait(until.elementTextMatches(txGasPrice, /^10$/), 10000)
|
|
|
|
await popoverCloseButton.click()
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Navigate transactions', function () {
|
|
|
|
it('adds multiple transactions', async function () {
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.waitUntilXWindowHandles(2)
|
|
|
|
const windowHandles = await driver.getAllWindowHandles()
|
|
|
|
const extension = windowHandles[0]
|
|
|
|
const dapp = windowHandles[1]
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
|
|
|
|
const send3eth = await driver.findClickableElement(By.xpath(`//button[contains(text(), 'Send')]`))
|
|
|
|
await send3eth.click()
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
|
|
|
|
const contractDeployment = await driver.findClickableElement(By.xpath(`//button[contains(text(), 'Deploy Contract')]`))
|
|
|
|
await contractDeployment.click()
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
|
|
|
|
await send3eth.click()
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
await contractDeployment.click()
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.transaction-list-item'))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('navigates the transactions', async function () {
|
|
|
|
await driver.clickElement(By.css('[data-testid="next-page"]'))
|
|
|
|
let navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
let navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('2'), true, 'changed transaction right')
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="next-page"]'))
|
|
|
|
navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('3'), true, 'changed transaction right')
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="next-page"]'))
|
|
|
|
navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('4'), true, 'changed transaction right')
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="first-page"]'))
|
|
|
|
navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('1'), true, 'navigate to first transaction')
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="last-page"]'))
|
|
|
|
navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.split('4').length, 3, 'navigate to last transaction')
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="previous-page"]'))
|
|
|
|
navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('3'), true, 'changed transaction left')
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="previous-page"]'))
|
|
|
|
navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('2'), true, 'changed transaction left')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('adds a transaction while confirm screen is in focus', async function () {
|
|
|
|
let navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
let navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('2'), true, 'second transaction in focus')
|
|
|
|
|
|
|
|
const windowHandles = await driver.getAllWindowHandles()
|
|
|
|
const extension = windowHandles[0]
|
|
|
|
const dapp = windowHandles[1]
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Send')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('2'), true, 'correct (same) transaction in focus')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('rejects a transaction', async function () {
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Reject')]`))
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
|
|
|
|
const navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
const navigationText = await navigationElement.getText()
|
|
|
|
assert.equal(navigationText.includes('4'), true, 'transaction rejected')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('confirms a transaction', async function () {
|
|
|
|
await driver.delay(tinyDelayMs / 2)
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const navigationElement = await driver.findElement(By.css('.confirm-page-container-navigation'))
|
|
|
|
await driver.delay(tinyDelayMs / 2)
|
|
|
|
const navigationText = await navigationElement.getText()
|
|
|
|
await driver.delay(tinyDelayMs / 2)
|
|
|
|
assert.equal(navigationText.includes('3'), true, 'transaction confirmed')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('rejects the rest of the transactions', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//a[contains(text(), 'Reject 3')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Reject All')]`))
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 5
|
|
|
|
}, 10000)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Deploy contract and call contract methods', function () {
|
|
|
|
let extension
|
|
|
|
let dapp
|
|
|
|
it('creates a deploy contract transaction', async function () {
|
|
|
|
const windowHandles = await driver.getAllWindowHandles()
|
|
|
|
extension = windowHandles[0]
|
|
|
|
dapp = windowHandles[1]
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('#deployButton'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//h2[contains(text(), 'Contract Deployment')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('displays the contract creation data', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//li[contains(text(), 'Data')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.findElement(By.xpath(`//div[contains(text(), '127.0.0.1')]`))
|
|
|
|
|
|
|
|
const confirmDataDiv = await driver.findElement(By.css('.confirm-page-container-content__data-box'))
|
|
|
|
const confirmDataText = await confirmDataDiv.getText()
|
|
|
|
assert.ok(confirmDataText.includes('Origin:'))
|
|
|
|
assert.ok(confirmDataText.includes('127.0.0.1'))
|
|
|
|
assert.ok(confirmDataText.includes('Bytes:'))
|
|
|
|
assert.ok(confirmDataText.includes('675'))
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//li[contains(text(), 'Details')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('confirms a deploy contract transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 6
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txAction = await driver.findElements(By.css('.list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(txAction[0], /Contract\sDeployment/), 10000)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('calls and confirms a contract method where ETH is sent', async function () {
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
let contractStatus = await driver.findElement(By.css('#contractStatus'))
|
|
|
|
await driver.wait(until.elementTextMatches(contractStatus, /Deployed/), 15000)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('#depositButton'))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
|
|
|
|
contractStatus = await driver.findElement(By.css('#contractStatus'))
|
|
|
|
await driver.wait(until.elementTextMatches(contractStatus, /Deposit\sinitiated/), 10000)
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
|
|
|
|
await driver.findElements(By.css('.transaction-list-item'))
|
|
|
|
const txListValue = await driver.findClickableElement(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txListValue, /-4\s*ETH/), 10000)
|
|
|
|
await txListValue.click()
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
// Set the gas limit
|
|
|
|
await driver.clickElement(By.css('.confirm-detail-row__header-text--edit'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const gasModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
await driver.clickElement(By.css('.page-container__tab:nth-of-type(2)'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const [gasPriceInput, gasLimitInput] = await driver.findElements(By.css('.advanced-gas-inputs__gas-edit-row__input'))
|
|
|
|
const gasLimitValue = await gasLimitInput.getAttribute('value')
|
|
|
|
assert(Number(gasLimitValue) < 100000, 'Gas Limit too high')
|
|
|
|
|
|
|
|
await gasPriceInput.clear()
|
|
|
|
await driver.delay(50)
|
|
|
|
await gasPriceInput.sendKeys('10')
|
|
|
|
await driver.delay(50)
|
|
|
|
|
|
|
|
await gasLimitInput.clear()
|
|
|
|
await driver.delay(50)
|
|
|
|
await gasLimitInput.sendKeys('60001')
|
Use `AdvancedGasInputs` in `AdvancedTabContent` (#7186)
* Use `AdvancedGasInputs` in `AdvancedTabContent`
The `AdvancedGasInputs` component was originally extracted from the
`AdvancedTabContent` component, duplicating much of the rendering
logic. They have since evolved separately, with bugs being fixed in one
place but not the other.
The inputs and outputs expected weren't exactly the same, as the
`AdvancedGasInputs` component converts the input custom gas price and
limit, and it converts them both in the setter methods as well.
The `GasModalPageContainer` had to be adjusted to avoid converting
these values multiple times.
Both components dealt with input debouncing separately, both in less
than ideal ways. `AdvancedTabContent` didn't debounce either field, but
it did debounce the check for whether the gas limit field was below the
minimum value. So if a less-than-minimum value was set, it would be
propogated upwards and could be saved if the user clicked 'Save'
quickly enough. After a second delay it would snap back to the minimum
value. The `AdvancedGasInputs` component debounced both fields, but
it would replace any gas limit below the minimum with the minimum
value. This led to a problem where a brief pause during typing would
reset the field to 21000.
The `AdvancedGasInputs` approach was chosen, except that it was
updated to no longer change the gas limit if it was below the minimum.
Instead it displays an error. Parent components were checked to ensure
they would detect the error case of the gas limit being set too low,
and prevent the form submission in those cases. Of the three parents,
one had already dealt with it correctly, one needed to convert the
gas limit from hex first, and another needed the gas limit check added.
Closes #6872
* Cleanup send components
Empty README files have been removed, and a mistake in the index file
for the send page has been corrected. The Gas Slider component class
name was updated as well; it looks like it was originally created from
`AdvancedTabContent`, so it still had that class name.
5 years ago
|
|
|
|
|
|
|
await driver.delay(1000)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Save')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.wait(until.stalenessOf(gasModal))
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 7
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValues = await driver.findElements(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txValues[0], /-4\s*ETH/), 10000)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('calls and confirms a contract method where ETH is received', async function () {
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('#withdrawButton'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.transaction-list__pending-transactions .transaction-list-item'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 8
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValues = await driver.findElement(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txValues, /-0\s*ETH/), 10000)
|
|
|
|
|
|
|
|
await driver.closeAllWindowHandlesExcept([extension, dapp])
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('renders the correct ETH balance', async function () {
|
|
|
|
const balance = await driver.findElement(By.css('[data-testid="eth-overview__primary-currency"]'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
await driver.wait(until.elementTextMatches(balance, /^87.*\s*ETH.*$/), 10000)
|
|
|
|
const tokenAmount = await balance.getText()
|
|
|
|
assert.ok(/^87.*\s*ETH.*$/.test(tokenAmount))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Add a custom token from a dapp', function () {
|
|
|
|
it('creates a new token', async function () {
|
|
|
|
let windowHandles = await driver.getAllWindowHandles()
|
|
|
|
const extension = windowHandles[0]
|
|
|
|
const dapp = windowHandles[1]
|
|
|
|
await driver.delay(regularDelayMs * 2)
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(regularDelayMs * 2)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Create Token')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
|
|
|
|
windowHandles = await driver.getAllWindowHandles()
|
|
|
|
const popup = windowHandles[2]
|
|
|
|
await driver.switchToWindow(popup)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.confirm-detail-row__header-text--edit'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//li[contains(text(), 'Advanced')]`))
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
|
|
|
|
const [gasPriceInput, gasLimitInput] = await driver.findElements(By.css('.advanced-gas-inputs__gas-edit-row__input'))
|
|
|
|
assert(gasPriceInput.getAttribute('value'), 20)
|
|
|
|
assert(gasLimitInput.getAttribute('value'), 4700000)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Save')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
|
|
|
|
const tokenContractAddress = await driver.findElement(By.css('#tokenAddress'))
|
|
|
|
await driver.wait(until.elementTextMatches(tokenContractAddress, /0x/))
|
|
|
|
tokenAddress = await tokenContractAddress.getText()
|
|
|
|
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
await driver.closeAllWindowHandlesExcept([extension, dapp])
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('clicks on the Add Token button', async function () {
|
|
|
|
await driver.clickElement(By.css(`[data-testid="home__asset-tab"]`))
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Add Token')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('picks the newly created Test token', async function () {
|
|
|
|
await driver.clickElement(By.xpath("//li[contains(text(), 'Custom Token')]"))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const newTokenAddress = await driver.findElement(By.css('#custom-address'))
|
|
|
|
await newTokenAddress.sendKeys(tokenAddress)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Next')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Add Tokens')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('renders the balance for the new token', async function () {
|
|
|
|
const balance = await driver.findElement(By.css('.wallet-overview .token-overview__primary-balance'))
|
|
|
|
await driver.wait(until.elementTextMatches(balance, /^10\s*TST\s*$/))
|
|
|
|
const tokenAmount = await balance.getText()
|
|
|
|
assert.ok(/^10\s*TST\s*$/.test(tokenAmount))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Send token from inside MetaMask', function () {
|
|
|
|
let gasModal
|
|
|
|
it('starts to send a transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Send')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const inputAddress = await driver.findElement(By.css('input[placeholder="Search, public address (0x), or ENS"]'))
|
|
|
|
await inputAddress.sendKeys('0x2f318C334780961FB129D2a6c30D0763d9a5C970')
|
|
|
|
|
|
|
|
const inputAmount = await driver.findElement(By.css('.unit-input__input'))
|
|
|
|
await inputAmount.sendKeys('1')
|
|
|
|
|
|
|
|
// Set the gas limit
|
|
|
|
await driver.clickElement(By.css('.advanced-gas-options-btn'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
gasModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('opens customize gas modal', async function () {
|
|
|
|
await driver.findElement(By.css('.page-container__title'))
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Save')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('transitions to the confirm screen', async function () {
|
|
|
|
await driver.wait(until.stalenessOf(gasModal))
|
|
|
|
|
|
|
|
// Continue to next screen
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Next')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('displays the token transfer data', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//li[contains(text(), 'Data')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const functionType = await driver.findElement(By.css('.confirm-page-container-content__function-type'))
|
|
|
|
const functionTypeText = await functionType.getText()
|
|
|
|
assert.equal(functionTypeText, 'Transfer')
|
|
|
|
|
|
|
|
const tokenAmount = await driver.findElement(By.css('.confirm-page-container-summary__title-text'))
|
|
|
|
const tokenAmountText = await tokenAmount.getText()
|
|
|
|
assert.equal(tokenAmountText, '1 TST')
|
|
|
|
|
|
|
|
const confirmDataDiv = await driver.findElement(By.css('.confirm-page-container-content__data-box'))
|
|
|
|
const confirmDataText = await confirmDataDiv.getText()
|
|
|
|
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
assert(confirmDataText.match(/0xa9059cbb0000000000000000000000002f318c334780961fb129d2a6c30d0763d9a5c97/))
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//li[contains(text(), 'Details')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('submits the transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 1
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValues = await driver.findElements(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
assert.equal(txValues.length, 1)
|
|
|
|
await driver.wait(until.elementTextMatches(txValues[0], /-1\s*TST/), 10000)
|
|
|
|
|
|
|
|
const txStatuses = await driver.findElements(By.css('.list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(txStatuses[0], /Send\sTST/i), 10000)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Send a custom token from dapp', function () {
|
|
|
|
let gasModal
|
|
|
|
it('sends an already created token', async function () {
|
|
|
|
const windowHandles = await driver.getAllWindowHandles()
|
|
|
|
const extension = windowHandles[0]
|
|
|
|
const dapp = await driver.switchToWindowWithTitle('E2E Test Dapp', windowHandles)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Transfer Tokens')]`))
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
|
|
|
|
await driver.findElements(By.css('.transaction-list__pending-transactions'))
|
|
|
|
const txListValue = await driver.findClickableElement(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txListValue, /-1.5\s*TST/), 10000)
|
|
|
|
await txListValue.click()
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const transactionAmounts = await driver.findElements(By.css('.currency-display-component__text'))
|
|
|
|
const transactionAmount = transactionAmounts[0]
|
|
|
|
assert(await transactionAmount.getText(), '1.5 TST')
|
|
|
|
|
|
|
|
// Set the gas limit
|
|
|
|
await driver.clickElement(By.css('.confirm-detail-row__header-text--edit'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
gasModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
})
|
|
|
|
|
|
|
|
it('customizes gas', async function () {
|
|
|
|
await driver.clickElement(By.css('.page-container__tab:nth-of-type(2)'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const [gasPriceInput, gasLimitInput] = await driver.findElements(By.css('.advanced-gas-inputs__gas-edit-row__input'))
|
|
|
|
|
|
|
|
await gasPriceInput.clear()
|
|
|
|
await driver.delay(50)
|
|
|
|
await gasPriceInput.sendKeys('10')
|
|
|
|
await driver.delay(50)
|
|
|
|
|
|
|
|
await gasLimitInput.clear()
|
|
|
|
await driver.delay(50)
|
|
|
|
await gasLimitInput.sendKeys('60000')
|
Use `AdvancedGasInputs` in `AdvancedTabContent` (#7186)
* Use `AdvancedGasInputs` in `AdvancedTabContent`
The `AdvancedGasInputs` component was originally extracted from the
`AdvancedTabContent` component, duplicating much of the rendering
logic. They have since evolved separately, with bugs being fixed in one
place but not the other.
The inputs and outputs expected weren't exactly the same, as the
`AdvancedGasInputs` component converts the input custom gas price and
limit, and it converts them both in the setter methods as well.
The `GasModalPageContainer` had to be adjusted to avoid converting
these values multiple times.
Both components dealt with input debouncing separately, both in less
than ideal ways. `AdvancedTabContent` didn't debounce either field, but
it did debounce the check for whether the gas limit field was below the
minimum value. So if a less-than-minimum value was set, it would be
propogated upwards and could be saved if the user clicked 'Save'
quickly enough. After a second delay it would snap back to the minimum
value. The `AdvancedGasInputs` component debounced both fields, but
it would replace any gas limit below the minimum with the minimum
value. This led to a problem where a brief pause during typing would
reset the field to 21000.
The `AdvancedGasInputs` approach was chosen, except that it was
updated to no longer change the gas limit if it was below the minimum.
Instead it displays an error. Parent components were checked to ensure
they would detect the error case of the gas limit being set too low,
and prevent the form submission in those cases. Of the three parents,
one had already dealt with it correctly, one needed to convert the
gas limit from hex first, and another needed the gas limit check added.
Closes #6872
* Cleanup send components
Empty README files have been removed, and a mistake in the index file
for the send page has been corrected. The Gas Slider component class
name was updated as well; it looks like it was originally created from
`AdvancedTabContent`, so it still had that class name.
5 years ago
|
|
|
|
|
|
|
await driver.delay(1000)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.page-container__footer-button'))
|
|
|
|
await driver.wait(until.stalenessOf(gasModal))
|
|
|
|
|
|
|
|
const gasFeeInputs = await driver.findElements(By.css('.confirm-detail-row__primary'))
|
|
|
|
const renderedGasFee = await gasFeeInputs[0].getText()
|
|
|
|
assert.equal(renderedGasFee, '0.0006')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('submits the transaction', async function () {
|
|
|
|
const tokenAmount = await driver.findElement(By.css('.confirm-page-container-summary__title-text'))
|
|
|
|
const tokenAmountText = await tokenAmount.getText()
|
|
|
|
assert.equal(tokenAmountText, '1.5 TST')
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 2
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValues = await driver.findElements(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txValues[0], /-1.5\s*TST/))
|
|
|
|
const txStatuses = await driver.findElements(By.css('.list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(txStatuses[0], /Send\sTST/), 10000)
|
|
|
|
|
|
|
|
const tokenBalanceAmount = await driver.findElements(By.css('.token-overview__primary-balance'))
|
|
|
|
await driver.wait(until.elementTextMatches(tokenBalanceAmount[0], /7.5\s*TST/), 10000)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Approves a custom token from dapp', function () {
|
|
|
|
let gasModal
|
|
|
|
it('approves an already created token', async function () {
|
|
|
|
const windowHandles = await driver.getAllWindowHandles()
|
|
|
|
const extension = windowHandles[0]
|
|
|
|
const dapp = await driver.switchToWindowWithTitle('E2E Test Dapp', windowHandles)
|
|
|
|
await driver.closeAllWindowHandlesExcept([extension, dapp])
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Approve Tokens')]`))
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const pendingTxes = await driver.findElements(By.css('.transaction-list__pending-transactions .transaction-list-item'))
|
|
|
|
return pendingTxes.length === 1
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const [txtListHeading] = await driver.findElements(By.css('.transaction-list-item .list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(txtListHeading, /Approve TST spend limit/))
|
|
|
|
await driver.clickElement(By.css('.transaction-list-item'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('displays the token approval data', async function () {
|
|
|
|
await driver.clickElement(By.css('.confirm-approve-content__view-full-tx-button'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const functionType = await driver.findElement(By.css('.confirm-approve-content__data .confirm-approve-content__small-text'))
|
|
|
|
const functionTypeText = await functionType.getText()
|
|
|
|
assert.equal(functionTypeText, 'Function: Approve')
|
|
|
|
|
|
|
|
const confirmDataDiv = await driver.findElement(By.css('.confirm-approve-content__data__data-block'))
|
|
|
|
const confirmDataText = await confirmDataDiv.getText()
|
|
|
|
assert(confirmDataText.match(/0x095ea7b30000000000000000000000009bc5baf874d2da8d216ae9f137804184ee5afef4/))
|
|
|
|
})
|
|
|
|
|
|
|
|
it('opens the gas edit modal', async function () {
|
|
|
|
await driver.clickElement(By.css('.confirm-approve-content__small-blue-text.cursor-pointer'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
gasModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
})
|
|
|
|
|
|
|
|
it('customizes gas', async function () {
|
|
|
|
await driver.clickElement(By.css('.page-container__tab:nth-of-type(2)'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const [gasPriceInput, gasLimitInput] = await driver.findElements(By.css('.advanced-gas-inputs__gas-edit-row__input'))
|
|
|
|
|
|
|
|
await gasPriceInput.clear()
|
|
|
|
await driver.delay(50)
|
|
|
|
await gasPriceInput.sendKeys('10')
|
|
|
|
await driver.delay(50)
|
|
|
|
|
|
|
|
await gasLimitInput.clear()
|
|
|
|
await driver.delay(50)
|
|
|
|
await gasLimitInput.sendKeys('60001')
|
Use `AdvancedGasInputs` in `AdvancedTabContent` (#7186)
* Use `AdvancedGasInputs` in `AdvancedTabContent`
The `AdvancedGasInputs` component was originally extracted from the
`AdvancedTabContent` component, duplicating much of the rendering
logic. They have since evolved separately, with bugs being fixed in one
place but not the other.
The inputs and outputs expected weren't exactly the same, as the
`AdvancedGasInputs` component converts the input custom gas price and
limit, and it converts them both in the setter methods as well.
The `GasModalPageContainer` had to be adjusted to avoid converting
these values multiple times.
Both components dealt with input debouncing separately, both in less
than ideal ways. `AdvancedTabContent` didn't debounce either field, but
it did debounce the check for whether the gas limit field was below the
minimum value. So if a less-than-minimum value was set, it would be
propogated upwards and could be saved if the user clicked 'Save'
quickly enough. After a second delay it would snap back to the minimum
value. The `AdvancedGasInputs` component debounced both fields, but
it would replace any gas limit below the minimum with the minimum
value. This led to a problem where a brief pause during typing would
reset the field to 21000.
The `AdvancedGasInputs` approach was chosen, except that it was
updated to no longer change the gas limit if it was below the minimum.
Instead it displays an error. Parent components were checked to ensure
they would detect the error case of the gas limit being set too low,
and prevent the form submission in those cases. Of the three parents,
one had already dealt with it correctly, one needed to convert the
gas limit from hex first, and another needed the gas limit check added.
Closes #6872
* Cleanup send components
Empty README files have been removed, and a mistake in the index file
for the send page has been corrected. The Gas Slider component class
name was updated as well; it looks like it was originally created from
`AdvancedTabContent`, so it still had that class name.
5 years ago
|
|
|
|
|
|
|
await driver.delay(1000)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.page-container__footer-button'))
|
|
|
|
await driver.wait(until.stalenessOf(gasModal))
|
|
|
|
|
|
|
|
const gasFeeInEth = await driver.findElement(By.css('.confirm-approve-content__transaction-details-content__secondary-fee'))
|
|
|
|
assert.equal(await gasFeeInEth.getText(), '0.0006 ETH')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('edits the permission', async function () {
|
|
|
|
const editButtons = await driver.findClickableElements(By.css('.confirm-approve-content__small-blue-text.cursor-pointer'))
|
|
|
|
await editButtons[1].click()
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const permissionModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
|
|
|
|
const radioButtons = await driver.findClickableElements(By.css('.edit-approval-permission__edit-section__radio-button'))
|
|
|
|
await radioButtons[1].click()
|
|
|
|
|
|
|
|
const customInput = await driver.findElement(By.css('input'))
|
|
|
|
await driver.delay(50)
|
|
|
|
await customInput.sendKeys('5')
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Save')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.wait(until.stalenessOf(permissionModal))
|
|
|
|
|
|
|
|
const permissionInfo = await driver.findElements(By.css('.confirm-approve-content__medium-text'))
|
|
|
|
const amountDiv = permissionInfo[0]
|
|
|
|
assert.equal(await amountDiv.getText(), '5 TST')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('submits the transaction', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 3
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txStatuses = await driver.findElements(By.css('.list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(txStatuses[0], /Approve TST spend limit/))
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Transfers a custom token from dapp when no gas value is specified', function () {
|
|
|
|
it('transfers an already created token, without specifying gas', async function () {
|
|
|
|
const windowHandles = await driver.getAllWindowHandles()
|
|
|
|
const extension = windowHandles[0]
|
|
|
|
const dapp = await driver.switchToWindowWithTitle('E2E Test Dapp', windowHandles)
|
|
|
|
await driver.closeAllWindowHandlesExcept([extension, dapp])
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Transfer Tokens Without Gas')]`))
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const pendingTxes = await driver.findElements(By.css('.transaction-list__pending-transactions .transaction-list-item'))
|
|
|
|
return pendingTxes.length === 1
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const [txListValue] = await driver.findElements(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txListValue, /-1.5\s*TST/))
|
|
|
|
await driver.clickElement(By.css('.transaction-list-item'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('submits the transaction', async function () {
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 4
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txValues = await driver.findElements(By.css('.transaction-list-item__primary-currency'))
|
|
|
|
await driver.wait(until.elementTextMatches(txValues[0], /-1.5\s*TST/))
|
|
|
|
const txStatuses = await driver.findElements(By.css('.list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(txStatuses[0], /Send TST/))
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Approves a custom token from dapp when no gas value is specified', function () {
|
|
|
|
it('approves an already created token', async function () {
|
|
|
|
const windowHandles = await driver.getAllWindowHandles()
|
|
|
|
const extension = windowHandles[0]
|
|
|
|
const dapp = await driver.switchToWindowWithTitle('E2E Test Dapp', windowHandles)
|
|
|
|
await driver.closeAllWindowHandlesExcept([extension, dapp])
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.switchToWindow(dapp)
|
|
|
|
await driver.delay(tinyDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Approve Tokens Without Gas')]`))
|
|
|
|
|
|
|
|
await driver.switchToWindow(extension)
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const pendingTxes = await driver.findElements(By.css('.transaction-list__pending-transactions .transaction-list-item'))
|
|
|
|
return pendingTxes.length === 1
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const [txtListHeading] = await driver.findElements(By.css('.transaction-list-item .list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(txtListHeading, /Approve TST spend limit/))
|
|
|
|
await driver.clickElement(By.css('.transaction-list-item'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('shows the correct recipient', async function () {
|
|
|
|
await driver.clickElement(By.css('.confirm-approve-content__view-full-tx-button'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const permissionInfo = await driver.findElements(By.css('.confirm-approve-content__medium-text'))
|
|
|
|
const recipientDiv = permissionInfo[1]
|
|
|
|
assert.equal(await recipientDiv.getText(), '0x2f318C33...C970')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('submits the transaction', async function () {
|
|
|
|
await driver.delay(1000)
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Confirm')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds the transaction in the transactions list', async function () {
|
|
|
|
await driver.wait(async () => {
|
|
|
|
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
|
|
|
|
return confirmedTxes.length === 5
|
|
|
|
}, 10000)
|
|
|
|
|
|
|
|
const txStatuses = await driver.findElements(By.css('.list-item__heading'))
|
|
|
|
await driver.wait(until.elementTextMatches(txStatuses[0], /Approve TST spend limit/))
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Hide token', function () {
|
|
|
|
it('hides the token when clicked', async function () {
|
|
|
|
await driver.clickElement(By.css('[data-testid="token-options__button"]'))
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="token-options__hide"]'))
|
|
|
|
|
|
|
|
const confirmHideModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('[data-testid="hide-token-confirmation__hide"]'))
|
|
|
|
|
|
|
|
await driver.wait(until.stalenessOf(confirmHideModal))
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Add existing token using search', function () {
|
|
|
|
it('clicks on the Add Token button', async function () {
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Add Token')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('can pick a token from the existing options', async function () {
|
|
|
|
const tokenSearch = await driver.findElement(By.css('#search-tokens'))
|
|
|
|
await tokenSearch.sendKeys('BAT')
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath("//span[contains(text(), 'BAT')]"))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Next')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//button[contains(text(), 'Add Tokens')]`))
|
|
|
|
await driver.delay(largeDelayMs)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('renders the balance for the chosen token', async function () {
|
|
|
|
const balance = await driver.findElement(By.css('.token-overview__primary-balance'))
|
|
|
|
await driver.wait(until.elementTextMatches(balance, /0\s*BAT/))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('Stores custom RPC history', function () {
|
|
|
|
const customRpcUrls = [
|
|
|
|
'http://127.0.0.1:8545/1',
|
|
|
|
'http://127.0.0.1:8545/2',
|
|
|
|
'http://127.0.0.1:8545/3',
|
|
|
|
'http://127.0.0.1:8545/4',
|
|
|
|
]
|
|
|
|
|
|
|
|
customRpcUrls.forEach((customRpcUrl) => {
|
|
|
|
it(`creates custom RPC: ${customRpcUrl}`, async function () {
|
|
|
|
await driver.clickElement(By.css('.network-name'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//span[contains(text(), 'Custom RPC')]`))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.findElement(By.css('.settings-page__sub-header-text'))
|
|
|
|
|
|
|
|
const customRpcInputs = await driver.findElements(By.css('input[type="text"]'))
|
|
|
|
const customRpcInput = customRpcInputs[1]
|
|
|
|
await customRpcInput.clear()
|
|
|
|
await customRpcInput.sendKeys(customRpcUrl)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.network-form__footer .btn-secondary'))
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
it('selects another provider', async function () {
|
|
|
|
await driver.clickElement(By.css('.network-name'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
await driver.clickElement(By.xpath(`//span[contains(text(), 'Main Ethereum Network')]`))
|
|
|
|
await driver.delay(largeDelayMs * 2)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('finds all recent RPCs in history', async function () {
|
|
|
|
await driver.clickElement(By.css('.network-name'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
// only recent 3 are found and in correct order (most recent at the top)
|
|
|
|
const customRpcs = await driver.findElements(By.xpath(`//span[contains(text(), 'http://127.0.0.1:8545/')]`))
|
|
|
|
|
|
|
|
assert.equal(customRpcs.length, customRpcUrls.length)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('deletes a custom RPC', async function () {
|
|
|
|
const networkListItems = await driver.findClickableElements(By.css('.networks-tab__networks-list-name'))
|
|
|
|
const lastNetworkListItem = networkListItems[networkListItems.length - 1]
|
|
|
|
await lastNetworkListItem.click()
|
|
|
|
await driver.delay(100)
|
|
|
|
|
|
|
|
await driver.clickElement(By.css('.btn-danger'))
|
|
|
|
await driver.delay(regularDelayMs)
|
|
|
|
|
|
|
|
const confirmDeleteNetworkModal = await driver.findElement(By.css('span .modal'))
|
|
|
|
|
|
|
|
const byConfirmDeleteNetworkButton = By.css('.button.btn-danger.modal-container__footer-button')
|
|
|
|
await driver.clickElement(byConfirmDeleteNetworkButton)
|
|
|
|
|
|
|
|
await driver.wait(until.stalenessOf(confirmDeleteNetworkModal))
|
|
|
|
|
|
|
|
const newNetworkListItems = await driver.findElements(By.css('.networks-tab__networks-list-name'))
|
|
|
|
|
|
|
|
assert.equal(networkListItems.length - 1, newNetworkListItems.length)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
})
|