From ceea1e845adb41f28568287708e956df682d1f7b Mon Sep 17 00:00:00 2001 From: Alexis Campailla Date: Tue, 10 Dec 2013 06:52:27 -0800 Subject: [PATCH] test: test-os win32 adhere standard tmp resolution The test is making the wrong assumptions about the value of os.tmpdir() on Windows --- test/simple/test-os.js | 24 ++++++++++++++++-------- 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/test/simple/test-os.js b/test/simple/test-os.js index 0060e47..02ea894 100644 --- a/test/simple/test-os.js +++ b/test/simple/test-os.js @@ -30,14 +30,22 @@ var os = require('os'); process.env.TMPDIR = '/tmpdir'; process.env.TMP = '/tmp'; process.env.TEMP = '/temp'; -var t = ( process.platform === 'win32' ? 'c:\\windows\\temp' : '/tmp' ); -assert.equal(os.tmpdir(), '/tmpdir'); -process.env.TMPDIR = ''; -assert.equal(os.tmpdir(), '/tmp'); -process.env.TMP = ''; -assert.equal(os.tmpdir(), '/temp'); -process.env.TEMP = ''; -assert.equal(os.tmpdir(), t); +if (process.platform === 'win32') { + assert.equal(os.tmpdir(), '/temp'); + process.env.TEMP = ''; + assert.equal(os.tmpdir(), '/tmp'); + process.env.TMP = ''; + var expected = (process.env.SystemRoot || process.env.windir) + '\\temp'; + assert.equal(os.tmpdir(), expected); +} else { + assert.equal(os.tmpdir(), '/tmpdir'); + process.env.TMPDIR = ''; + assert.equal(os.tmpdir(), '/tmp'); + process.env.TMP = ''; + assert.equal(os.tmpdir(), '/temp'); + process.env.TEMP = ''; + assert.equal(os.tmpdir(), '/tmp'); +} var endianness = os.endianness(); console.log('endianness = %s', endianness); -- 2.7.4