ParserTest::string_to_amount_provider() - Code Metrics - Inspection of "Update Travis PHP version from 7.2 to 7.3." - pronamic/wp-money - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — develop ( 63d4ce...c4a746 )

by Reüel

created 2019-03-07 12:54 UTC

ParserTest::string_to_amount_provider() A

↳ Parent: ParserTest

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	51
rs	9.069
c	0
b	0
f	0

How to fix Long Method

<?php
/**
 * Parser
 *
 * @author    Pronamic <[email protected]>
 * @copyright 2005-2019 Pronamic
 * @license   GPL-3.0-or-later
 * @package   Pronamic\WordPress\Money
 */

namespace Pronamic\WordPress\Money;

use WP_UnitTestCase;

/**
 * Parser
 *
 * @author  Remco Tolsma
 * @version 1.2.0
 * @since   1.1.0
 */
class ParserTest extends WP_UnitTestCase {
	/**
	 * Setup.
	 */
	public function setUp() {

		parent::setUp();

		$this->parser = new Parser();
class MyClass { }

$x = new MyClass();
$x->foo = true;
	}

	/**
	 * Test string to amount.
	 *
	 * @link https://github.com/pronamic/wp-pronamic-ideal/blob/3.7.3/classes/Pronamic/WP/Pay/Settings.php#L71-L91
	 * @link https://github.com/WordPress/WordPress/blob/4.9.6/wp-includes/class-wp-locale.php
	 * @link https://github.com/WordPress/WordPress/blob/4.9.6/wp-includes/functions.php#L206-L237
	 *
	 * @dataProvider string_to_amount_provider
	 *
	 * @param string $thousands_sep Thousands seperator.
	 * @param string $decimal_sep   Decimal seperator.
	 * @param string $string        String value to convert.
	 * @param float  $expected      Expected float value.
	 */
	public function test_string_to_amount( $thousands_sep, $decimal_sep, $string, $expected ) {
		global $wp_locale;

		$wp_locale->number_format['thousands_sep'] = $thousands_sep;
		$wp_locale->number_format['decimal_point'] = $decimal_sep;

		$money = $this->parser->parse( $string );

		$this->assertEquals( $expected, $money->get_value() );
	}

	/**
	 * String to amount provider.
	 *
	 * @return array
	 */
	public function string_to_amount_provider() {
		return array(
			// Thousands seperator is '' and decimal seperator is '.'.
			array( '', '.', '1', 1 ),
			array( '', '.', '2,5', 2.5 ),
			array( '', '.', '2,50', 2.5 ),
			array( '', '.', '1250,00', 1250 ),
			array( '', '.', '1250,75', 1250.75 ),
			array( '', '.', '1250.75', 1250.75 ),
			array( '', '.', '1.250,00', 1250 ),
			array( '', '.', '2.500,75', 2500.75 ),
			// Thousands seperator is '.' and decimal seperator is ','.
			array( '.', ',', '1', 1 ),
			array( '.', ',', '2,5', 2.5 ),
			array( '.', ',', '2,50', 2.5 ),
			array( '.', ',', '1250,00', 1250 ),
			array( '.', ',', '2500,75', 2500.75 ),
			array( '.', ',', '1.250,00', 1250 ),
			array( '.', ',', '2.500,75', 2500.75 ),
			array( '.', ',', '2.500,750', 2500.75 ),
			array( '.', ',', '1.234.567.890', 1234567890 ),
			// Thousands seperator is ',' and decimal seperator is '.'.
			array( ',', '.', '1', 1 ),
			array( ',', '.', '2.5', 2.5 ),
			array( ',', '.', '2.50', 2.5 ),
			array( ',', '.', '1250.00', 1250 ),
			array( ',', '.', '1250.75', 1250.75 ),
			array( ',', '.', '1,250.00', 1250 ),
			array( ',', '.', '2,500.75', 2500.75 ),
			array( ',', '.', '2,500.', 2500 ),
			// Thousands seperator is ' ' and decimal seperator is '.'.
			array( ' ', '.', '2 500.75', 2500.75 ),
			// Thousands seperator is 't' and decimal seperator is '.'.
			array( 't', '.', '2t500.75', 2500.75 ),
			array( 't', '.', '2t500.7', 2500.7 ),
			// Thousands seperator is 't' and decimal seperator is '-'.
			array( 't', '-', '2t500-75', 2500.75 ),
			array( 't', '-', '2t500-7', 2500.7 ),
			// Thousands seperator is 't' and decimal seperator is ' '.
			array( 't', ' ', '2t500 75', 2500.75 ),
			array( 't', ' ', '2t500 7', 2500.7 ),
			// Thousands seperator is ' ' and decimal seperator is 'd'.
			array( ' ', 'd', '2 500d75', 2500.75 ),
			array( ' ', 'd', '2 500d7', 2500.7 ),
			array( ' ', 'd', '-2 500d75', -2500.75 ),
			array( ' ', 'd', '-2 500d7', -2500.7 ),
			// Other.
			array( '', '', '123456789', 123456789 ),
			array( false, false, '123 456 789', 123456789 ),
		);
	}
}


1			<?php
2			/**
3			* Parser
4			*
5			* @author Pronamic <[email protected]>
6			* @copyright 2005-2019 Pronamic
7			* @license GPL-3.0-or-later
8			* @package Pronamic\WordPress\Money
9			*/
10
11			namespace Pronamic\WordPress\Money;
12
13			use WP_UnitTestCase;
14
15			/**
16			* Parser
17			*
18			* @author Remco Tolsma
19			* @version 1.2.0
20			* @since 1.1.0
21			*/
22			class ParserTest extends WP_UnitTestCase {
23			/**
24			* Setup.
25			*/
26			public function setUp() {
			0 ignored issues – show Coding Style introduced 2018-07-16 15:04 UTC by Report Bug Copy Issue Report The function name `setUp` is in camel caps, but expected `set_up` instead as per the coding standard. Loading history...
27			parent::setUp();
28
29			$this->parser = new Parser();
			0 ignored issues – show Bug introduced 2018-07-16 15:04 UTC by Report Bug Copy Issue Report The property `parser` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
30			}
31
32			/**
33			* Test string to amount.
34			*
35			* @link https://github.com/pronamic/wp-pronamic-ideal/blob/3.7.3/classes/Pronamic/WP/Pay/Settings.php#L71-L91
36			* @link https://github.com/WordPress/WordPress/blob/4.9.6/wp-includes/class-wp-locale.php
37			* @link https://github.com/WordPress/WordPress/blob/4.9.6/wp-includes/functions.php#L206-L237
38			*
39			* @dataProvider string_to_amount_provider
40			*
41			* @param string $thousands_sep Thousands seperator.
42			* @param string $decimal_sep Decimal seperator.
43			* @param string $string String value to convert.
44			* @param float $expected Expected float value.
45			*/
46			public function test_string_to_amount( $thousands_sep, $decimal_sep, $string, $expected ) {
47			global $wp_locale;
48
49			$wp_locale->number_format['thousands_sep'] = $thousands_sep;
50			$wp_locale->number_format['decimal_point'] = $decimal_sep;
51
52			$money = $this->parser->parse( $string );
53
54			$this->assertEquals( $expected, $money->get_value() );
55			}
56
57			/**
58			* String to amount provider.
59			*
60			* @return array
61			*/
62			public function string_to_amount_provider() {
63			return array(
64			// Thousands seperator is '' and decimal seperator is '.'.
65			array( '', '.', '1', 1 ),
66			array( '', '.', '2,5', 2.5 ),
67			array( '', '.', '2,50', 2.5 ),
68			array( '', '.', '1250,00', 1250 ),
69			array( '', '.', '1250,75', 1250.75 ),
70			array( '', '.', '1250.75', 1250.75 ),
71			array( '', '.', '1.250,00', 1250 ),
72			array( '', '.', '2.500,75', 2500.75 ),
73			// Thousands seperator is '.' and decimal seperator is ','.
74			array( '.', ',', '1', 1 ),
75			array( '.', ',', '2,5', 2.5 ),
76			array( '.', ',', '2,50', 2.5 ),
77			array( '.', ',', '1250,00', 1250 ),
78			array( '.', ',', '2500,75', 2500.75 ),
79			array( '.', ',', '1.250,00', 1250 ),
80			array( '.', ',', '2.500,75', 2500.75 ),
81			array( '.', ',', '2.500,750', 2500.75 ),
82			array( '.', ',', '1.234.567.890', 1234567890 ),
83			// Thousands seperator is ',' and decimal seperator is '.'.
84			array( ',', '.', '1', 1 ),
85			array( ',', '.', '2.5', 2.5 ),
86			array( ',', '.', '2.50', 2.5 ),
87			array( ',', '.', '1250.00', 1250 ),
88			array( ',', '.', '1250.75', 1250.75 ),
89			array( ',', '.', '1,250.00', 1250 ),
90			array( ',', '.', '2,500.75', 2500.75 ),
91			array( ',', '.', '2,500.', 2500 ),
92			// Thousands seperator is ' ' and decimal seperator is '.'.
93			array( ' ', '.', '2 500.75', 2500.75 ),
94			// Thousands seperator is 't' and decimal seperator is '.'.
95			array( 't', '.', '2t500.75', 2500.75 ),
96			array( 't', '.', '2t500.7', 2500.7 ),
97			// Thousands seperator is 't' and decimal seperator is '-'.
98			array( 't', '-', '2t500-75', 2500.75 ),
99			array( 't', '-', '2t500-7', 2500.7 ),
100			// Thousands seperator is 't' and decimal seperator is ' '.
101			array( 't', ' ', '2t500 75', 2500.75 ),
102			array( 't', ' ', '2t500 7', 2500.7 ),
103			// Thousands seperator is ' ' and decimal seperator is 'd'.
104			array( ' ', 'd', '2 500d75', 2500.75 ),
105			array( ' ', 'd', '2 500d7', 2500.7 ),
106			array( ' ', 'd', '-2 500d75', -2500.75 ),
107			array( ' ', 'd', '-2 500d7', -2500.7 ),
108			// Other.
109			array( '', '', '123456789', 123456789 ),
110			array( false, false, '123 456 789', 123456789 ),
111			);
112			}
113			}
114

pronamic / wp-money

Push — develop ( 63d4ce...c4a746 )

ParserTest::string_to_amount_provider() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like