improved info message for pool test, minor cleanup, see #167 and #208

js/configurePreferences.js

@@ -76,20 +76,8 @@ module.exports = async function() {
   preferences.productionServerName = preferences.productionServerName || 'phet-server.int.colorado.edu';
 
   // check that the DB server is running and that a basic query can be performed
-  winston.log( 'info', 'testing database connection (results will likely be several log entries after this one)' );
-  // !!! THIS WORKS
-  // const pool = new Pool();
-  // pool.query( 'SELECT NOW()', ( err, res ) => {
-  //   if ( err ) {
-  //     winston.log( 'error', 'database connection failed, short term string storage will not work, err = ' + err );
-  //   }
-  //   else {
-  //     winston.log( 'info', 'database test using SELECT NOW() succeeded, now = ' + res.rows[ 0 ].now );
-  //   }
-  // } );
-
+  winston.log( 'info', 'testing database connection (results may be several log entries after this one)' );
   const pool = new Pool();
-
   try {
     const { rows } = await pool.query( 'SELECT NOW()' );
     winston.log( 'info', 'database test using SELECT NOW() succeeded, now = ' + rows[ 0 ].now );