Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

feat: add support for async/await setup in the swarm tests #529

Merged
merged 5 commits into from
Sep 16, 2019

Conversation

hugomrdias
Copy link
Contributor

@hugomrdias hugomrdias commented Sep 15, 2019

This PR increases a couples of timeout and adds support for async/await setup in the swarm tests

More PRs will follow to move everything to async/await setup

the following PRs add support async setup:
ipfs-inactive/js-ipfs-http-client#1105
ipfs/js-ipfs#2428

@hugomrdias hugomrdias marked this pull request as ready for review September 16, 2019 11:41
Copy link
Collaborator

@achingbrain achingbrain left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -8,13 +8,13 @@ module.exports = (createCommon, options) => {
const it = getIt(options)
const common = createCommon()

describe('.id', () => {
describe('.id', function () {
this.timeout(60 * 1000)
let ipfs

before(function (done) {
// CI takes longer to instantiate the daemon, so we need to increase the
// timeout for the before step
Copy link
Collaborator

Choose a reason for hiding this comment

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

This comment can probably go if we're removing the this.timeout

@hugomrdias hugomrdias changed the title Fix/fix electron feat: add support for async/await setup in the swarm tests Sep 16, 2019
@hugomrdias hugomrdias merged commit 044409c into master Sep 16, 2019
@hugomrdias hugomrdias deleted the fix/fix-electron branch September 16, 2019 14:01
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