Merge pull request #1002 from spmjs/node_0_11

fix isroot testcase for node 0.11
spmjs · Sep 11, 2014 · 6de2171 · 6de2171
2 parents 78415cf + 412a2d3
commit 6de2171
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/lib/sdk/file.js b/lib/sdk/file.js
@@ -6,8 +6,8 @@ var log = require('../utils/log');
 
 var file = module.exports = {};
 
-file.isroot = function(str) {
-  if (process.platform === 'win32') {
+file.isroot = function(str, platform) {
+  if ((platform || process.platform) === 'win32') {
     return path.normalize(str).slice(1, 3) === ':\\';
   } else {
     return str.charAt(0) === '/';

diff --git a/tests/file.js b/tests/file.js
@@ -5,9 +5,8 @@ describe('file', function() {
   it('isroot', function() {
     file.isroot('/').should.eql(true);
     var oldPlatform = process.platform;
-    process.platform = 'win32';
-    file.isroot('C:\\').should.eql(true);
-    file.isroot('D:\\').should.eql(true);
+    file.isroot('C:\\', 'win32').should.eql(true);
+    file.isroot('D:\\', 'win32').should.eql(true);
     process.platform = oldPlatform;
   });