Fix tmp directory path

Author: marcbachmann

lib/scripts/pdf_a4_portrait.js

@@ -62,7 +62,7 @@ page.onLoadFinished = function (status) {
     quality: options.quality || 75
   }
 
-  var filename = options.filename || (options.directory || '/tmp' + '/html-pdf-' + system.pid + '.' + fileOptions.type)
+  var filename = options.filename || (options.directory || '/tmp') + '/html-pdf-' + system.pid + '.' + fileOptions.type
   page.render(filename, fileOptions)
   system.stdout.write(JSON.stringify({filename: filename}))
 

test/index.js

@@ -53,7 +53,7 @@ test('pdf.create(html[, options]).toFile([filename, ]callback)', function (t) {
   })
 })
 
-test('pdf.create(html[, options]).toBuffer(callback)', function (t) {
+test('pdf.create(html).toBuffer(callback)', function (t) {
   t.plan(3)
 
   pdf.create(html).toBuffer(function (err, pdf) {
@@ -63,6 +63,15 @@ test('pdf.create(html[, options]).toBuffer(callback)', function (t) {
   })
 })
 
+test('pdf.create(html, {directory: "/tmp"}).toBuffer(callback)', function (t) {
+  t.plan(2)
+
+  pdf.create(html, {directory: '/tmp'}).toBuffer(function (err, pdf) {
+    t.error(err)
+    t.assert(Buffer.isBuffer(pdf), "uses the passed directory as tmp dir")
+  })
+})
+
 test('pdf.create(html[, options]).toStream(callback)', function (t) {
   t.plan(3)