Test_Atomic_Admin_Menu - Code Metrics - Inspection of "Add more use-cases where we can auto-label" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — test/auto-label-plugin ( 097985...f0c981 )

by Jeremy

created 2021-02-09 15:27 UTC

Test_Atomic_Admin_Menu A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	314
Duplicated Lines	13.69 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

Metric	Value
dl	43
loc	314
rs	10
c	0
b	0
f	0
wmc	17
lcom	1
cbo	2

15 Methods

Rating	Name	Duplication	Size	Complexity
A	test-class-atomic-admin-menu.php ➔ site_is_private()	0	3	1
A	wpSetUpBeforeClass()	6	6	1
A	setUp()	12	12	1
A	test_get_instance()	9	9	1
A	test_add_browse_sites_link()	0	7	1
A	test_add_browse_sites_link_multisite()	0	29	2
A	test_add_new_site_link()	0	21	1
A	test_add_site_card_menu()	0	27	2
A	test_set_site_card_menu_class()	0	26	2
A	wpcomsh_site_icon_url()	0	3	1
A	custom_site_icon_url()	0	3	1
A	test_add_wpcom_upgrades_menu()	0	13	1
A	test_add_tools_menu()	15	15	1
A	test_add_options_menu()	0	8	1
A	test_add_plugins_menu()	0	8	1

How to fix Duplicated Code

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

use Automattic\Jetpack\Dashboard_Customizations\Atomic_Admin_Menu;
use Automattic\Jetpack\Status;

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

/**
 * Class Test_Atomic_Admin_Menu.
 *
 * @coversDefaultClass Automattic\Jetpack\Dashboard_Customizations\Atomic_Admin_Menu
 */
class Test_Atomic_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 Atomic_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 = Atomic_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;
    }
}
	}

	/**
	 * Test get_instance.
	 *
	 * @covers ::get_instance
	 * @covers ::__construct
	 */
	public function test_get_instance() {
		$instance = Atomic_Admin_Menu::get_instance();

		$this->assertInstanceOf( Atomic_Admin_Menu::class, $instance );
		$this->assertSame( $instance, static::$admin_menu );

		$this->assertSame( 99999, has_action( 'admin_menu', array( $instance, 'reregister_menu_items' ) ) );
		$this->assertSame( 10, has_action( 'admin_enqueue_scripts', array( $instance, 'enqueue_scripts' ) ) );
	}

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

		// No output when executed in single site mode.
		static::$admin_menu->add_browse_sites_link();
		$this->assertArrayNotHasKey( 0, $menu );
	}

	/**
	 * Tests add_browse_sites_link.
	 *
	 * @covers ::add_browse_sites_link
	 */
	public function test_add_browse_sites_link_multisite() {
		if ( ! is_multisite() ) {
			$this->markTestSkipped( 'Only used on multisite' );
		}

		global $menu;

		// No output when user has just one site.
		static::$admin_menu->add_browse_sites_link();
		$this->assertArrayNotHasKey( 0, $menu );

		// Give user a second site.
		update_user_option( static::$user_id, 'wpcom_site_count', 2 );
class YourClass
{
    private static $someVariable;

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

		static::$admin_menu->add_browse_sites_link();

		$browse_sites_menu_item = array(
			'Browse sites',
			'read',
			'https://wordpress.com/home',
			'site-switcher',
			'menu-top toplevel_page_https://wordpress.com/home',
			'toplevel_page_https://wordpress.com/home',
			'dashicons-arrow-left-alt2',
		);
		$this->assertSame( $menu[0], $browse_sites_menu_item );

		delete_user_option( static::$user_id, 'wpcom_site_count' );
class YourClass
{
    private static $someVariable;

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

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

		// Set jetpack user data.
		update_user_option( static::$user_id, 'wpcom_site_count', 1 );
class YourClass
{
    private static $someVariable;

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

		static::$admin_menu->add_new_site_link();

		$new_site_menu_item = array(
			'Add new site',
			'read',
			'https://wordpress.com/start?ref=calypso-sidebar',
			'Add new site',
			'menu-top toplevel_page_https://wordpress.com/start?ref=calypso-sidebar',
			'toplevel_page_https://wordpress.com/start?ref=calypso-sidebar',
			'dashicons-plus-alt',
		);
		$this->assertSame( $menu[1002], $new_site_menu_item ); // 1001 is the separator position, 1002 is the link position

		delete_user_option( static::$user_id, 'wpcom_site_count' );
class YourClass
{
    private static $someVariable;

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

	/**
	 * Tests add_site_card_menu
	 *
	 * @covers ::add_site_card_menu
	 */
	public function test_add_site_card_menu() {
		global $menu;

		if ( ! function_exists( 'site_is_private' ) ) {
			function site_is_private() { // phpcs:ignore
				return false;
			}
		}
		static::$admin_menu->add_site_card_menu();

		$home_url            = home_url();
		$site_card_menu_item = array(
			// phpcs:ignore Squiz.Strings.DoubleQuoteUsage.NotRequired
			'
<div class="site__info">
	<div class="site__title">' . get_option( 'blogname' ) . '</div>
	<div class="site__domain">' . static::$domain . "</div>\n\t\n</div>",
			'read',
			$home_url,
			'site-card',
			'menu-top toplevel_page_' . $home_url,
			'toplevel_page_' . $home_url,
			'data:image/svg+xml,%3Csvg%20class%3D%22gridicon%22%20xmlns%3D%22http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg%22%20viewBox%3D%220%200%2024%2024%22%3E%3Ctitle%3EGlobe%3C%2Ftitle%3E%3Crect%20fill-opacity%3D%220%22%20x%3D%220%22%20width%3D%2224%22%20height%3D%2224%22%2F%3E%3Cg%3E%3Cpath%20fill%3D%22%23fff%22%20d%3D%22M12%202C6.477%202%202%206.477%202%2012s4.477%2010%2010%2010%2010-4.477%2010-10S17.523%202%2012%202zm0%2018l2-2%201-1v-2h-2v-1l-1-1H9v3l2%202v1.93c-3.94-.494-7-3.858-7-7.93l1%201h2v-2h2l3-3V6h-2L9%205v-.41C9.927%204.21%2010.94%204%2012%204s2.073.212%203%20.59V6l-1%201v2l1%201%203.13-3.13c.752.897%201.304%201.964%201.606%203.13H18l-2%202v2l1%201h2l.286.286C18.03%2018.06%2015.24%2020%2012%2020z%22%2F%3E%3C%2Fg%3E%3C%2Fsvg%3E',
		);

		$this->assertEquals( $menu[1], $site_card_menu_item );
	}

	/**
	 * Tests set_site_card_menu_class
	 *
	 * @covers ::set_site_card_menu_class
	 */
	public function test_set_site_card_menu_class() {
		global $menu;

		if ( ! function_exists( 'site_is_private' ) ) {
			function site_is_private() { // phpcs:ignore
/**
 * @ignore
 */
function getUser() {

}

function getUser($id, $realm) {

}
				return false;
			}
		}

		static::$admin_menu->add_site_card_menu();

		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
		$this->assertNotContains( 'has-site-icon', $menu[1][4] );

		// Atomic fallback site icon counts as no site icon.
		add_filter( 'get_site_icon_url', array( $this, 'wpcomsh_site_icon_url' ) );
		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
		remove_filter( 'get_site_icon_url', array( $this, 'wpcomsh_site_icon_url' ) );
		$this->assertNotContains( 'has-site-icon', $menu[1][4] );

		// Custom site icon triggers CSS class.
		add_filter( 'get_site_icon_url', array( $this, 'custom_site_icon_url' ) );
		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
		remove_filter( 'get_site_icon_url', array( $this, 'custom_site_icon_url' ) );
		$this->assertContains( 'has-site-icon', $menu[1][4] );
	}

	/**
	 * Shim wpcomsh fallback site icon.
	 *
	 * @return string
	 */
	public function wpcomsh_site_icon_url() {
		return 'https://s0.wp.com/i/webclip.png';
	}

	/**
	 * Custom site icon.
	 *
	 * @return string
	 */
	public function custom_site_icon_url() {
		return 'https://s0.wp.com/i/jetpack.png';
	}

	/**
	 * Tests add_upgrades_menu
	 *
	 * @covers ::add_upgrades_menu
	 */
	public function test_add_wpcom_upgrades_menu() {
		global $submenu;

		static::$admin_menu->add_upgrades_menu();

		$domains_submenu_item = array(
			'Domains',
			'manage_options',
			'https://wordpress.com/domains/manage/' . static::$domain,
			'Domains',
		);
		$this->assertContains( $domains_submenu_item, $submenu[ 'https://wordpress.com/plans/' . static::$domain ] );
	}

	/**
	 * Tests add_tools_menu
	 *
	 * @covers ::add_tools_menu
	 */
	public function test_add_tools_menu() {
		global $submenu;

		$slug = 'https://wordpress.com/marketing/tools/' . static::$domain;
		static::$admin_menu->add_tools_menu( false, false );

		// Check Export menu always links to WP Admin.
		$export_submenu_item = array(
			'Export',
			'export',
			'export.php',
			'Export',
		);
		$this->assertContains( $export_submenu_item, $submenu[ $slug ] );
	}

	/**
	 * Tests add_options_menu
	 *
	 * @covers ::add_options_menu
	 */
	public function test_add_options_menu() {
		global $submenu;

		$slug = 'https://wordpress.com/settings/general/' . static::$domain;
		static::$admin_menu->add_options_menu( false );

		$this->assertContains( 'Hosting Configuration', $submenu[ $slug ][6] );
	}

	/**
	 * Tests add_plugins_menu
	 *
	 * @covers ::add_plugins_menu
	 */
	public function test_add_plugins_menu() {
		global $menu;

		static::$admin_menu->add_plugins_menu( false );

		// Check Plugins menu always links to WP Admin.
		$this->assertContains( 'plugins.php', $menu[65] );
	}
}


1		<?php
2		/**
3		* Tests for Atomic_Admin_Menu class.
4		*
5		* @package automattic/jetpack
6		*/
7
8		use Automattic\Jetpack\Dashboard_Customizations\Atomic_Admin_Menu;
9		use Automattic\Jetpack\Status;
10
11		require_jetpack_file( 'modules/masterbar/admin-menu/class-atomic-admin-menu.php' );
12		require_jetpack_file( 'tests/php/modules/masterbar/data/admin-menu.php' );
13
14		/**
15		* Class Test_Atomic_Admin_Menu.
16		*
17		* @coversDefaultClass Automattic\Jetpack\Dashboard_Customizations\Atomic_Admin_Menu
18		*/
19		class Test_Atomic_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 Atomic_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 2020-12-17 22:10 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 = Atomic_Admin_Menu::get_instance();
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-22 04:58 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		* Test get_instance.
93		*
94		* @covers ::get_instance
95		* @covers ::__construct
96		*/
97	View Code Duplication	public function test_get_instance() {
98		$instance = Atomic_Admin_Menu::get_instance();
99
100		$this->assertInstanceOf( Atomic_Admin_Menu::class, $instance );
101		$this->assertSame( $instance, static::$admin_menu );
102
103		$this->assertSame( 99999, has_action( 'admin_menu', array( $instance, 'reregister_menu_items' ) ) );
104		$this->assertSame( 10, has_action( 'admin_enqueue_scripts', array( $instance, 'enqueue_scripts' ) ) );
105		}
106
107		/**
108		* Tests add_browse_sites_link.
109		*
110		* @covers ::add_browse_sites_link
111		*/
112		public function test_add_browse_sites_link() {
113		global $menu;
114
115		// No output when executed in single site mode.
116		static::$admin_menu->add_browse_sites_link();
117		$this->assertArrayNotHasKey( 0, $menu );
118		}
119
120		/**
121		* Tests add_browse_sites_link.
122		*
123		* @covers ::add_browse_sites_link
124		*/
125		public function test_add_browse_sites_link_multisite() {
126		if ( ! is_multisite() ) {
127		$this->markTestSkipped( 'Only used on multisite' );
128		}
129
130		global $menu;
131
132		// No output when user has just one site.
133		static::$admin_menu->add_browse_sites_link();
134		$this->assertArrayNotHasKey( 0, $menu );
135
136		// Give user a second site.
137		update_user_option( static::$user_id, 'wpcom_site_count', 2 );
		0 ignored issues – show Bug introduced 2021-01-21 03:25 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...
138
139		static::$admin_menu->add_browse_sites_link();
140
141		$browse_sites_menu_item = array(
142		'Browse sites',
143		'read',
144		'https://wordpress.com/home',
145		'site-switcher',
146		'menu-top toplevel_page_https://wordpress.com/home',
147		'toplevel_page_https://wordpress.com/home',
148		'dashicons-arrow-left-alt2',
149		);
150		$this->assertSame( $menu[0], $browse_sites_menu_item );
151
152		delete_user_option( static::$user_id, 'wpcom_site_count' );
		0 ignored issues – show Bug introduced 2021-01-21 03:25 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...
153		}
154
155		/**
156		* Tests add_new_site_link.
157		*
158		* @covers ::add_new_site_link
159		*/
160		public function test_add_new_site_link() {
161		global $menu;
162
163		// Set jetpack user data.
164		update_user_option( static::$user_id, 'wpcom_site_count', 1 );
		0 ignored issues – show Bug introduced 2021-01-21 03:25 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...
165
166		static::$admin_menu->add_new_site_link();
167
168		$new_site_menu_item = array(
169		'Add new site',
170		'read',
171		'https://wordpress.com/start?ref=calypso-sidebar',
172		'Add new site',
173		'menu-top toplevel_page_https://wordpress.com/start?ref=calypso-sidebar',
174		'toplevel_page_https://wordpress.com/start?ref=calypso-sidebar',
175		'dashicons-plus-alt',
176		);
177		$this->assertSame( $menu[1002], $new_site_menu_item ); // 1001 is the separator position, 1002 is the link position
178
179		delete_user_option( static::$user_id, 'wpcom_site_count' );
		0 ignored issues – show Bug introduced 2021-01-21 03:25 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...
180		}
181
182		/**
183		* Tests add_site_card_menu
184		*
185		* @covers ::add_site_card_menu
186		*/
187		public function test_add_site_card_menu() {
188		global $menu;
189
190		if ( ! function_exists( 'site_is_private' ) ) {
191		function site_is_private() { // phpcs:ignore
192		return false;
193		}
194		}
195		static::$admin_menu->add_site_card_menu();
196
197		$home_url = home_url();
198		$site_card_menu_item = array(
199		// phpcs:ignore Squiz.Strings.DoubleQuoteUsage.NotRequired
200		'
201		<div class="site__info">
202		<div class="site__title">' . get_option( 'blogname' ) . '</div>
203		<div class="site__domain">' . static::$domain . "</div>\n\t\n</div>",
204		'read',
205		$home_url,
206		'site-card',
207		'menu-top toplevel_page_' . $home_url,
208		'toplevel_page_' . $home_url,
209		'data:image/svg+xml,%3Csvg%20class%3D%22gridicon%22%20xmlns%3D%22http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg%22%20viewBox%3D%220%200%2024%2024%22%3E%3Ctitle%3EGlobe%3C%2Ftitle%3E%3Crect%20fill-opacity%3D%220%22%20x%3D%220%22%20width%3D%2224%22%20height%3D%2224%22%2F%3E%3Cg%3E%3Cpath%20fill%3D%22%23fff%22%20d%3D%22M12%202C6.477%202%202%206.477%202%2012s4.477%2010%2010%2010%2010-4.477%2010-10S17.523%202%2012%202zm0%2018l2-2%201-1v-2h-2v-1l-1-1H9v3l2%202v1.93c-3.94-.494-7-3.858-7-7.93l1%201h2v-2h2l3-3V6h-2L9%205v-.41C9.927%204.21%2010.94%204%2012%204s2.073.212%203%20.59V6l-1%201v2l1%201%203.13-3.13c.752.897%201.304%201.964%201.606%203.13H18l-2%202v2l1%201h2l.286.286C18.03%2018.06%2015.24%2020%2012%2020z%22%2F%3E%3C%2Fg%3E%3C%2Fsvg%3E',
210		);
211
212		$this->assertEquals( $menu[1], $site_card_menu_item );
213		}
214
215		/**
216		* Tests set_site_card_menu_class
217		*
218		* @covers ::set_site_card_menu_class
219		*/
220		public function test_set_site_card_menu_class() {
221		global $menu;
222
223		if ( ! function_exists( 'site_is_private' ) ) {
224		function site_is_private() { // phpcs:ignore
		0 ignored issues – show Best Practice introduced 2021-01-05 05:08 UTC by Report Bug Copy Issue Report The function `site_is_private()` has been defined more than once; this definition is ignored, only the first definition in this file (L191-193) is considered. This check looks for functions that have already been defined in the same file. Some Codebases, like WordPress, make a practice of defining functions multiple times. This may lead to problems with the detection of function parameters and types. If you really need to do this, you can mark the duplicate definition with the `@ignore` annotation. /** * @ignore */ function getUser() { } function getUser($id, $realm) { } See also the PhpDoc documentation for @ignore. Loading history...
225		return false;
226		}
227		}
228
229		static::$admin_menu->add_site_card_menu();
230
231		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
232		$this->assertNotContains( 'has-site-icon', $menu[1][4] );
233
234		// Atomic fallback site icon counts as no site icon.
235		add_filter( 'get_site_icon_url', array( $this, 'wpcomsh_site_icon_url' ) );
236		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
237		remove_filter( 'get_site_icon_url', array( $this, 'wpcomsh_site_icon_url' ) );
238		$this->assertNotContains( 'has-site-icon', $menu[1][4] );
239
240		// Custom site icon triggers CSS class.
241		add_filter( 'get_site_icon_url', array( $this, 'custom_site_icon_url' ) );
242		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
243		remove_filter( 'get_site_icon_url', array( $this, 'custom_site_icon_url' ) );
244		$this->assertContains( 'has-site-icon', $menu[1][4] );
245		}
246
247		/**
248		* Shim wpcomsh fallback site icon.
249		*
250		* @return string
251		*/
252		public function wpcomsh_site_icon_url() {
253		return 'https://s0.wp.com/i/webclip.png';
254		}
255
256		/**
257		* Custom site icon.
258		*
259		* @return string
260		*/
261		public function custom_site_icon_url() {
262		return 'https://s0.wp.com/i/jetpack.png';
263		}
264
265		/**
266		* Tests add_upgrades_menu
267		*
268		* @covers ::add_upgrades_menu
269		*/
270		public function test_add_wpcom_upgrades_menu() {
271		global $submenu;
272
273		static::$admin_menu->add_upgrades_menu();
274
275		$domains_submenu_item = array(
276		'Domains',
277		'manage_options',
278		'https://wordpress.com/domains/manage/' . static::$domain,
279		'Domains',
280		);
281		$this->assertContains( $domains_submenu_item, $submenu[ 'https://wordpress.com/plans/' . static::$domain ] );
282		}
283
284		/**
285		* Tests add_tools_menu
286		*
287		* @covers ::add_tools_menu
288		*/
289	View Code Duplication	public function test_add_tools_menu() {
290		global $submenu;
291
292		$slug = 'https://wordpress.com/marketing/tools/' . static::$domain;
293		static::$admin_menu->add_tools_menu( false, false );
294
295		// Check Export menu always links to WP Admin.
296		$export_submenu_item = array(
297		'Export',
298		'export',
299		'export.php',
300		'Export',
301		);
302		$this->assertContains( $export_submenu_item, $submenu[ $slug ] );
303		}
304
305		/**
306		* Tests add_options_menu
307		*
308		* @covers ::add_options_menu
309		*/
310		public function test_add_options_menu() {
311		global $submenu;
312
313		$slug = 'https://wordpress.com/settings/general/' . static::$domain;
314		static::$admin_menu->add_options_menu( false );
315
316		$this->assertContains( 'Hosting Configuration', $submenu[ $slug ][6] );
317		}
318
319		/**
320		* Tests add_plugins_menu
321		*
322		* @covers ::add_plugins_menu
323		*/
324		public function test_add_plugins_menu() {
325		global $menu;
326
327		static::$admin_menu->add_plugins_menu( false );
328
329		// Check Plugins menu always links to WP Admin.
330		$this->assertContains( 'plugins.php', $menu[65] );
331		}
332		}
333

Automattic / jetpack

Push — test/auto-label-plugin ( 097985...f0c981 )

Test_Atomic_Admin_Menu A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

15 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like