Fix: Properly handle undefined user & add test

Author: phated

index.js

@@ -8,7 +8,7 @@ const path = require('path');
 const crypto = require('crypto');
 const execFile = require('child_process').execFile;
 const env = process.env;
-const user = env.LOGNAME || env.USER || env.LNAME || env.USERNAME;
+const user = env.LOGNAME || env.USER || env.LNAME || env.USERNAME || '';
 const exclusions = ['--help'];
 
 const configfile = '.v8flags.'+process.versions.v8+'.'+crypto.createHash('md5').update(user).digest('hex')+'.json';

test.js

@@ -119,6 +119,17 @@ describe('v8flags', function () {
     delete require.cache[require.resolve('./')];
     var v8flags = require('./');
     console.log(v8flags.configfile);
+    v8flags(function (err, flags) {
+      expect(err).to.be.null;
+      done();
+    });
+  });
+
+  it('should handle undefined usernames', function(done) {
+    eraseHome();
+    delete require.cache[require.resolve('./')];
+    var v8flags = require('./');
+    console.log(v8flags.configfile);
     v8flags(function (err, flags) {
       expect(err).to.be.null;
       done();