From bae8ef8b70f8799358fb2fa5ab85683a396c4f5c Mon Sep 17 00:00:00 2001 From: Mark Stacey Date: Thu, 30 Apr 2020 11:39:25 -0300 Subject: [PATCH] Fix broken test artifact directory name (#8480) The test artifact directory for failed test "verbose reports" was mistakenly being set to `[browser]/undefined`. This was broken during the refactor in #7798, when the `driver` parameter was mistakenly left in after the `verboseReportOnFailure` function was converted to a method being called on `driver`. --- test/e2e/address-book.spec.js | 2 +- test/e2e/ethereum-on.spec.js | 2 +- test/e2e/from-import-ui.spec.js | 2 +- test/e2e/incremental-security.spec.js | 2 +- test/e2e/metamask-responsive-ui.spec.js | 2 +- test/e2e/metamask-ui.spec.js | 2 +- test/e2e/permissions.spec.js | 2 +- test/e2e/send-edit.spec.js | 2 +- test/e2e/signature-request.spec.js | 2 +- test/e2e/threebox.spec.js | 2 +- test/e2e/web3.spec.js | 2 +- 11 files changed, 11 insertions(+), 11 deletions(-) diff --git a/test/e2e/address-book.spec.js b/test/e2e/address-book.spec.js index 6171dc222..e283fb61d 100644 --- a/test/e2e/address-book.spec.js +++ b/test/e2e/address-book.spec.js @@ -43,7 +43,7 @@ describe('MetaMask', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/ethereum-on.spec.js b/test/e2e/ethereum-on.spec.js index 53a87874a..ddc76c23a 100644 --- a/test/e2e/ethereum-on.spec.js +++ b/test/e2e/ethereum-on.spec.js @@ -42,7 +42,7 @@ describe('MetaMask', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/from-import-ui.spec.js b/test/e2e/from-import-ui.spec.js index ab8832392..de9c588a5 100644 --- a/test/e2e/from-import-ui.spec.js +++ b/test/e2e/from-import-ui.spec.js @@ -47,7 +47,7 @@ describe('Using MetaMask with an existing account', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/incremental-security.spec.js b/test/e2e/incremental-security.spec.js index 4964dfb0f..8def26110 100644 --- a/test/e2e/incremental-security.spec.js +++ b/test/e2e/incremental-security.spec.js @@ -47,7 +47,7 @@ describe('MetaMask', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/metamask-responsive-ui.spec.js b/test/e2e/metamask-responsive-ui.spec.js index c0c816984..5553e2c8e 100644 --- a/test/e2e/metamask-responsive-ui.spec.js +++ b/test/e2e/metamask-responsive-ui.spec.js @@ -37,7 +37,7 @@ describe('MetaMask', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/metamask-ui.spec.js b/test/e2e/metamask-ui.spec.js index fdf1901b0..9932fc922 100644 --- a/test/e2e/metamask-ui.spec.js +++ b/test/e2e/metamask-ui.spec.js @@ -38,7 +38,7 @@ describe('MetaMask', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/permissions.spec.js b/test/e2e/permissions.spec.js index b032b3418..3b79e603b 100644 --- a/test/e2e/permissions.spec.js +++ b/test/e2e/permissions.spec.js @@ -42,7 +42,7 @@ describe('MetaMask', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/send-edit.spec.js b/test/e2e/send-edit.spec.js index ee91ddecf..c2ba99f95 100644 --- a/test/e2e/send-edit.spec.js +++ b/test/e2e/send-edit.spec.js @@ -44,7 +44,7 @@ describe('Using MetaMask with an existing account', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/signature-request.spec.js b/test/e2e/signature-request.spec.js index c90f37c04..5aeaa67b6 100644 --- a/test/e2e/signature-request.spec.js +++ b/test/e2e/signature-request.spec.js @@ -41,7 +41,7 @@ describe('MetaMask', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/threebox.spec.js b/test/e2e/threebox.spec.js index 85ba75834..57624735f 100644 --- a/test/e2e/threebox.spec.js +++ b/test/e2e/threebox.spec.js @@ -45,7 +45,7 @@ describe('MetaMask', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } }) diff --git a/test/e2e/web3.spec.js b/test/e2e/web3.spec.js index 108fdc8c5..2aae92bc1 100644 --- a/test/e2e/web3.spec.js +++ b/test/e2e/web3.spec.js @@ -44,7 +44,7 @@ describe('Using MetaMask with an existing account', function () { } } if (this.currentTest.state === 'failed') { - await driver.verboseReportOnFailure(driver, this.currentTest) + await driver.verboseReportOnFailure(this.currentTest) } })