WP_Test_Autoloader - Code Metrics - Inspection of "Skip WP_Test_Lazy_Images::test_wp_get_attachment_i..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — update/phpunit-php-8-packages ( 8ca541 )

unknown

created 2020-12-02 22:13 UTC

WP_Test_Autoloader A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

Metric	Value
dl	0
loc	33
rs	10
c	0
b	0
f	0
wmc	2
lcom	0
cbo	3

2 Methods

Rating	Name	Duplication	Size	Complexity
A	set_up()	0	3	1
A	test_autoloader_works()	0	18	1

<?php // phpcs:ignore WordPress.Files.FileName
/**
 * Autoloader test suite.
 *
 * @package automattic/jetpack-autoloader
 */

use Jetpack\AutoloaderTestData\Plugin\Test;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use PHPUnit\Framework\TestCase;

/**
 * Test suite class for the Autoloader.
 */
class WP_Test_Autoloader extends TestCase {

	/**
	 * Setup runs before each test.
	 *
	 * @before
	 */
	public function set_up() {
		spl_autoload_register( 'autoloader' );
	}

	/**
	 * Tests whether manifest registration works with autoloading.
	 */
	public function test_autoloader_works() {
		global $jetpack_autoloader_loader;
		$jetpack_autoloader_loader = new Version_Loader(
			new Version_Selector(),
			array(
				Test::class => array(
					'version' => '1.0.0.0',
					'path'    => TEST_DATA_PATH . '/plugins/plugin_current/includes/class-test.php',
				),
			),
			null,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			null
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

		$class = new Test();

		$this->assertInstanceOf( Test::class, $class );
	}
}


1			<?php // phpcs:ignore WordPress.Files.FileName
2			/**
3			* Autoloader test suite.
4			*
5			* @package automattic/jetpack-autoloader
6			*/
7
8			use Jetpack\AutoloaderTestData\Plugin\Test;
			0 ignored issues – show Bug introduced 2020-10-28 19:16 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `Test`. Let’s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let’s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
9			use PHPUnit\Framework\TestCase;
10
11			/**
12			* Test suite class for the Autoloader.
13			*/
14			class WP_Test_Autoloader extends TestCase {
15
16			/**
17			* Setup runs before each test.
18			*
19			* @before
20			*/
21			public function set_up() {
22			spl_autoload_register( 'autoloader' );
23			}
24
25			/**
26			* Tests whether manifest registration works with autoloading.
27			*/
28			public function test_autoloader_works() {
29			global $jetpack_autoloader_loader;
30			$jetpack_autoloader_loader = new Version_Loader(
31			new Version_Selector(),
32			array(
33			Test::class => array(
34			'version' => '1.0.0.0',
35			'path' => TEST_DATA_PATH . '/plugins/plugin_current/includes/class-test.php',
36			),
37			),
38			null,
			0 ignored issues – show Documentation introduced 2020-09-03 15:36 UTC by Report Bug Copy Issue Report `null` is of type `null`, 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...
39			null
			0 ignored issues – show Documentation introduced 2020-09-03 15:36 UTC by Report Bug Copy Issue Report `null` is of type `null`, 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...
40			);
41
42			$class = new Test();
43
44			$this->assertInstanceOf( Test::class, $class );
45			}
46			}
47

Automattic / jetpack

Push — update/phpunit-php-8-packages ( 8ca541 )

WP_Test_Autoloader A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

2 Methods

Duplication Side-by-Side

Filter issues like