fix: always get the executor's homedir (#17)

lib/utils/index.js

@@ -1,8 +1,9 @@
 'use strict';
 
 const interopRequire = require('interop-require');
+const homedir = require('node-homedir');
 
-module.exports = exports = {
+module.exports = {
 
   loadFile(filepath) {
     let exports;
@@ -25,7 +26,7 @@ module.exports = exports = {
   },
 
   getHomedir() {
-    return process.env.HOME || '/home/admin';
+    return homedir() || '/home/admin';
   },
 
   methods: [ 'head', 'options', 'get', 'put', 'patch', 'post', 'delete' ],

package.json

@@ -55,6 +55,7 @@
     "is-type-of": "^1.0.0",
     "koa": "^1.2.1",
     "koa-router": "^5.4.0",
+    "node-homedir": "^0.0.1",
     "ready-callback": "^1.0.0",
     "utility": "^1.8.0"
   }

test/utils/index.test.js

@@ -1,20 +1,31 @@
 'use strict';
 
 const mm = require('mm');
+const os = require('os');
 const utils = require('../../lib/utils');
+const should = require('should');
 
 describe('test/utils/index.test.js', () => {
 
   afterEach(mm.restore);
 
   describe('utils.getHomedir()', () => {
     it('should return process.env.HOME', () => {
+      if (os.userInfo && os.userInfo().homedir) {
+        const userInfo = os.userInfo();
+        delete userInfo.homedir;
+        mm(os, 'userInfo', () => userInfo);
+      }
       utils.getHomedir().should.equal(process.env.HOME);
     });
 
     it('should return /home/admin when process.env.HOME is not exist', () => {
       mm(process.env, 'HOME', '');
-      utils.getHomedir().should.equal('/home/admin');
+      if (os.userInfo && os.userInfo().homedir) {
+        should.ok(utils.getHomedir().indexOf(process.env.USER) > -1);
+      } else {
+        utils.getHomedir().should.equal('/home/admin');
+      }
     });
   });
 });