Clean up, add TODOs & decision docs notes

lib/docassemble/server_install.js

@@ -52,5 +52,5 @@ session_vars.validateEnvironment();
 const argv = require(`minimist`)(process.argv.slice(2));
 const log = new Log({ path: argv.path, context: `server_install` });
 
-// 3. run in a wrapper
-log.with_cleanup({ todo: server_install });
+// 3. run
+server_install();

lib/docassemble/setup.js

@@ -71,5 +71,5 @@ session_vars.validateEnvironment();
 const argv = require(`minimist`)(process.argv.slice(2));
 const log = new Log({ path: argv.path, context: `setup` });
 
-// 3. run in a wrapper
-log.with_cleanup({ todo: setup });
+// 3. run
+setup();

lib/docassemble/takedown.js

@@ -42,5 +42,5 @@ session_vars.validateEnvironment();
 const argv = require(`minimist`)( process.argv.slice(2) );
 const log = new Log({ path: argv.path, context: `takedown` });
 
-// 3. run in a wrapper
-log.with_cleanup({ todo: takedown });
+// 3. run
+takedown();

lib/run_cucumber.js

@@ -197,5 +197,5 @@ const argv = require(`minimist`)( process.argv.slice(2) );
 // their own file.
 const log = new Log({ path: argv.path, context: `run` });
 
-// 3. run in a wrapper
-log.with_cleanup({ todo: main });
+// 3. run
+main();

lib/steps.js

@@ -1441,10 +1441,12 @@ After(async function(scenario) {
   }
 
   // Not sure if this is necessary
-  if ( changeable_test_status !== 'PASSED' ) {
-    log.stdout({ records_only: true }, changeable_test_status[0].toUpperCase() );
-  } else {
+  if ( changeable_test_status === 'PASSED' ) {
     log.stdout({ records_only: true }, `.` );
+  } else if ( changeable_test_status === `SKIPPED` ) {
+    log.stdout({ records_only: true }, `-` );
+  } else {
+    log.stdout({ records_only: true }, changeable_test_status[0].toUpperCase() );
   }
 
   let signout_succeeded = true;