Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

chore: fix flaky test #3680

Merged
merged 1 commit into from
May 10, 2021
Merged

chore: fix flaky test #3680

merged 1 commit into from
May 10, 2021

Conversation

achingbrain
Copy link
Member

Because the requests are initiated with Promise.all, there's no
guarantee which order they arrive at the server in so when we check
the responses, just make sure the length and contents are correct
instead of the order.

Because the requests are initiated with `Promise.all`, there's no
guarentee which order they arrive at the server in so when we check
the responses, just make sure the length and contents are correct
instead of the order.
Copy link
Member

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@achingbrain achingbrain merged commit 71e5f53 into master May 10, 2021
@achingbrain achingbrain deleted the chore/fix-flaky-test branch May 10, 2021 11:34
SgtPooki referenced this pull request in ipfs/js-kubo-rpc-client Aug 18, 2022
Because the requests are initiated with `Promise.all`, there's no
guarantee which order they arrive at the server in so when we check
the responses, just make sure the length and contents are correct
instead of the order.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants