Skip to content

Commit c1bd9cf

Browse files
jacobheunvmx
authored andcommitted
test: fix potential done called multiple times issue
1 parent 96dcd41 commit c1bd9cf

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ lib-cov
1616

1717
# Coverage directory used by tools like istanbul
1818
coverage
19+
.nyc_output/
1920

2021
# Grunt intermediate storage (http://gruntjs.com/creating-plugins#storing-task-files)
2122
.grunt

test/wantmanager/index.spec.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,14 +63,16 @@ describe('WantManager', () => {
6363
expect(calls.connects).to.have.length(6)
6464
expect(calls.messages).to.have.length(6)
6565

66-
calls.messages.forEach((m, i) => {
67-
expect(m[0]).to.be.eql(calls.connects[i])
66+
for (let i = 0; i < calls.messages.length; i++) {
67+
const m = calls.messages[i]
68+
const connect = calls.connects[i]
69+
expect(m[0]).to.be.eql(connect)
6870
if (!m[1].equals(msgs[i])) {
6971
return done(
7072
new Error(`expected ${m[1].toString()} to equal ${msgs[i].toString()}`)
7173
)
7274
}
73-
})
75+
}
7476

7577
done()
7678
})

0 commit comments

Comments
 (0)