Merge pull request benjamingr#8 from piranna/master

Pass arguments to `tmp`

Author: benjamingr

index.js

@@ -2,9 +2,10 @@ var tmp = require("tmp");
 var Promise = require("bluebird");
 
 
-// file 
+// file
 module.exports.fileSync = tmp.fileSync;
 var file = Promise.promisify(tmp.file, {multiArgs: true});
+
 module.exports.file = function file$promise() {
   return file.apply(tmp, arguments).spread(function (path, fd, cleanup) {
     return {path: path, fd: fd, cleanup : cleanup };
@@ -13,16 +14,21 @@ module.exports.file = function file$promise() {
 
 module.exports.withFile = function withFile(fn) {
   var cleanup;
-  return module.exports.file.apply(tmp).then(function context(o) { 
+
+  var params = Array.prototype.slice.call(arguments, 1);
+
+  return module.exports.file.apply(tmp, params).then(function context(o) {
     cleanup = o.cleanup;
     delete o.cleanup;
     return fn(o);
   }).finally(cleanup);
 };
 
-// directory 
+
+// directory
 module.exports.dirSync = tmp.dirSync;
 var dir = Promise.promisify(tmp.dir, {multiArgs: true});
+
 module.exports.dir = function dir$promise() {
   return dir.apply(tmp, arguments).spread(function (path, cleanup) {
     return {path: path, cleanup: cleanup};
@@ -31,19 +37,21 @@ module.exports.dir = function dir$promise() {
 
 module.exports.withDir = function withDir(fn) {
   var cleanup;
-  return module.exports.dir.apply(tmp, arguments).then(function context(o) { 
+
+  var params = Array.prototype.slice.call(arguments, 1);
+
+  return module.exports.dir.apply(tmp, params).then(function context(o) {
     cleanup = o.cleanup;
     delete o.cleanup;
     return fn(o);
   }).finally(cleanup);
 };
 
+
 // name generation
 module.exports.tmpNameSync = tmp.tmpNameSync;
 module.exports.tmpName = Promise.promisify(tmp.tmpName);
 
-
 module.exports.tmpdir = tmp.tmpdir;
 
-
 module.exports.setGracefulCleanup = tmp.setGracefulCleanup;

package.json

@@ -4,7 +4,7 @@
   "description": "The tmp package with promises support and disposers.",
   "main": "index.js",
   "scripts": {
-    "test": ""
+    "test": "mocha"
   },
   "keywords": [
     "tmp",
@@ -22,5 +22,8 @@
   "dependencies": {
     "bluebird": "^3.3.1",
     "tmp": "0.0.28"
+  },
+  "devDependencies": {
+    "mocha": "^3.1.2"
   }
 }

test.js

@@ -0,0 +1,53 @@
+var accessSync = require('fs').accessSync
+var assert = require('assert')
+var extname = require('path').extname
+
+var tmp = require('.')
+
+
+describe('withFile', function()
+{
+  it("file don't exists after going out of scope", function()
+  {
+    var filepath
+
+    return tmp.withFile(function(o)
+    {
+      filepath = o.path
+
+      accessSync(filepath)
+      assert.strictEqual(extname(filepath), '.txt')
+    }, {postfix: '.txt'})
+    .then(function()
+    {
+      assert.throws(function()
+      {
+        accessSync(filepath)
+      })
+    })
+  })
+})
+
+
+describe('withDir', function()
+{
+  it("dir don't exists after going out of scope", function()
+  {
+    var filepath
+
+    return tmp.withDir(function(o)
+    {
+      filepath = o.path
+
+      accessSync(filepath)
+      assert.strictEqual(extname(filepath), '.dir')
+    }, {postfix: '.dir'})
+    .then(function()
+    {
+      assert.throws(function()
+      {
+        accessSync(filepath)
+      })
+    })
+  })
+})