Issues in class-jetpack-cxn-test-base.php - New Issues - Inspection of "Moving gutenblock code to blocks directory for aut..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — update/ad-gutenblock-load ( 0c433b...289acb )

unknown

created 2019-02-22 20:07 UTC

_inc/lib/debugger/class-jetpack-cxn-test-base.php (4 issues)

Labels

Severity

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
/**
 * Jetpack Connection Testing
 *
 * Framework for various "unit tests" against the Jetpack connection.
 *
 * Individual tests should be added to the class-jetpack-cxn-tests.php file.
 *
 * @author Brandon Kraft
 * @package Jetpack
 */

/**
 * "Unit Tests" for the Jetpack connection.
 */
class Jetpack_Cxn_Test_Base {

	/**
	 * Tests to run on the Jetpack connection.
	 *
	 * @var array $tests
	 */
	protected $tests = array();

	/**
	 * Results of the Jetpack connection tests.
	 *
	 * @var array $results
	 */
	protected $results = array();

	/**
	 * Status of the testing suite.
	 *
	 * Used internally to determine if a test should be skipped since the tests are already failing. Assume passing.
	 *
	 * @var bool $pass
	 */
	protected $pass = true;

	/**
	 * Jetpack_Cxn_Test constructor.
	 */
	public function __construct() {
		$this->tests   = array();
		$this->results = array();
	}

	/**
	 * Adds a new test to the Jetpack Connection Testing suite.
	 *
	 * @param callable $callable Test to add to queue.
	 * @param array    $groups Testing groups to add test to.
	 *
	 * @return bool True if successfully added. False for a failure.
	 */
	public function add_test( $callable, $groups = array( 'default' ) ) {
		if ( is_callable( $callable ) ) {
			$this->tests[] = array(
				'test'  => $callable,
				'group' => $groups,
			);
			return true;
		}

		return false;
	}

	/**
	 * Runs the Jetpack connection suite.
	 */
	public function run_tests() {
		foreach ( $this->tests as $test ) {
			$result          = call_user_func( $test['test'] );
			$result['group'] = $test['group'];
			$this->results[] = $result;
			if ( false === $result['pass'] ) {
				$this->pass = false;
			}
		}
	}

	/**
	 * Returns the full results array.
	 *
	 * @param string $group Testing group whose results we want. Defaults to "default" group. Use "all" for all tests.
	 * @return array Array of test results.
	 */
	public function raw_results( $group = 'default' ) {
		if ( ! $this->results ) {

			$this->run_tests();
		}

		$results = $this->results;

		if ( 'all' === $group ) {
			return $results;
		}

		foreach ( $results as $test => $result ) {
			if ( ! in_array( $group, $result['group'], true ) ) {
				unset( $results[ $test ] );
			}
		}

		return $results;
	}

	/**
	 * Returns the status of the connection suite.
	 *
	 * @param string $group Testing group to check status of. Optional, default all tests.
	 *
	 * @return true|array True if all tests pass. Array of failed tests.
	 */
	public function pass( $group = 'default' ) {
		$results = $this->raw_results( $group );

		foreach ( $results as $result ) {
			// 'pass' could be true, false, or 'skipped'. We only want false.
			if ( isset( $result['pass'] ) && false === $result['pass'] ) {
				return false;
			}
		}

		return true;

	}

	/**
	 * Return array of failed test messages.
	 *
	 * @param string $group Testing group whose failures we want. Defaults to "default". Use "all" for all tests.
	 *
	 * @return false|array False if no failed tests. Otherwise, array of failed tests.
	 */
	public function list_fails( $group = 'default' ) {
		$results = $this->raw_results( $group );

		foreach ( $results as $test => $result ) {
			// We do not want tests that passed or ones that are misconfigured (no pass status or no failure message).
			if ( ! isset( $result['pass'] ) || false !== $result['pass'] || ! isset( $result['message'] ) ) {
				unset( $results[ $test ] );
			}
		}

		return $results;
	}

	/**
	 * Helper function to return consistent responses for a passing test.
	 *
	 * @param string $name Test name.
	 *
	 * @return array Test results.
	 */
	public static function passing_test( $name = 'Unnamed' ) {
		return array(
			'name'       => $name,
			'pass'       => true,
			'message'    => __( 'Test Passed!', 'jetpack' ),
			'resolution' => false,
		);
	}

	/**
	 * Helper function to return consistent responses for a skipped test.
	 *
	 * @param string $name Test name.
	 * @param string $message Reason for skipping the test. Optional.

	 *
	 * @return array Test results.
	 */
	public static function skipped_test( $name = 'Unnamed', $message = false ) {
		return array(
			'name'       => $name,
			'pass'       => 'skipped',
			'message'    => $message,
			'resolution' => false,
		);
	}

	/**
	 * Helper function to return consistent responses for a failing test.
	 *
	 * @param string $name Test name.
	 * @param string $message Message detailing the failure.
	 * @param string $resolution Steps to resolve.

	 *
	 * @return array Test results.
	 */
	public static function failing_test( $name, $message, $resolution = false ) {
		// Provide standard resolutions steps, but allow pass-through of non-standard ones.
		switch ( $resolution ) {
			case 'cycle_connection':
				$resolution = __( 'Please disconnect and reconnect Jetpack.', 'jetpack' ); // @todo: Link.
				break;
			case 'outbound_requests':
				$resolution = __( 'Please ask your hosting provider to confirm your server can make outbound requests to jetpack.com.', 'jetpack' );
				break;
			case 'support':
				$resolution = __( 'Please contact support.', 'jetpack' ); // @todo: Link to support.
				break;
		}

		return array(
			'name'       => $name,
			'pass'       => false,
			'message'    => $message,
			'resolution' => $resolution,
		);
	}

	/**
	 * Provide WP_CLI friendly testing results.
	 *
	 * @param string $group Testing group whose results we are outputting. Default "default". Use "all" for all tests.
	 */
	public function output_results_for_cli( $group = 'default' ) {
		if ( defined( 'WP_CLI' ) && WP_CLI ) {
			if ( Jetpack::is_development_mode() ) {
				WP_CLI::line( __( 'Jetpack is in Development Mode:', 'jetpack' ) );
				WP_CLI::line( Jetpack::development_mode_trigger_text() );
			}
			WP_CLI::line( __( 'TEST RESULTS:', 'jetpack' ) );
			foreach ( $this->raw_results( $group ) as $test ) {
				if ( true === $test['pass'] ) {
					WP_CLI::log( WP_CLI::colorize( '%gPassed:%n  ' . $test['name'] ) );
				} elseif ( 'skipped' === $test['pass'] ) {
					WP_CLI::log( WP_CLI::colorize( '%ySkipped:%n ' . $test['name'] ) );
					if ( $test['message'] ) {
						WP_CLI::log( '         ' . $test['message'] ); // Number of spaces to "tab indent" the reason.
					}
				} else { // Failed.
					WP_CLI::log( WP_CLI::colorize( '%rFailed:%n  ' . $test['name'] ) );
					WP_CLI::log( '         ' . $test['message'] ); // Number of spaces to "tab indent" the reason.
				}
			}
		}
	}

	/**
	 * Provide single WP Error instance of all failures.
	 *
	 * @param string $group Testing group whose failures we want converted. Default "default". Use "all" for all tests.
	 *
	 * @return WP_Error|false WP_Error with all failed tests or false if there were no failures.
	 */
	public function output_fails_as_wp_error( $group = 'default' ) {
		if ( $this->pass( $group ) ) {
			return false;
		}
		$fails = $this->list_fails( $group );
		$error = false;

		foreach ( $fails as $result ) {
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
			$code    = 'failed_' . $result['name'];
			$message = $result['message'];
			$data    = array(
				'resolution' => $result['resolution'],
			);
			if ( ! $error ) {
				$error = new WP_Error( $code, $message, $data );
			} else {
				$error->add( $code, $message, $data );
			}
		}

		return $error;
	}

	/**
	 * Encrypt data for sending to WordPress.com.
	 *
	 * @todo When PHP minimum is 5.3+, add cipher detection to use an agreed better cipher than RC4. RC4 should be the last resort.
	 *
	 * @param string $data Data to encrypt with the WP.com Public Key.
	 *
	 * @return false|array False if functionality not available. Array of encrypted data, encryption key.
	 */
	public function encrypt_string_for_wpcom( $data ) {
		$return = false;
		if ( ! function_exists( 'openssl_get_publickey' ) || ! function_exists( 'openssl_seal' ) ) {
			return $return;
		}

		$public_key = openssl_get_publickey( JETPACK__DEBUGGER_PUBLIC_KEY );

		if ( $public_key && openssl_seal( $data, $encrypted_data, $env_key, array( $public_key ) ) ) {
			// We are returning base64-encoded values to ensure they're characters we can use in JSON responses without issue.
			$return = array(
				'data'   => base64_encode( $encrypted_data ), // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.obfuscation_base64_encode
				'key'    => base64_encode( $env_key[0] ), // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.obfuscation_base64_encode
				'cipher' => 'RC4', // When Jetpack's minimum WP version is at PHP 5.3+, we will add in detecting and using a stronger one.
			);
		}

		openssl_free_key( $public_key );

		return $return;
	}
}


1			<?php
2			/**
3			* Jetpack Connection Testing
4			*
5			* Framework for various "unit tests" against the Jetpack connection.
6			*
7			* Individual tests should be added to the class-jetpack-cxn-tests.php file.
8			*
9			* @author Brandon Kraft
10			* @package Jetpack
11			*/
12
13			/**
14			* "Unit Tests" for the Jetpack connection.
15			*/
16			class Jetpack_Cxn_Test_Base {
17
18			/**
19			* Tests to run on the Jetpack connection.
20			*
21			* @var array $tests
22			*/
23			protected $tests = array();
24
25			/**
26			* Results of the Jetpack connection tests.
27			*
28			* @var array $results
29			*/
30			protected $results = array();
31
32			/**
33			* Status of the testing suite.
34			*
35			* Used internally to determine if a test should be skipped since the tests are already failing. Assume passing.
36			*
37			* @var bool $pass
38			*/
39			protected $pass = true;
40
41			/**
42			* Jetpack_Cxn_Test constructor.
43			*/
44			public function __construct() {
45			$this->tests = array();
46			$this->results = array();
47			}
48
49			/**
50			* Adds a new test to the Jetpack Connection Testing suite.
51			*
52			* @param callable $callable Test to add to queue.
53			* @param array $groups Testing groups to add test to.
54			*
55			* @return bool True if successfully added. False for a failure.
56			*/
57			public function add_test( $callable, $groups = array( 'default' ) ) {
58			if ( is_callable( $callable ) ) {
59			$this->tests[] = array(
60			'test' => $callable,
61			'group' => $groups,
62			);
63			return true;
64			}
65
66			return false;
67			}
68
69			/**
70			* Runs the Jetpack connection suite.
71			*/
72			public function run_tests() {
73			foreach ( $this->tests as $test ) {
74			$result = call_user_func( $test['test'] );
75			$result['group'] = $test['group'];
76			$this->results[] = $result;
77			if ( false === $result['pass'] ) {
78			$this->pass = false;
79			}
80			}
81			}
82
83			/**
84			* Returns the full results array.
85			*
86			* @param string $group Testing group whose results we want. Defaults to "default" group. Use "all" for all tests.
87			* @return array Array of test results.
88			*/
89			public function raw_results( $group = 'default' ) {
90			if ( ! $this->results ) {
			0 ignored issues – show Bug Best Practice introduced 2018-11-07 02:07 UTC by Report Bug Copy Issue Report Show Similar Issues like this The expression `$this->results` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
91			$this->run_tests();
92			}
93
94			$results = $this->results;
95
96			if ( 'all' === $group ) {
97			return $results;
98			}
99
100			foreach ( $results as $test => $result ) {
101			if ( ! in_array( $group, $result['group'], true ) ) {
102			unset( $results[ $test ] );
103			}
104			}
105
106			return $results;
107			}
108
109			/**
110			* Returns the status of the connection suite.
111			*
112			* @param string $group Testing group to check status of. Optional, default all tests.
113			*
114			* @return true\|array True if all tests pass. Array of failed tests.
115			*/
116			public function pass( $group = 'default' ) {
117			$results = $this->raw_results( $group );
118
119			foreach ( $results as $result ) {
120			// 'pass' could be true, false, or 'skipped'. We only want false.
121			if ( isset( $result['pass'] ) && false === $result['pass'] ) {
122			return false;
123			}
124			}
125
126			return true;
127
128			}
129
130			/**
131			* Return array of failed test messages.
132			*
133			* @param string $group Testing group whose failures we want. Defaults to "default". Use "all" for all tests.
134			*
135			* @return false\|array False if no failed tests. Otherwise, array of failed tests.
136			*/
137			public function list_fails( $group = 'default' ) {
138			$results = $this->raw_results( $group );
139
140			foreach ( $results as $test => $result ) {
141			// We do not want tests that passed or ones that are misconfigured (no pass status or no failure message).
142			if ( ! isset( $result['pass'] ) \|\| false !== $result['pass'] \|\| ! isset( $result['message'] ) ) {
143			unset( $results[ $test ] );
144			}
145			}
146
147			return $results;
148			}
149
150			/**
151			* Helper function to return consistent responses for a passing test.
152			*
153			* @param string $name Test name.
154			*
155			* @return array Test results.
156			*/
157			public static function passing_test( $name = 'Unnamed' ) {
158			return array(
159			'name' => $name,
160			'pass' => true,
161			'message' => __( 'Test Passed!', 'jetpack' ),
162			'resolution' => false,
163			);
164			}
165
166			/**
167			* Helper function to return consistent responses for a skipped test.
168			*
169			* @param string $name Test name.
170			* @param string $message Reason for skipping the test. Optional.
			0 ignored issues – show Documentation introduced 2018-11-09 18:29 UTC by Report Bug Copy Issue Report Show Similar Issues like this Should the type for parameter `$message` not be `false\|string`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
171			*
172			* @return array Test results.
173			*/
174			public static function skipped_test( $name = 'Unnamed', $message = false ) {
175			return array(
176			'name' => $name,
177			'pass' => 'skipped',
178			'message' => $message,
179			'resolution' => false,
180			);
181			}
182
183			/**
184			* Helper function to return consistent responses for a failing test.
185			*
186			* @param string $name Test name.
187			* @param string $message Message detailing the failure.
188			* @param string $resolution Steps to resolve.
			0 ignored issues – show Documentation introduced 2018-11-08 00:25 UTC by Report Bug Copy Issue Report Show Similar Issues like this Should the type for parameter `$resolution` not be `false\|string`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
189			*
190			* @return array Test results.
191			*/
192			public static function failing_test( $name, $message, $resolution = false ) {
193			// Provide standard resolutions steps, but allow pass-through of non-standard ones.
194			switch ( $resolution ) {
195			case 'cycle_connection':
196			$resolution = __( 'Please disconnect and reconnect Jetpack.', 'jetpack' ); // @todo: Link.
197			break;
198			case 'outbound_requests':
199			$resolution = __( 'Please ask your hosting provider to confirm your server can make outbound requests to jetpack.com.', 'jetpack' );
200			break;
201			case 'support':
202			$resolution = __( 'Please contact support.', 'jetpack' ); // @todo: Link to support.
203			break;
204			}
205
206			return array(
207			'name' => $name,
208			'pass' => false,
209			'message' => $message,
210			'resolution' => $resolution,
211			);
212			}
213
214			/**
215			* Provide WP_CLI friendly testing results.
216			*
217			* @param string $group Testing group whose results we are outputting. Default "default". Use "all" for all tests.
218			*/
219			public function output_results_for_cli( $group = 'default' ) {
220			if ( defined( 'WP_CLI' ) && WP_CLI ) {
221			if ( Jetpack::is_development_mode() ) {
222			WP_CLI::line( __( 'Jetpack is in Development Mode:', 'jetpack' ) );
223			WP_CLI::line( Jetpack::development_mode_trigger_text() );
224			}
225			WP_CLI::line( __( 'TEST RESULTS:', 'jetpack' ) );
226			foreach ( $this->raw_results( $group ) as $test ) {
227			if ( true === $test['pass'] ) {
228			WP_CLI::log( WP_CLI::colorize( '%gPassed:%n ' . $test['name'] ) );
229			} elseif ( 'skipped' === $test['pass'] ) {
230			WP_CLI::log( WP_CLI::colorize( '%ySkipped:%n ' . $test['name'] ) );
231			if ( $test['message'] ) {
232			WP_CLI::log( ' ' . $test['message'] ); // Number of spaces to "tab indent" the reason.
233			}
234			} else { // Failed.
235			WP_CLI::log( WP_CLI::colorize( '%rFailed:%n ' . $test['name'] ) );
236			WP_CLI::log( ' ' . $test['message'] ); // Number of spaces to "tab indent" the reason.
237			}
238			}
239			}
240			}
241
242			/**
243			* Provide single WP Error instance of all failures.
244			*
245			* @param string $group Testing group whose failures we want converted. Default "default". Use "all" for all tests.
246			*
247			* @return WP_Error\|false WP_Error with all failed tests or false if there were no failures.
248			*/
249			public function output_fails_as_wp_error( $group = 'default' ) {
250			if ( $this->pass( $group ) ) {
251			return false;
252			}
253			$fails = $this->list_fails( $group );
254			$error = false;
255
256			foreach ( $fails as $result ) {
			0 ignored issues – show Bug introduced 2018-11-08 19:47 UTC by Report Bug Copy Issue Report Show Similar Issues like this The expression `$fails` of type `false\|array` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
257			$code = 'failed_' . $result['name'];
258			$message = $result['message'];
259			$data = array(
260			'resolution' => $result['resolution'],
261			);
262			if ( ! $error ) {
263			$error = new WP_Error( $code, $message, $data );
264			} else {
265			$error->add( $code, $message, $data );
266			}
267			}
268
269			return $error;
270			}
271
272			/**
273			* Encrypt data for sending to WordPress.com.
274			*
275			* @todo When PHP minimum is 5.3+, add cipher detection to use an agreed better cipher than RC4. RC4 should be the last resort.
276			*
277			* @param string $data Data to encrypt with the WP.com Public Key.
278			*
279			* @return false\|array False if functionality not available. Array of encrypted data, encryption key.
280			*/
281			public function encrypt_string_for_wpcom( $data ) {
282			$return = false;
283			if ( ! function_exists( 'openssl_get_publickey' ) \|\| ! function_exists( 'openssl_seal' ) ) {
284			return $return;
285			}
286
287			$public_key = openssl_get_publickey( JETPACK__DEBUGGER_PUBLIC_KEY );
288
289			if ( $public_key && openssl_seal( $data, $encrypted_data, $env_key, array( $public_key ) ) ) {
290			// We are returning base64-encoded values to ensure they're characters we can use in JSON responses without issue.
291			$return = array(
292			'data' => base64_encode( $encrypted_data ), // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.obfuscation_base64_encode
293			'key' => base64_encode( $env_key[0] ), // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.obfuscation_base64_encode
294			'cipher' => 'RC4', // When Jetpack's minimum WP version is at PHP 5.3+, we will add in detecting and using a stronger one.
295			);
296			}
297
298			openssl_free_key( $public_key );
299
300			return $return;
301			}
302			}
303

Automattic / jetpack

Push — update/ad-gutenblock-load ( 0c433b...289acb )

_inc/lib/debugger/class-jetpack-cxn-test-base.php (4 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like