wp-env: Better errors when Docker is not started (#30882)

.eslintrc.js

@@ -204,7 +204,7 @@ module.exports = {
 			},
 		},
 		{
-			files: [ 'bin/**/*.js' ],
+			files: [ 'bin/**/*.js', 'packages/env/**' ],
 			rules: {
 				'no-console': 'off',
 			},

packages/env/lib/cli.js

@@ -6,6 +6,7 @@ const chalk = require( 'chalk' );
 const ora = require( 'ora' );
 const yargs = require( 'yargs' );
 const terminalLink = require( 'terminal-link' );
+const { execSync } = require( 'child_process' );
 
 /**
  * Internal dependencies
@@ -63,7 +64,6 @@ const withSpinner = ( command ) => ( ...args ) => {
 					typeof error === 'string' ? error : error.message
 				);
 				// Disable reason: Using console.error() means we get a stack trace.
-				// eslint-disable-next-line no-console
 				console.error( error );
 				process.exit( 1 );
 			} else {
@@ -75,6 +75,16 @@ const withSpinner = ( command ) => ( ...args ) => {
 };
 
 module.exports = function cli() {
+	// Do nothing if Docker is unavailable.
+	try {
+		execSync( 'docker info', { stdio: 'ignore' } );
+	} catch {
+		console.error(
+			chalk.red( 'Could not connect to Docker. Is it running?' )
+		);
+		process.exit( 1 );
+	}
+
 	yargs.usage( wpPrimary( '$0 <command>' ) );
 	yargs.option( 'debug', {
 		type: 'boolean',

packages/env/lib/commands/logs.js

@@ -60,12 +60,10 @@ module.exports = async function logs( { environment, watch, spinner, debug } ) {
 	);
 
 	if ( result.out.length ) {
-		// eslint-disable-next-line no-console
 		console.log(
 			process.stdout.isTTY ? `\n\n${ result.out }\n\n` : result.out
 		);
 	} else if ( result.err.length ) {
-		// eslint-disable-next-line no-console
 		console.error(
 			process.stdout.isTTY ? `\n\n${ result.err }\n\n` : result.err
 		);

packages/env/test/cli.js

@@ -82,7 +82,6 @@ describe( 'env cli', () => {
 	} );
 
 	it( 'handles failed commands with messages.', async () => {
-		/* eslint-disable no-console */
 		env.start.mockRejectedValueOnce( {
 			message: 'failure message',
 		} );
@@ -100,10 +99,8 @@ describe( 'env cli', () => {
 		expect( process.exit ).toHaveBeenCalledWith( 1 );
 		console.error = consoleError;
 		process.exit = processExit;
-		/* eslint-enable no-console */
 	} );
 	it( 'handles failed docker commands with errors.', async () => {
-		/* eslint-disable no-console */
 		env.start.mockRejectedValueOnce( {
 			err: 'failure error',
 			out: 'message',
@@ -128,6 +125,5 @@ describe( 'env cli', () => {
 		console.error = consoleError;
 		process.exit = processExit;
 		process.stderr.write = stderr;
-		/* eslint-enable no-console */
 	} );
 } );