PaymentRequestTest::test() - Code Metrics - Inspection of "Merge branch 'release/1.0.0' into main" - wp-pay-gateways/payvision - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 061772...28b955 )

by Remco

created 2020-11-18 07:56 UTC

PaymentRequestTest::test() A

↳ Parent: PaymentRequestTest

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	21
rs	9.584
c	0
b	0
f	0

<?php
/**
 * Payment Request Test
 *
 * @author    Pronamic <[email protected]>
 * @copyright 2005-2020 Pronamic
 * @license   GPL-3.0-or-later
 * @package   Pronamic\WordPress\Pay\Gateways\Payvision
 */

namespace Pronamic\WordPress\Pay\Gateways\Payvision;

/**
 * Payment Request Test
 *
 * @author  Remco Tolsma
 * @version 1.0.0
 * @since   1.0.0
 */
class PaymentRequestTest extends \WP_UnitTestCase {
	/**
	 * Test.
	 */
	public function test() {
		$header = new RequestHeader( '123456' );

		$transaction = new Transaction( '1', 50, 'EUR', '12345678' );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

		$payment_request = new PaymentRequest( $header, $transaction );

		$bank = new BankDetails();
		$bank->set_issuer_id( IssuerIdIDeal::ABN_AMRO );

		$payment_request->set_bank( $bank );

		$this->assertInstanceOf( PaymentRequest::class, $payment_request );

		// JSON.
		$json_file = __DIR__ . '/../json/payment-request.json';

		$json_string = \wp_json_encode( $payment_request, \JSON_PRETTY_PRINT );

		$this->assertJsonStringEqualsJsonFile( $json_file, $json_string );
	}
}


1			<?php
2			/**
3			* Payment Request Test
4			*
5			* @author Pronamic <[email protected]>
6			* @copyright 2005-2020 Pronamic
7			* @license GPL-3.0-or-later
8			* @package Pronamic\WordPress\Pay\Gateways\Payvision
9			*/
10
11			namespace Pronamic\WordPress\Pay\Gateways\Payvision;
12
13			/**
14			* Payment Request Test
15			*
16			* @author Remco Tolsma
17			* @version 1.0.0
18			* @since 1.0.0
19			*/
20			class PaymentRequestTest extends \WP_UnitTestCase {
21			/**
22			* Test.
23			*/
24			public function test() {
25			$header = new RequestHeader( '123456' );
26
27			$transaction = new Transaction( '1', 50, 'EUR', '12345678' );
			0 ignored issues – show Documentation introduced 2020-11-18 08:03 UTC by Report Bug Copy Issue Report `'12345678'` is of type `string`, but the function expects a `object<Pronamic\WordPres...Payvision\TrackingCode>`. 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
29			$payment_request = new PaymentRequest( $header, $transaction );
30
31			$bank = new BankDetails();
32			$bank->set_issuer_id( IssuerIdIDeal::ABN_AMRO );
33
34			$payment_request->set_bank( $bank );
35
36			$this->assertInstanceOf( PaymentRequest::class, $payment_request );
37
38			// JSON.
39			$json_file = __DIR__ . '/../json/payment-request.json';
40
41			$json_string = \wp_json_encode( $payment_request, \JSON_PRETTY_PRINT );
42
43			$this->assertJsonStringEqualsJsonFile( $json_file, $json_string );
44			}
45			}
46

wp-pay-gateways / payvision

Push — main ( 061772...28b955 )

PaymentRequestTest::test() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like