-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Resolve #1594, #1603 #1592
Resolve #1594, #1603 #1592
Conversation
|
||
class RequestInterceptor { | ||
intercept = false | ||
byPassMatchObj = null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add flow type definition
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
class RequestInterceptor { | ||
intercept = false | ||
byPassMatchObj = null | ||
stats = {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add flow type definition
intercept = false | ||
byPassMatchObj = null | ||
stats = {} | ||
cache = {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add flow type definition
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -104,7 +104,7 @@ const CRYPTO_ACCOUNTS = [ | |||
// ledger | |||
{ | |||
addedAt: 1583802157, | |||
address: '0xd3ced3b16c8977ed0e345d162d982b899e978588', | |||
address: '0x6F6a68eC176AAfFE76e6771F2Ea79607f6B79801', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the reason of making this change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bypass 0xd3ced3b16c8977ed0e345d162d982b899e978588 for allowance setting for metamask, but we do not want to bypass other types of wallets. This further reduced the number of calls.
@@ -34,3 +34,6 @@ page | |||
.on('requestfailed', request => | |||
console.log(chalk.magenta(`${request.failure().errorText} ${request.url()}`)) | |||
) | |||
|
|||
const RequestInterceptor = require('../utils/requestInterceptor').default | |||
global.requestInterceptor = new RequestInterceptor() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a new line at the end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
}), | ||
page.goto(`${process.env.E2E_TEST_URL}/contacts`) | ||
]) | ||
//await requestInterceptor.setRequestInterception(true) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove if not used
@@ -7,7 +7,15 @@ const timeout = 180000 | |||
|
|||
describe('Direct transfer form tests', () => { | |||
beforeAll(async () => { | |||
/* browser.on('targetcreated', async (target) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove if not used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
) | ||
} | ||
|
||
byPass (matchObj) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add comment to this function
this.startIntercept() | ||
} | ||
|
||
incrementStat (key, val) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add comment to this function
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
a845858
to
514a938
Compare
No description provided.