SignatureTest::join_with_equal_sign_data_provider() - Code Metrics - Inspection of "Add nested array support for join_with_equal_sign" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — add/nested-query-string-suppor... ( 782fd1 )

unknown

created 2020-09-01 22:00 UTC

join_with_equal_sign_data_provider() A

↳ Parent: SignatureTest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	54
rs	9.0036
c	0
b	0
f	0

How to fix Long Method

<?php // phpcs:ignore WordPress.Files.FileName.InvalidClassFileName
/**
 * The SignatureTest class file.
 *
 * @package automattic/jetpack-connection
 */

namespace Automattic\Jetpack\Connection;

use PHPUnit\Framework\TestCase;

/**
 * Provides unit tests for the methods in the Jetpack_Signature class.
 */
class SignatureTest extends TestCase {
	/**
	 * Tests the Jetpack_Signature->join_with_equal_sign() method.
	 *
	 * @covers Automattic\Jetpack\Connection\Jetpack_Signature->join_with_equal_sign
	 * @dataProvider join_with_equal_sign_data_provider
	 *
	 * @param string       $name Query string key value.
	 * @param string|array $value Associated value for query string key.
	 * @param string|array $expected_output The expected output of $signature->join_with_equal_sign.
	 */
	public function test_join_with_equal_sign( $name, $value, $expected_output ) {
		$signature = new \Jetpack_Signature( 'some-secret', 0 );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		$this->assertEquals( $expected_output, $signature->join_with_equal_sign( $name, $value ) );
	}

	/**
	 * Data provider for test_join_with_equal_sign.
	 *
	 * The test data arrays have the format:
	 *    'name'            => The value that the constant will be set to. Null if the constant will not be set.
	 *    'value'           => The name of the constant.
	 *    'expected_output' => The expected output of Utils::jetpack_api_constant_filter().
	 */
	public function join_with_equal_sign_data_provider() {
		return array(
			'string_value'                   =>
				array(
					'name'            => 'street',
					'value'           => '1600 Pennsylvania Ave',
					'expected_output' => 'street=1600 Pennsylvania Ave',
				),
			'array_value'                    =>
				array(
					'name'            => 'first_names',
					'value'           => array( 'Michael', 'Jim', 'Pam' ),
					'expected_output' => array( 'first_names[0]=Michael', 'first_names[1]=Jim', 'first_names[2]=Pam' ),
				),
			'nested_array_value'             =>
				array(
					'name'            => 'numbers',
					'value'           => array( array( 0, 1 ), array( 2, 3 ), array( 4, 5 ) ),
					'expected_output' => array(
						'numbers[0][0]=0',
						'numbers[0][1]=1',
						'numbers[1][0]=2',
						'numbers[1][1]=3',
						'numbers[2][0]=4',
						'numbers[2][1]=5',
					),
				),
			'nested_associative_array_value' =>
				array(
					'name'            => 'people',
					'value'           => array(
						array(
							'last_name'  => 'Scott',
							'first_name' => 'Michael',
							'city'       => 'Boulder',
						),
						array(
							'first_name' => 'Jim',
							'state'      => 'Texas',
							'last_name'  => 'Halpert',
						),
					),
					// Note: Expected output is sorted.
					'expected_output' => array(
						'people[0][city]=Boulder',
						'people[0][first_name]=Michael',
						'people[0][last_name]=Scott',
						'people[1][first_name]=Jim',
						'people[1][last_name]=Halpert',
						'people[1][state]=Texas',
					),
				),
		);
	}
}


1			<?php // phpcs:ignore WordPress.Files.FileName.InvalidClassFileName
2			/**
3			* The SignatureTest class file.
4			*
5			* @package automattic/jetpack-connection
6			*/
7
8			namespace Automattic\Jetpack\Connection;
9
10			use PHPUnit\Framework\TestCase;
11
12			/**
13			* Provides unit tests for the methods in the Jetpack_Signature class.
14			*/
15			class SignatureTest extends TestCase {
16			/**
17			* Tests the Jetpack_Signature->join_with_equal_sign() method.
18			*
19			* @covers Automattic\Jetpack\Connection\Jetpack_Signature->join_with_equal_sign
20			* @dataProvider join_with_equal_sign_data_provider
21			*
22			* @param string $name Query string key value.
23			* @param string\|array $value Associated value for query string key.
24			* @param string\|array $expected_output The expected output of $signature->join_with_equal_sign.
25			*/
26			public function test_join_with_equal_sign( $name, $value, $expected_output ) {
27			$signature = new \Jetpack_Signature( 'some-secret', 0 );
			0 ignored issues – show Documentation introduced 2020-09-01 22:15 UTC by Report Bug Copy Issue Report `'some-secret'` is of type `string`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
28			$this->assertEquals( $expected_output, $signature->join_with_equal_sign( $name, $value ) );
29			}
30
31			/**
32			* Data provider for test_join_with_equal_sign.
33			*
34			* The test data arrays have the format:
35			* 'name' => The value that the constant will be set to. Null if the constant will not be set.
36			* 'value' => The name of the constant.
37			* 'expected_output' => The expected output of Utils::jetpack_api_constant_filter().
38			*/
39			public function join_with_equal_sign_data_provider() {
40			return array(
41			'string_value' =>
42			array(
43			'name' => 'street',
44			'value' => '1600 Pennsylvania Ave',
45			'expected_output' => 'street=1600 Pennsylvania Ave',
46			),
47			'array_value' =>
48			array(
49			'name' => 'first_names',
50			'value' => array( 'Michael', 'Jim', 'Pam' ),
51			'expected_output' => array( 'first_names[0]=Michael', 'first_names[1]=Jim', 'first_names[2]=Pam' ),
52			),
53			'nested_array_value' =>
54			array(
55			'name' => 'numbers',
56			'value' => array( array( 0, 1 ), array( 2, 3 ), array( 4, 5 ) ),
57			'expected_output' => array(
58			'numbers[0][0]=0',
59			'numbers[0][1]=1',
60			'numbers[1][0]=2',
61			'numbers[1][1]=3',
62			'numbers[2][0]=4',
63			'numbers[2][1]=5',
64			),
65			),
66			'nested_associative_array_value' =>
67			array(
68			'name' => 'people',
69			'value' => array(
70			array(
71			'last_name' => 'Scott',
72			'first_name' => 'Michael',
73			'city' => 'Boulder',
74			),
75			array(
76			'first_name' => 'Jim',
77			'state' => 'Texas',
78			'last_name' => 'Halpert',
79			),
80			),
81			// Note: Expected output is sorted.
82			'expected_output' => array(
83			'people[0][city]=Boulder',
84			'people[0][first_name]=Michael',
85			'people[0][last_name]=Scott',
86			'people[1][first_name]=Jim',
87			'people[1][last_name]=Halpert',
88			'people[1][state]=Texas',
89			),
90			),
91			);
92			}
93			}
94

Automattic / jetpack

Push — add/nested-query-string-suppor... ( 782fd1 )

join_with_equal_sign_data_provider() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like