Test_Domain_Only_Admin_Menu - Code Metrics - Inspection of "Admin menu: Add class for domain-only sites" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — add/admin-menu-domain-only ( 7cf836 )

unknown

created 2021-03-23 10:26 UTC

Test_Domain_Only_Admin_Menu A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	86
Duplicated Lines	20.93 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

Metric	Value
dl	18
loc	86
rs	10
c	0
b	0
f	0
wmc	3
lcom	1
cbo	2

3 Methods

Rating	Name	Duplication	Size	Complexity
A	wpSetUpBeforeClass()	6	6	1
A	setUp()	12	12	1
A	test_reregister_menu_items()	0	8	1

How to fix Duplicated Code

<?php
/**
 * Tests for Domain_Only_Admin_Menu class.
 *
 * @package automattic/jetpack
 */

use Automattic\Jetpack\Dashboard_Customizations\Domain_Only_Admin_Menu;
use Automattic\Jetpack\Status;

require_jetpack_file( 'modules/masterbar/admin-menu/class-domain-only-admin-menu.php' );
require_jetpack_file( 'tests/php/modules/masterbar/data/admin-menu.php' );

/**
 * Class Test_Domain_Only_Admin_Menu.
 *
 * @coversDefaultClass Automattic\Jetpack\Dashboard_Customizations\Domain_Only_Admin_Menu
 */
class Test_Domain_Only_Admin_Menu extends WP_UnitTestCase {

	/**
	 * Menu data fixture.
	 *
	 * @var array
	 */
	public static $menu_data;

	/**
	 * Submenu data fixture.
	 *
	 * @var array
	 */
	public static $submenu_data;

	/**
	 * Test domain.
	 *
	 * @var string
	 */
	public static $domain;

	/**
	 * Whether this testsuite is run on WP.com.
	 *
	 * @var bool
	 */
	public static $is_wpcom;

	/**
	 * Admin menu instance.
	 *
	 * @var Domain_Only_Admin_Menu
	 */
	public static $admin_menu;

	/**
	 * Mock user ID.
	 *
	 * @var int
	 */
	private static $user_id = 0;

	/**
	 * Create shared fixtures.
	 *
	 * @param WP_UnitTest_Factory $factory Fixture factory.
	 */
	public static function wpSetUpBeforeClass( $factory ) {
		static::$domain       = ( new Status() )->get_site_suffix();
		static::$user_id      = $factory->user->create( array( 'role' => 'administrator' ) );
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
		static::$menu_data    = get_menu_fixture();
		static::$submenu_data = get_submenu_fixture();
	}

	/**
	 * Set up data.
	 */
	public function setUp() {
		parent::setUp();
		global $menu, $submenu;

		// Initialize in setUp so it registers hooks for every test.
		static::$admin_menu = Domain_Only_Admin_Menu::get_instance();


		$menu    = static::$menu_data;
		$submenu = static::$submenu_data;

		wp_set_current_user( static::$user_id );
class YourClass
{
    private static $someVariable;

    public static function getSomeVariable()
    {
        return static::$someVariable;
    }
}
	}

	/**
	 * Tests reregister_menu_items.
	 *
	 * @covers ::reregister_menu_items
	 */
	public function test_reregister_menu_items() {
		global $menu;

		static::$admin_menu->reregister_menu_items();

		$this->assertCount( 1, $menu );
		$this->assertEquals( 'https://wordpress.com/domains/manage/' . static::$domain, $menu[0][2] );
	}
}


1		<?php
2		/**
3		* Tests for Domain_Only_Admin_Menu class.
4		*
5		* @package automattic/jetpack
6		*/
7
8		use Automattic\Jetpack\Dashboard_Customizations\Domain_Only_Admin_Menu;
9		use Automattic\Jetpack\Status;
10
11		require_jetpack_file( 'modules/masterbar/admin-menu/class-domain-only-admin-menu.php' );
12		require_jetpack_file( 'tests/php/modules/masterbar/data/admin-menu.php' );
13
14		/**
15		* Class Test_Domain_Only_Admin_Menu.
16		*
17		* @coversDefaultClass Automattic\Jetpack\Dashboard_Customizations\Domain_Only_Admin_Menu
18		*/
19		class Test_Domain_Only_Admin_Menu extends WP_UnitTestCase {
20
21		/**
22		* Menu data fixture.
23		*
24		* @var array
25		*/
26		public static $menu_data;
27
28		/**
29		* Submenu data fixture.
30		*
31		* @var array
32		*/
33		public static $submenu_data;
34
35		/**
36		* Test domain.
37		*
38		* @var string
39		*/
40		public static $domain;
41
42		/**
43		* Whether this testsuite is run on WP.com.
44		*
45		* @var bool
46		*/
47		public static $is_wpcom;
48
49		/**
50		* Admin menu instance.
51		*
52		* @var Domain_Only_Admin_Menu
53		*/
54		public static $admin_menu;
55
56		/**
57		* Mock user ID.
58		*
59		* @var int
60		*/
61		private static $user_id = 0;
62
63		/**
64		* Create shared fixtures.
65		*
66		* @param WP_UnitTest_Factory $factory Fixture factory.
67		*/
68	View Code Duplication	public static function wpSetUpBeforeClass( $factory ) {
69		static::$domain = ( new Status() )->get_site_suffix();
70		static::$user_id = $factory->user->create( array( 'role' => 'administrator' ) );
		0 ignored issues – show Bug introduced 2021-02-23 11:23 UTC by Report Bug Copy Issue Report Since `$user_id` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$user_id` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
71		static::$menu_data = get_menu_fixture();
72		static::$submenu_data = get_submenu_fixture();
73		}
74
75		/**
76		* Set up data.
77		*/
78	View Code Duplication	public function setUp() {
79		parent::setUp();
80		global $menu, $submenu;
81
82		// Initialize in setUp so it registers hooks for every test.
83		static::$admin_menu = Domain_Only_Admin_Menu::get_instance();
		0 ignored issues – show Documentation Bug introduced 2021-03-23 10:50 UTC by Report Bug Copy Issue Report It seems like `\Automattic\Jetpack\Dash...in_Menu::get_instance()` of type `object<Automattic\Jetpac...tomizations\Admin_Menu>` is incompatible with the declared type `object<Automattic\Jetpac...Domain_Only_Admin_Menu>` of property `$admin_menu`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
84
85		$menu = static::$menu_data;
86		$submenu = static::$submenu_data;
87
88		wp_set_current_user( static::$user_id );
		0 ignored issues – show Bug introduced 2021-01-14 15:29 UTC by Report Bug Copy Issue Report Since `$user_id` is declared private, accessing it with `static` will lead to errors in possible sub-classes; consider using `self`, or increasing the visibility of `$user_id` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static $someVariable; public static function getSomeVariable() { return static::$someVariable; } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { } YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class SomeClass { private static $someVariable; public static function getSomeVariable() { return self::$someVariable; // self works fine with private. } } Loading history...
89		}
90
91		/**
92		* Tests reregister_menu_items.
93		*
94		* @covers ::reregister_menu_items
95		*/
96		public function test_reregister_menu_items() {
97		global $menu;
98
99		static::$admin_menu->reregister_menu_items();
100
101		$this->assertCount( 1, $menu );
102		$this->assertEquals( 'https://wordpress.com/domains/manage/' . static::$domain, $menu[0][2] );
103		}
104		}
105

Automattic / jetpack

Push — add/admin-menu-domain-only ( 7cf836 )

Test_Domain_Only_Admin_Menu A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like