Fix wp db status to accept parameters and run before WordPress loads

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>

Author: Copilot

features/db-status.feature

@@ -54,3 +54,69 @@ Feature: Display database status overview
       """
       Check Status:      OK
       """
+
+  Scenario: Run db status with MySQL defaults to check the database
+    Given a WP install
+
+    When I run `wp db status --defaults`
+    Then STDOUT should contain:
+      """
+      Database Name:
+      """
+    And STDOUT should contain:
+      """
+      Check Status:      OK
+      """
+
+  Scenario: Run db status with --no-defaults to check the database
+    Given a WP install
+
+    When I run `wp db status --no-defaults`
+    Then STDOUT should contain:
+      """
+      Database Name:
+      """
+    And STDOUT should contain:
+      """
+      Check Status:      OK
+      """
+
+  Scenario: Run db status with passed-in options
+    Given a WP install
+
+    When I run `wp db status --dbuser=wp_cli_test`
+    Then STDOUT should contain:
+      """
+      Database Name:
+      """
+    And STDOUT should contain:
+      """
+      Check Status:      OK
+      """
+
+    When I run `wp db status --dbpass=password1`
+    Then STDOUT should contain:
+      """
+      Database Name:
+      """
+    And STDOUT should contain:
+      """
+      Check Status:      OK
+      """
+
+    When I run `wp db status --dbuser=wp_cli_test --dbpass=password1`
+    Then STDOUT should contain:
+      """
+      Database Name:
+      """
+    And STDOUT should contain:
+      """
+      Check Status:      OK
+      """
+
+    When I try `wp db status --dbuser=no_such_user`
+    Then the return code should not be 0
+
+    When I try `wp db status --dbpass=no_such_pass`
+    Then the return code should not be 0
+

src/DB_Command.php

@@ -1255,6 +1255,17 @@ public function prefix() {
 	 * command is useful for getting a quick snapshot of database health
 	 * without needing to run multiple separate commands.
 	 *
+	 * ## OPTIONS
+	 *
+	 * [--dbuser=<value>]
+	 * : Username to pass to mysql. Defaults to DB_USER.
+	 *
+	 * [--dbpass=<value>]
+	 * : Password to pass to mysql. Defaults to DB_PASSWORD.
+	 *
+	 * [--defaults]
+	 * : Loads the environment's MySQL option files. Default behavior is to skip loading them to avoid failures due to misconfiguration.
+	 *
 	 * ## EXAMPLES
 	 *
 	 *     $ wp db status
@@ -1267,9 +1278,9 @@ public function prefix() {
 	 *     Collation:         utf8mb4_unicode_ci
 	 *     Check Status:      OK
 	 *
-	 * @when after_wp_load
+	 * @when before_wp_load
 	 */
-	public function status() {
+	public function status( $_, $assoc_args ) {
 		global $wpdb;
 
 		// Get database name.
@@ -1355,16 +1366,15 @@ public function status() {
 		}
 		// Run database check silently to get status.
 		if ( $table_count > 0 ) {
-			$check_args                          = [];
 			$command                             = sprintf(
 				'/usr/bin/env %s%s %s',
 				Utils\get_sql_check_command(),
-				$this->get_defaults_flag_string( $check_args ),
+				$this->get_defaults_flag_string( $assoc_args ),
 				'%s'
 			);
 			list( $stdout, $stderr, $exit_code ) = self::run(
 				Utils\esc_cmd( $command, DB_NAME ),
-				[ 'check' => true ],
+				array_merge( [ 'check' => true ], $assoc_args ),
 				false
 			);
 			$check_status                        = ( 0 === $exit_code ) ? 'OK' : 'Error';