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

Test Failed

Push — main ( e0d6a9...e106fe )

by Reüel

created 2021-04-26 19:23 UTC

TransactionTest::test() A

↳ Parent: TransactionTest

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	18
rs	9.6666
c	0
b	0
f	0

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

namespace Pronamic\WordPress\Pay\Gateways\Payvision;

/**
 * Transaction Test
 *
 * @author  Remco Tolsma
 * @version 1.1.0
 * @since   1.0.0
 */
class TransactionTest extends \WP_UnitTestCase {
	/**
	 * Test.
	 */
	public function test() {
		$transaction = new Transaction( '123456', 50, 'EUR', '12345678' );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

		$transaction->set_brand_id( BrandId::IDEAL );
		$transaction->set_purchase_id( 123456 );
		$transaction->set_return_url( 'https://example.com/' );

		$this->assertInstanceOf( Transaction::class, $transaction );

		$this->assertEquals( BrandId::IDEAL, $transaction->get_brand_id() );

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

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

		$this->assertJsonStringEqualsJsonFile( $json_file, $json_string );
<?php

function getDate($date)
{
    if ($date !== null) {
        return new DateTime($date);
    }

    return false;
}
	}
}


1			<?php
2			/**
3			* Transaction Test
4			*
5			* @author Pronamic <[email protected]>
6			* @copyright 2005-2021 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			* Transaction Test
15			*
16			* @author Remco Tolsma
17			* @version 1.1.0
18			* @since 1.0.0
19			*/
20			class TransactionTest extends \WP_UnitTestCase {
21			/**
22			* Test.
23			*/
24			public function test() {
25			$transaction = new Transaction( '123456', 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...
26
27			$transaction->set_brand_id( BrandId::IDEAL );
28			$transaction->set_purchase_id( 123456 );
29			$transaction->set_return_url( 'https://example.com/' );
30
31			$this->assertInstanceOf( Transaction::class, $transaction );
32
33			$this->assertEquals( BrandId::IDEAL, $transaction->get_brand_id() );
34
35			// JSON.
36			$json_file = __DIR__ . '/../json/transaction-request.json';
37
38			$json_string = \wp_json_encode( $transaction, \JSON_PRETTY_PRINT );
39
40			$this->assertJsonStringEqualsJsonFile( $json_file, $json_string );
			0 ignored issues – show Security Bug introduced 2021-02-08 10:12 UTC by Report Bug Copy Issue Report It seems like `$json_string` defined by `\wp_json_encode($transaction, \JSON_PRETTY_PRINT)` on line `38` can also be of type `false`; however, `PHPUnit\Framework\Assert...nStringEqualsJsonFile()` does only seem to accept `string`, did you maybe forget to handle an error condition? This check looks for type mismatches where the missing type is `false`. This is usually indicative of an error condtion. Consider the follow example <?php function getDate($date) { if ($date !== null) { return new DateTime($date); } return false; } This function either returns a new `DateTime` object or false, if there was an error. This is a typical pattern in PHP programming to show that an error has occurred without raising an exception. The calling code should check for this returned `false` before passing on the value to another function or method that may not be able to handle a `false`. Loading history...
41			}
42			}
43

wp-pay-gateways / payvision

Push — main ( e0d6a9...e106fe )

TransactionTest::test() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like