test: reenable test-worker.js for all workers

Also use t.shutdown in attempt to cleanup after failure, although I think
this doesn't work in all cases (the test run hangs if something goes wrong,
which is annoying).

packages/SwingSet/test/workers/test-worker.js

@@ -12,38 +12,31 @@ maybeTestXS('xs vat manager', async t => {
   const config = await loadBasedir(__dirname);
   config.vats.target.creationOptions = { managerType: 'xs-worker' };
   const c = await buildVatController(config, []);
+  t.teardown(c.shutdown);
 
   await c.run();
   t.is(c.kpStatus(c.bootstrapResult), 'fulfilled');
   t.deepEqual(c.dump().log, ['testLog works']);
-
-  await c.shutdown();
 });
 
-// XXX Test temporarily disabled on account of breakage due to some kind of
-// mysterious node worker mysteriousity.
-test.skip('nodeWorker vat manager', async t => {
+test('nodeWorker vat manager', async t => {
   const config = await loadBasedir(__dirname);
   config.vats.target.creationOptions = { managerType: 'nodeWorker' };
   const c = await buildVatController(config, []);
+  t.teardown(c.shutdown);
 
   await c.run();
   t.is(c.kpStatus(c.bootstrapResult), 'fulfilled');
   t.deepEqual(c.dump().log, ['testLog works']);
-
-  await c.shutdown();
 });
 
-/* // disabling for now due to possible buffering issue on MacOS
 test('node-subprocess vat manager', async t => {
   const config = await loadBasedir(__dirname);
   config.vats.target.creationOptions = { managerType: 'node-subprocess' };
   const c = await buildVatController(config, []);
+  t.teardown(c.shutdown);
 
   await c.run();
   t.is(c.kpStatus(c.bootstrapResult), 'fulfilled');
   t.deepEqual(c.dump().log, ['testLog works']);
-
-  await c.shutdown();
 });
-*/