Made a distinction between errors and warnings when checking the DB command.

Author: janw-me

src/Context/FeatureContext.php

@@ -1134,12 +1134,19 @@ public function test_connection(): void {
 				'send_to_shell' => false,
 			]
 		);
-		if ( ! empty( $sql_result['stderr'] ) ) {
+
+		if ( $sql_result['exit_code'] !== 0 ) {
 			# WP_CLI output functions are suppressed in behat context.
 			echo 'There was an error connecting to the database:' . \PHP_EOL;
-			echo '    ' . trim( $sql_result['stderr'] ) . \PHP_EOL;
+			if ( ! empty( $sql_result['stderr'] ) ) {
+				echo '  ' . trim( $sql_result['stderr'] ) . \PHP_EOL;
+			}
 			echo 'run `composer prepare-tests` to connect to the database.' . \PHP_EOL;
-			die( 1 );
+			die( $sql_result['exit_code'] );
+		} elseif ( ! empty( $sql_result['stderr'] ) ) {
+			// There is "error" output but not an exit code.
+			// Probably a warning, still display it.
+			echo trim( $sql_result['stderr'] ) . \PHP_EOL;
 		}
 	}