Test_WPcom_Admin_Menu - Code Metrics - Inspection of "Merge branch 'master' into kraftbj-patch-2" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — kraftbj-patch-2 ( f1a4bb...ba1272 )

unknown

created 2021-03-01 20:05 UTC

Test_WPcom_Admin_Menu A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	344
Duplicated Lines	18.31 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

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

15 Methods

Rating	Name	Duplication	Size	Complexity
A	wpSetUpBeforeClass()	6	6	1
A	setUp()	12	12	1
A	test_get_instance()	9	9	1
A	test_add_browse_sites_link()	0	29	2
A	test_add_new_site_link()	0	16	1
A	test_add_site_card_menu()	0	26	2
A	test_set_site_card_menu_class()	0	24	2
A	wpcomsh_site_icon_url()	0	3	1
A	custom_site_icon_url()	0	3	1
A	test_add_users_menu()	0	8	1
A	test_add_tools_menu()	15	15	1
A	test_add_options_menu()	0	16	1
A	test_add_gutenberg_menus()	0	23	1
A	test_add_wpcom_upgrades_menu()	13	13	1
A	test_add_plugins_menu()	8	8	1

How to fix Duplicated Code

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

use Automattic\Jetpack\Dashboard_Customizations\WPcom_Admin_Menu;
use Automattic\Jetpack\Status;

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

/**
 * Class Test_WPcom_Admin_Menu.
 *
 * @coversDefaultClass Automattic\Jetpack\Dashboard_Customizations\WPcom_Admin_Menu
 */
class Test_WPcom_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 WPcom_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_wpcom_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 = WPcom_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 = WPcom_Admin_Menu::get_instance();

		$this->assertInstanceOf( WPcom_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() {
		if ( ! function_exists( 'add_user_to_blog' ) ) {
			$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.
		$blog_id = $this->factory->blog->create();
		add_user_to_blog( $blog_id, get_current_user_id(), 'editor' );

		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 );

		remove_user_from_blog( get_current_user_id(), $blog_id );
	}

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

		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
	}

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

		if ( ! static::$is_wpcom ) {
			$this->markTestSkipped( 'Only used on WP.com.' );
		}

		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</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 ( ! static::$is_wpcom ) {
			$this->markTestSkipped( 'Only used on WP.com.' );
		}

		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_users_menu
	 *
	 * @covers ::add_users_menu
	 */
	public function test_add_users_menu() {
		global $menu;

		static::$admin_menu->add_users_menu( true );

		// Check that menu always links to Calypso.
		$this->assertSame( $menu[70][2], 'https://wordpress.com/people/team/' . 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, true );

		// Check Export menu item always links to Calypso.
		$export_submenu_item = array(
			'Export',
			'export',
			'https://wordpress.com/export/' . static::$domain,
			'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] );

		$sharing_submenu_item = array(
			'Sharing',
			'publish_posts',
			'https://wordpress.com/marketing/sharing-buttons/' . static::$domain,
			'Sharing Settings',
		);
		$this->assertContains( $sharing_submenu_item, $submenu[ $slug ] );
	}

	/**
	 * Tests add_gutenberg_menus
	 *
	 * @covers ::add_gutenberg_menus
	 */
	public function test_add_gutenberg_menus() {
		global $menu;
		static::$admin_menu->add_gutenberg_menus( false );

		// Gutenberg plugin menu should not be visible.
		$this->assertArrayNotHasKey( 101, $menu );

		// FSE is no longer where it was put by default.
		$this->assertArrayNotHasKey( 100, $menu );
		$this->assertArrayHasKey( 61, $menu );

		$fse_link = 'https://wordpress.com/site-editor/' . static::$domain;
		$fse_menu = array(
			'Site Editor',
			'edit_theme_options',
			$fse_link,
			'Site Editor',
			'menu-top toplevel_page_' . $fse_link,
			'toplevel_page_' . $fse_link,
			'dashicons-layout',
		);
		$this->assertSame( $menu[61], $fse_menu );
	}

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

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

		// Check Plugins menu always links to Calypso.
		$this->assertContains( 'https://wordpress.com/plugins/' . static::$domain, $menu[65] );
	}
}


1		<?php
2		/**
3		* Tests for WPcom_Admin_Menu class.
4		*
5		* @package automattic/jetpack
6		*/
7
8		use Automattic\Jetpack\Dashboard_Customizations\WPcom_Admin_Menu;
9		use Automattic\Jetpack\Status;
10
11		require_jetpack_file( 'modules/masterbar/admin-menu/class-wpcom-admin-menu.php' );
12		require_jetpack_file( 'tests/php/modules/masterbar/data/admin-menu.php' );
13
14		/**
15		* Class Test_WPcom_Admin_Menu.
16		*
17		* @coversDefaultClass Automattic\Jetpack\Dashboard_Customizations\WPcom_Admin_Menu
18		*/
19		class Test_WPcom_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 WPcom_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-01-29 13:27 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_wpcom_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 = WPcom_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-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		* Test get_instance.
93		*
94		* @covers ::get_instance
95		* @covers ::__construct
96		*/
97	View Code Duplication	public function test_get_instance() {
98		$instance = WPcom_Admin_Menu::get_instance();
99
100		$this->assertInstanceOf( WPcom_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		if ( ! function_exists( 'add_user_to_blog' ) ) {
114		$this->markTestSkipped( 'Only used on multisite' );
115		}
116		global $menu;
117
118		// No output when user has just one site.
119		static::$admin_menu->add_browse_sites_link();
120		$this->assertArrayNotHasKey( 0, $menu );
121
122		// Give user a second site.
123		$blog_id = $this->factory->blog->create();
124		add_user_to_blog( $blog_id, get_current_user_id(), 'editor' );
125
126		static::$admin_menu->add_browse_sites_link();
127
128		$browse_sites_menu_item = array(
129		'Browse sites',
130		'read',
131		'https://wordpress.com/home',
132		'site-switcher',
133		'menu-top toplevel_page_https://wordpress.com/home',
134		'toplevel_page_https://wordpress.com/home',
135		'dashicons-arrow-left-alt2',
136		);
137		$this->assertSame( $menu[0], $browse_sites_menu_item );
138
139		remove_user_from_blog( get_current_user_id(), $blog_id );
140		}
141
142		/**
143		* Tests add_new_site_link.
144		*
145		* @covers ::add_new_site_link
146		*/
147		public function test_add_new_site_link() {
148		global $menu;
149
150		static::$admin_menu->add_new_site_link();
151
152		$new_site_menu_item = array(
153		'Add new site',
154		'read',
155		'https://wordpress.com/start?ref=calypso-sidebar',
156		'Add new site',
157		'menu-top toplevel_page_https://wordpress.com/start?ref=calypso-sidebar',
158		'toplevel_page_https://wordpress.com/start?ref=calypso-sidebar',
159		'dashicons-plus-alt',
160		);
161		$this->assertSame( $menu[1002], $new_site_menu_item ); // 1001 is the separator position, 1002 is the link position
162		}
163
164		/**
165		* Tests add_site_card_menu
166		*
167		* @covers ::add_site_card_menu
168		*/
169		public function test_add_site_card_menu() {
170		global $menu;
171
172		if ( ! static::$is_wpcom ) {
173		$this->markTestSkipped( 'Only used on WP.com.' );
174		}
175
176		static::$admin_menu->add_site_card_menu();
177
178		$home_url = home_url();
179		$site_card_menu_item = array(
180		// phpcs:ignore Squiz.Strings.DoubleQuoteUsage.NotRequired
181		'
182		<div class="site__info">
183		<div class="site__title">' . get_option( 'blogname' ) . '</div>
184		<div class="site__domain">' . static::$domain . "</div>\n\t</div>",
185		'read',
186		$home_url,
187		'site-card',
188		'menu-top toplevel_page_' . $home_url,
189		'toplevel_page_' . $home_url,
190		'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',
191		);
192
193		$this->assertEquals( $menu[1], $site_card_menu_item );
194		}
195
196		/**
197		* Tests set_site_card_menu_class
198		*
199		* @covers ::set_site_card_menu_class
200		*/
201		public function test_set_site_card_menu_class() {
202		global $menu;
203
204		if ( ! static::$is_wpcom ) {
205		$this->markTestSkipped( 'Only used on WP.com.' );
206		}
207
208		static::$admin_menu->add_site_card_menu();
209
210		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
211		$this->assertNotContains( 'has-site-icon', $menu[1][4] );
212
213		// Atomic fallback site icon counts as no site icon.
214		add_filter( 'get_site_icon_url', array( $this, 'wpcomsh_site_icon_url' ) );
215		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
216		remove_filter( 'get_site_icon_url', array( $this, 'wpcomsh_site_icon_url' ) );
217		$this->assertNotContains( 'has-site-icon', $menu[1][4] );
218
219		// Custom site icon triggers CSS class.
220		add_filter( 'get_site_icon_url', array( $this, 'custom_site_icon_url' ) );
221		$menu = static::$admin_menu->set_site_card_menu_class( $menu );
222		remove_filter( 'get_site_icon_url', array( $this, 'custom_site_icon_url' ) );
223		$this->assertContains( 'has-site-icon', $menu[1][4] );
224		}
225
226		/**
227		* Shim wpcomsh fallback site icon.
228		*
229		* @return string
230		*/
231		public function wpcomsh_site_icon_url() {
232		return 'https://s0.wp.com/i/webclip.png';
233		}
234
235		/**
236		* Custom site icon.
237		*
238		* @return string
239		*/
240		public function custom_site_icon_url() {
241		return 'https://s0.wp.com/i/jetpack.png';
242		}
243
244		/**
245		* Tests add_upgrades_menu
246		*
247		* @covers ::add_upgrades_menu
248		*/
249	View Code Duplication	public function test_add_wpcom_upgrades_menu() {
250		global $submenu;
251
252		static::$admin_menu->add_upgrades_menu();
253
254		$domains_submenu_item = array(
255		'Domains',
256		'manage_options',
257		'https://wordpress.com/domains/manage/' . static::$domain,
258		'Domains',
259		);
260		$this->assertContains( $domains_submenu_item, $submenu[ 'https://wordpress.com/plans/' . static::$domain ] );
261		}
262
263		/**
264		* Tests add_users_menu
265		*
266		* @covers ::add_users_menu
267		*/
268		public function test_add_users_menu() {
269		global $menu;
270
271		static::$admin_menu->add_users_menu( true );
272
273		// Check that menu always links to Calypso.
274		$this->assertSame( $menu[70][2], 'https://wordpress.com/people/team/' . static::$domain );
275		}
276
277		/**
278		* Tests add_tools_menu
279		*
280		* @covers ::add_tools_menu
281		*/
282	View Code Duplication	public function test_add_tools_menu() {
283		global $submenu;
284
285		$slug = 'https://wordpress.com/marketing/tools/' . static::$domain;
286		static::$admin_menu->add_tools_menu( false, true );
287
288		// Check Export menu item always links to Calypso.
289		$export_submenu_item = array(
290		'Export',
291		'export',
292		'https://wordpress.com/export/' . static::$domain,
293		'Export',
294		);
295		$this->assertContains( $export_submenu_item, $submenu[ $slug ] );
296		}
297
298		/**
299		* Tests add_options_menu
300		*
301		* @covers ::add_options_menu
302		*/
303		public function test_add_options_menu() {
304		global $submenu;
305
306		$slug = 'https://wordpress.com/settings/general/' . static::$domain;
307		static::$admin_menu->add_options_menu( false );
308
309		$this->assertContains( 'Hosting Configuration', $submenu[ $slug ][6] );
310
311		$sharing_submenu_item = array(
312		'Sharing',
313		'publish_posts',
314		'https://wordpress.com/marketing/sharing-buttons/' . static::$domain,
315		'Sharing Settings',
316		);
317		$this->assertContains( $sharing_submenu_item, $submenu[ $slug ] );
318		}
319
320		/**
321		* Tests add_gutenberg_menus
322		*
323		* @covers ::add_gutenberg_menus
324		*/
325		public function test_add_gutenberg_menus() {
326		global $menu;
327		static::$admin_menu->add_gutenberg_menus( false );
328
329		// Gutenberg plugin menu should not be visible.
330		$this->assertArrayNotHasKey( 101, $menu );
331
332		// FSE is no longer where it was put by default.
333		$this->assertArrayNotHasKey( 100, $menu );
334		$this->assertArrayHasKey( 61, $menu );
335
336		$fse_link = 'https://wordpress.com/site-editor/' . static::$domain;
337		$fse_menu = array(
338		'Site Editor',
339		'edit_theme_options',
340		$fse_link,
341		'Site Editor',
342		'menu-top toplevel_page_' . $fse_link,
343		'toplevel_page_' . $fse_link,
344		'dashicons-layout',
345		);
346		$this->assertSame( $menu[61], $fse_menu );
347		}
348
349		/**
350		* Tests add_plugins_menu
351		*
352		* @covers ::add_plugins_menu
353		*/
354	View Code Duplication	public function test_add_plugins_menu() {
355		global $menu;
356
357		static::$admin_menu->add_plugins_menu( true );
358
359		// Check Plugins menu always links to Calypso.
360		$this->assertContains( 'https://wordpress.com/plugins/' . static::$domain, $menu[65] );
361		}
362		}
363

Automattic / jetpack

Push — kraftbj-patch-2 ( f1a4bb...ba1272 )

Test_WPcom_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