Fix: Default flags to empty array (fixes #53)

index.js

@@ -83,7 +83,10 @@ function getFlags(cb) {
     if (execErr) {
       return cb(execErr);
     }
-    var flags = result.match(/\s\s--[\w-]+/gm)
+    // If the binary doesn't return flags in the way we expect, default to an empty array
+    // Reference https://github.com/gulpjs/v8flags/issues/53
+    var matchedFlags = result.match(/\s\s--[\w-]+/gm) || [];
+    var flags = matchedFlags
       .map(normalizeFlagName)
       .filter(function(name) {
         return exclusions.indexOf(name) === -1;

test/index.js

@@ -110,6 +110,28 @@ describe('v8flags', function() {
     });
   });
 
+  it('will not throw on non-matching return from node --v8-options', function(done) {
+    if (os.platform() === 'win32') {
+      this.skip();
+    }
+
+    eraseHome();
+    var v8flags = require('../');
+
+    // Save original execPath
+    var execPath = process.execPath;
+    // Set execPath to our fake-bin
+    process.execPath = __dirname + '/fake-bin';
+
+    v8flags(function(err, flags) {
+      expect(err).toNotExist();
+      expect(flags).toEqual([]);
+      // Restore original execPath
+      process.execPath = execPath;
+      done();
+    });
+  });
+
   it('should back with an empty array if the runtime is electron', function(done) {
     process.versions.electron = 'set';
     var v8flags = require('../');