Mark tested_up_to as optional

Author: tfirdaus

features/plugin.feature

@@ -746,11 +746,23 @@ Feature: Manage WordPress plugins
       License URI: https://www.gnu.org/licenses/gpl-2.0.html
       """
     And I run `wp plugin activate foo`
+
+    When I run `wp plugin list`
+    Then STDOUT should be a table containing rows:
+      | name       | status   | update  | version  | update_version | auto_update |
+      | foo        | active   | none    |          |                | off         |
+
     When I run `wp plugin list --fields=name,tested_up_to`
     Then STDOUT should be a table containing rows:
       | name            | tested_up_to     |
       | foo             | 3.4              |
 
+    And I run `wp plugin list --name=foo --field=tested_up_to`
+    Then STDOUT should be:
+      """
+      3.4
+      """
+
   Scenario: Listing plugins should include tested_up_to from the 'tested' header
     Given a WP install
     And a wp-content/plugins/foo/foo.php file:
@@ -776,7 +788,19 @@ Feature: Manage WordPress plugins
       License URI: https://www.gnu.org/licenses/gpl-2.0.html
       """
     And I run `wp plugin activate foo`
+
+    When I run `wp plugin list`
+    Then STDOUT should be a table containing rows:
+      | name       | status   | update  | version  | update_version | auto_update |
+      | foo        | active   | none    |          |                | off         |
+
     When I run `wp plugin list --fields=name,tested_up_to`
     Then STDOUT should be a table containing rows:
       | name            | tested_up_to     |
       | foo             | 5.5              |
+
+    And I run `wp plugin list --name=foo --field=tested_up_to`
+    Then STDOUT should be:
+      """
+      5.5
+      """

src/Plugin_Command.php

@@ -52,6 +52,9 @@ class Plugin_Command extends \WP_CLI\CommandWithUpgrade {
 		'status'       => false,
 		'last_updated' => false,
 	];
+	protected $check_headers     = [
+		'tested_up_to' => false,
+	];
 
 	protected $obj_fields = array(
 		'name',
@@ -743,12 +746,14 @@ protected function get_item_list() {
 				'wporg_last_updated' => $wporg_info['last_updated'],
 			];
 
-			// Include information from the plugin readme.txt headers.
-			$plugin_readme = WP_PLUGIN_DIR . '/' . $name . '/readme.txt';
+			if ( $this->check_headers['tested_up_to'] ) {
+				// Include information from the plugin readme.txt headers.
+				$plugin_readme = WP_PLUGIN_DIR . '/' . $name . '/readme.txt';
 
-			if ( file_exists( $plugin_readme ) ) {
-				$readme_parser                  = new Parser( $plugin_readme );
-				$items[ $file ]['tested_up_to'] = $readme_parser->tested ? $readme_parser->tested : '';
+				if ( file_exists( $plugin_readme ) ) {
+					$readme_parser                  = new Parser( $plugin_readme );
+					$items[ $file ]['tested_up_to'] = $readme_parser->tested ? $readme_parser->tested : '';
+				}
 			}
 
 			if ( null === $update_info ) {
@@ -1263,7 +1268,6 @@ public function delete( $args, $assoc_args = array() ) {
 	 * * version
 	 * * update_version
 	 * * auto_update
-	 * * tested_up_to
 	 *
 	 * These fields are optionally available:
 	 *
@@ -1273,6 +1277,7 @@ public function delete( $args, $assoc_args = array() ) {
 	 * * description
 	 * * file
 	 * * author
+	 * * tested_up_to
 	 * * wporg_status
 	 * * wporg_last_updated
 	 *
@@ -1316,6 +1321,8 @@ public function list_( $_, $assoc_args ) {
 			$fields                            = explode( ',', $fields );
 			$this->check_wporg['status']       = in_array( 'wporg_status', $fields, true );
 			$this->check_wporg['last_updated'] = in_array( 'wporg_last_updated', $fields, true );
+
+			$this->check_headers['tested_up_to'] = in_array( 'tested_up_to', $fields, true );
 		}
 
 		$field = Utils\get_flag_value( $assoc_args, 'field' );
@@ -1325,6 +1332,8 @@ public function list_( $_, $assoc_args ) {
 			$this->check_wporg['last_updated'] = true;
 		}
 
+		$this->check_headers['tested_up_to'] = 'tested_up_to' === $field || $this->check_headers['tested_up_to'];
+
 		parent::_list( $_, $assoc_args );
 	}