proper clean up of fabricator children. fixes #254

lib/fabricator.js

@@ -128,6 +128,7 @@ export function fabricateTwice (bakes, fabricator, snap, body, cb) {
 export function shutdown () {
   for (const key in children) {
     const child = children[key];
+    delete children[key];
     child.kill();
   }
 }

test/test-50-api/main.js

@@ -0,0 +1,38 @@
+#!/usr/bin/env node
+
+'use strict';
+
+const path = require('path');
+const assert = require('assert');
+const utils = require('../utils.js');
+
+assert(!module.parent);
+assert(__dirname === process.cwd());
+
+const target = process.argv[2] || 'host';
+const input = './test-x-index.js';
+const output = './test-output.exe';
+
+let right;
+utils.mkdirp.sync(path.dirname(output));
+
+// calling twice
+require('../../').exec([
+  '--target', target,
+  '--output', output, input
+]).then(function () {
+  return require('../../').exec([
+    '--target', target,
+    '--output', output, input
+  ]).then(function () {
+    right = utils.spawn.sync(
+      output, [], {}
+    );
+
+    assert.equal(right, '42\n');
+    utils.vacuum.sync(output);
+  });
+}).catch(function (error) {
+  console.error(error);
+  process.exit(2);
+});

test/test-50-api/test-x-index.js

@@ -0,0 +1,3 @@
+'use strict';
+
+console.log(42);