tests: use temp-path for paths

expressjs · Apr 13, 2017 · 08135aa · 08135aa
1 parent ca3ce4e
commit 08135aa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/package.json b/package.json
@@ -24,7 +24,8 @@
     "eslint-plugin-standard": "2.1.1",
     "istanbul": "0.4.5",
     "mocha": "2.5.3",
-    "supertest": "1.1.0"
+    "supertest": "1.1.0",
+    "temp-path": "1.0.0"
   },
   "files": [
     "LICENSE",

diff --git a/test/support/tempIcon.js b/test/support/tempIcon.js
@@ -1,9 +1,7 @@
 
 var crypto = require('crypto')
 var fs = require('fs')
-var os = require('os')
-var path = require('path')
-var tmpdir = os.tmpdir || os.tmpDir
+var tempPath = require('temp-path')
 
 module.exports = TempIcon
 
@@ -28,8 +26,3 @@ TempIcon.prototype.writeSync = function writeSync () {
     this.exists = true
   }
 }
-
-function tempPath () {
-  var fileName = 'favicon-' + (Date.now() + Math.random()) + '.ico'
-  return path.join(tmpdir(), fileName)
-}