Clean up temporary file on exit.
authorBen Noordhuis <info@bnoordhuis.nl>
Mon, 4 Jul 2011 23:06:39 +0000 (01:06 +0200)
committerBen Noordhuis <info@bnoordhuis.nl>
Mon, 4 Jul 2011 23:38:07 +0000 (01:38 +0200)
Unbreaks test/simple/test-http-get-pipeline-problem.js,
it assumed a fixed number of files in the tmp directory.

test/simple/test-http-curl-chunk-problem.js

index 794b320..409c769 100644 (file)
@@ -27,12 +27,19 @@ if (!process.versions.openssl) {
 // http://groups.google.com/group/nodejs/browse_thread/thread/f66cd3c960406919
 var common = require('../common');
 var assert = require('assert');
-var http = require('http'),
-    cp = require('child_process');
-
+var http = require('http');
+var cp = require('child_process');
+var fs = require('fs');
 
 var filename = require('path').join(common.tmpDir || '/tmp', 'big');
 
+// Clean up after ourselves. Leaving files around
+// in the tmp/ directory may break tests that depend
+// on a certain number of files being there.
+process.on('exit', function() {
+  fs.unlink(filename);
+});
+
 var count = 0;
 function maybeMakeRequest() {
   if (++count < 2) return;