-
Notifications
You must be signed in to change notification settings - Fork 41
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
Omit erroneous overview elements #1393
Conversation
Pull Request Test Coverage Report for Build 8686808883Details
💛 - Coveralls |
it('forwards error responses from the Config Service', async () => { | ||
const chain = chainBuilder().with('chainId', '10').build(); | ||
const safeInfo = safeBuilder().build(); | ||
it('omits erroroneous Safes on chains returning an error from the Config Service', async () => { |
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.
Nit: erroneous
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 in 56fee09.
it('forwards error responses from the Transaction Service', async () => { | ||
const chain = chainBuilder().with('chainId', '10').build(); | ||
const safeInfo = safeBuilder().build(); | ||
it('omits erroroneous Safes from the Transaction Service', async () => { |
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.
Nit: erroneous
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 in 56fee09.
const tokenAddress = faker.finance.ethereumAddress(); | ||
const secondTokenAddress = faker.finance.ethereumAddress(); | ||
const transactionApiBalancesResponse = [ | ||
it('omits erroroneous Safes validation of the Transaction Service return Promise.rejects', async () => { |
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.
Nit: erroneous
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 in 56fee09.
Superb job! I've left a tiny nit thing only. |
Summary
This filters the return array of the Safe overview endpoint (
/v1/safes
), removing any elements that threw. This prevents one thrown retrieval of aSafeOverview
from blocking others.Changes
Promise.all
toPromise.allSettled
'rejected'
elements, logging if existent