fix: add async support to setup (#11)

This commit is contained in:
Jacob Heun 2019-09-27 13:32:45 +02:00 committed by Vasco Santos
parent 6bb1f2a4cb
commit 2814c76e23
3 changed files with 11 additions and 5 deletions

View File

@ -38,10 +38,10 @@ Include this badge in your readme if you make a new module that uses interface-p
### Node.js
Install `interface-peer-discovery` as one of the dependencies of your project and as a test file. Then, using `mocha` (for JavaScript) or a test runner with compatible API, do:
Install `interface-discovery` as one of the dependencies of your project and as a test file. Then, using `mocha` (for JavaScript) or a test runner with compatible API, do:
```js
const test = require('interface-peer-discovery')
const test = require('interface-discovery')
const common = {
setup () {

View File

@ -5,12 +5,14 @@ module.exports = (common) => {
describe('interface-peer-discovery', () => {
let discovery
before(() => {
discovery = common.setup()
before(async () => {
discovery = await common.setup()
})
after(() => common.teardown && common.teardown())
afterEach('ensure discovery was stopped', () => discovery.stop())
it('can start the service', async () => {
await discovery.start()
})

View File

@ -6,8 +6,12 @@ const MockDiscovery = require('./mock-discovery')
describe('compliance tests', () => {
tests({
setup () {
async setup () {
await new Promise(resolve => setTimeout(resolve, 10))
return new MockDiscovery()
},
async teardown () {
await new Promise(resolve => setTimeout(resolve, 10))
}
})
})