Skip to content

Commit

Permalink
test: unref stdin after tests
Browse files Browse the repository at this point in the history
Apparently when stdin is a pipe, it's not unreffing?

Probably a bug in node.
  • Loading branch information
isaacs committed Jul 15, 2013
1 parent 6dad823 commit 191baae
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/basic.js
Expand Up @@ -52,6 +52,8 @@ function child () {
pass: pass,
verify: pass2,
passMatch: (pass === pass2)}))
if (process.stdin.unref)
process.stdin.unref()
})
})
})
Expand Down
2 changes: 2 additions & 0 deletions test/defaults.js
Expand Up @@ -52,6 +52,8 @@ function child () {
pass: pass,
verify: pass2,
passMatch: (pass === pass2)}))
if (process.stdin.unref)
process.stdin.unref()
})
})
})
Expand Down
2 changes: 2 additions & 0 deletions test/many.js
Expand Up @@ -32,6 +32,8 @@ function child () {
read({prompt:'18'}, function (er, r18) {if (er) throw er
console.log(r1, r2, r3, r4, r5, r6, r7, r8, r9, r10,
r11, r12, r13, r14, r15, r16, r17, r18)
if (process.stdin.unref)
process.stdin.unref()
})})})})})})})})})})})})})})})})})})
}

Expand Down

0 comments on commit 191baae

Please sign in to comment.