woocommerce / woocommerce-rest-api

unit-tests/Tests/Version4/Reports/Variations.php

Spacing +60 added lines, -60 removed lines

@@ -8,7 +8,7 @@ 
 
 namespace WooCommerce\RestApi\UnitTests\Tests\Version4\Reports;
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 use \WooCommerce\RestApi\UnitTests\AbstractReportsTest;
 use \WP_REST_Request;
@@ -39,7 +39,7 @@ 
 	public function test_register_routes() {
 		$routes = $this->server->get_routes();
 
-		$this->assertArrayHasKey( $this->endpoint, $routes );
+		$this->assertArrayHasKey($this->endpoint, $routes);
 	}
 
 	/**
@@ -50,33 +50,33 @@ 
 	public function test_get_reports() {
 		// Populate all of the data.
 		$variation = new \WC_Product_Variation();
-		$variation->set_name( 'Test Variation' );
-		$variation->set_regular_price( 25 );
-		$variation->set_attributes( array( 'color' => 'green' ) );
+		$variation->set_name('Test Variation');
+		$variation->set_regular_price(25);
+		$variation->set_attributes(array('color' => 'green'));
 		$variation->save();
 
-		$order = OrderHelper::create_order( 1, $variation );
-		$order->set_status( 'completed' );
-		$order->set_total( 100 ); // $25 x 4.
+		$order = OrderHelper::create_order(1, $variation);
+		$order->set_status('completed');
+		$order->set_total(100); // $25 x 4.
 		$order->save();
 
 		QueueHelper::run_all_pending();
 
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', $this->endpoint ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', $this->endpoint));
 		$reports  = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( 1, count( $reports ) );
+		$this->assertEquals(200, $response->get_status());
+		$this->assertEquals(1, count($reports));
 
-		$variation_report = reset( $reports );
+		$variation_report = reset($reports);
 
-		$this->assertEquals( $variation->get_id(), $variation_report['variation_id'] );
-		$this->assertEquals( 4, $variation_report['items_sold'] );
-		$this->assertEquals( 1, $variation_report['orders_count'] );
-		$this->assertArrayHasKey( '_links', $variation_report );
-		$this->assertArrayHasKey( 'extended_info', $variation_report );
-		$this->assertArrayHasKey( 'product', $variation_report['_links'] );
-		$this->assertArrayHasKey( 'variation', $variation_report['_links'] );
+		$this->assertEquals($variation->get_id(), $variation_report['variation_id']);
+		$this->assertEquals(4, $variation_report['items_sold']);
+		$this->assertEquals(1, $variation_report['orders_count']);
+		$this->assertArrayHasKey('_links', $variation_report);
+		$this->assertArrayHasKey('extended_info', $variation_report);
+		$this->assertArrayHasKey('product', $variation_report['_links']);
+		$this->assertArrayHasKey('variation', $variation_report['_links']);
 	}
 
 	/**
@@ -87,25 +87,25 @@ 
 	public function test_get_reports_variations_param() {
 		// Populate all of the data.
 		$variation = new \WC_Product_Variation();
-		$variation->set_name( 'Test Variation' );
-		$variation->set_regular_price( 25 );
-		$variation->set_attributes( array( 'color' => 'green' ) );
+		$variation->set_name('Test Variation');
+		$variation->set_regular_price(25);
+		$variation->set_attributes(array('color' => 'green'));
 		$variation->save();
 
 		$variation_2 = new \WC_Product_Variation();
-		$variation_2->set_name( 'Test Variation 2' );
-		$variation_2->set_regular_price( 100 );
-		$variation_2->set_attributes( array( 'color' => 'red' ) );
+		$variation_2->set_name('Test Variation 2');
+		$variation_2->set_regular_price(100);
+		$variation_2->set_attributes(array('color' => 'red'));
 		$variation_2->save();
 
-		$order = OrderHelper::create_order( 1, $variation );
-		$order->set_status( 'completed' );
-		$order->set_total( 100 ); // $25 x 4.
+		$order = OrderHelper::create_order(1, $variation);
+		$order->set_status('completed');
+		$order->set_total(100); // $25 x 4.
 		$order->save();
 
 		QueueHelper::run_all_pending();
 
-		$request = new WP_REST_Request( 'GET', $this->endpoint );
+		$request = new WP_REST_Request('GET', $this->endpoint);
 		$request->set_query_params(
 			array(
 				'product_includes' => $variation->get_parent_id(),
@@ -113,31 +113,31 @@ 
 				'variations'       => $variation->get_id() . ',' . $variation_2->get_id(),
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$reports  = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( 2, count( $reports ) );
+		$this->assertEquals(200, $response->get_status());
+		$this->assertEquals(2, count($reports));
 
-		$variation_report = reset( $reports );
+		$variation_report = reset($reports);
 
-		$this->assertEquals( $variation->get_id(), $variation_report['variation_id'] );
-		$this->assertEquals( 4, $variation_report['items_sold'] );
-		$this->assertEquals( 1, $variation_report['orders_count'] );
-		$this->assertArrayHasKey( '_links', $variation_report );
-		$this->assertArrayHasKey( 'extended_info', $variation_report );
-		$this->assertArrayHasKey( 'product', $variation_report['_links'] );
-		$this->assertArrayHasKey( 'variation', $variation_report['_links'] );
+		$this->assertEquals($variation->get_id(), $variation_report['variation_id']);
+		$this->assertEquals(4, $variation_report['items_sold']);
+		$this->assertEquals(1, $variation_report['orders_count']);
+		$this->assertArrayHasKey('_links', $variation_report);
+		$this->assertArrayHasKey('extended_info', $variation_report);
+		$this->assertArrayHasKey('product', $variation_report['_links']);
+		$this->assertArrayHasKey('variation', $variation_report['_links']);
 
-		$variation_report = next( $reports );
+		$variation_report = next($reports);
 
-		$this->assertEquals( $variation_2->get_id(), $variation_report['variation_id'] );
-		$this->assertEquals( 0, $variation_report['items_sold'] );
-		$this->assertEquals( 0, $variation_report['orders_count'] );
-		$this->assertArrayHasKey( '_links', $variation_report );
-		$this->assertArrayHasKey( 'extended_info', $variation_report );
-		$this->assertArrayHasKey( 'product', $variation_report['_links'] );
-		$this->assertArrayHasKey( 'variation', $variation_report['_links'] );
+		$this->assertEquals($variation_2->get_id(), $variation_report['variation_id']);
+		$this->assertEquals(0, $variation_report['items_sold']);
+		$this->assertEquals(0, $variation_report['orders_count']);
+		$this->assertArrayHasKey('_links', $variation_report);
+		$this->assertArrayHasKey('extended_info', $variation_report);
+		$this->assertArrayHasKey('product', $variation_report['_links']);
+		$this->assertArrayHasKey('variation', $variation_report['_links']);
 	}
 
 	/**
@@ -146,9 +146,9 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_reports_without_permission() {
-		wp_set_current_user( 0 );
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', $this->endpoint ) );
-		$this->assertEquals( 401, $response->get_status() );
+		wp_set_current_user(0);
+		$response = $this->server->dispatch(new WP_REST_Request('GET', $this->endpoint));
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -157,17 +157,17 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_reports_schema() {
-		$request    = new WP_REST_Request( 'OPTIONS', $this->endpoint );
-		$response   = $this->server->dispatch( $request );
+		$request    = new WP_REST_Request('OPTIONS', $this->endpoint);
+		$response   = $this->server->dispatch($request);
 		$data       = $response->get_data();
 		$properties = $data['schema']['properties'];
 
-		$this->assertEquals( 6, count( $properties ) );
-		$this->assertArrayHasKey( 'product_id', $properties );
-		$this->assertArrayHasKey( 'variation_id', $properties );
-		$this->assertArrayHasKey( 'items_sold', $properties );
-		$this->assertArrayHasKey( 'net_revenue', $properties );
-		$this->assertArrayHasKey( 'orders_count', $properties );
-		$this->assertArrayHasKey( 'extended_info', $properties );
+		$this->assertEquals(6, count($properties));
+		$this->assertArrayHasKey('product_id', $properties);
+		$this->assertArrayHasKey('variation_id', $properties);
+		$this->assertArrayHasKey('items_sold', $properties);
+		$this->assertArrayHasKey('net_revenue', $properties);
+		$this->assertArrayHasKey('orders_count', $properties);
+		$this->assertArrayHasKey('extended_info', $properties);
 	}
 }

unit-tests/Tests/Version4/Data.php

Spacing +37 added lines, -37 removed lines

@@ -7,7 +7,7 @@ 
 
 namespace WooCommerce\RestApi\UnitTests\Tests\Version4;
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 use \WooCommerce\RestApi\UnitTests\AbstractReportsTest;
 
@@ -27,13 +27,13 @@ 
 	 * Test that the list of data endpoints includes download-ips.
 	 */
 	public function test_get_items_contains_download_ips() {
-		$request  = new \WP_REST_Request( 'GET', $this->endpoint );
-		$response = $this->server->dispatch( $request );
+		$request  = new \WP_REST_Request('GET', $this->endpoint);
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( 4, count( $data ) );
-		$this->assertEquals( 'download-ips', $data[3]['slug'] );
+		$this->assertEquals(200, $response->get_status());
+		$this->assertEquals(4, count($data));
+		$this->assertEquals('download-ips', $data[3]['slug']);
 	}
 
 	/**
@@ -41,67 +41,67 @@ 
 	 */
 	public function test_download_ips() {
 		$prod_download = new \WC_Product_Download();
-		$prod_download->set_file( plugin_dir_url( __FILE__ ) . '/assets/images/help.png' );
-		$prod_download->set_id( 1 );
+		$prod_download->set_file(plugin_dir_url(__FILE__) . '/assets/images/help.png');
+		$prod_download->set_id(1);
 
 		$product = new \WC_Product_Simple();
-		$product->set_name( 'Test Product' );
-		$product->set_downloadable( 'yes' );
-		$product->set_downloads( array( $prod_download ) );
-		$product->set_regular_price( 25 );
+		$product->set_name('Test Product');
+		$product->set_downloadable('yes');
+		$product->set_downloads(array($prod_download));
+		$product->set_regular_price(25);
 		$product->save();
 
-		$order = \WooCommerce\RestApi\UnitTests\Helpers\OrderHelper::create_order( 1, $product );
-		$order->set_status( 'completed' );
-		$order->set_total( 100 );
+		$order = \WooCommerce\RestApi\UnitTests\Helpers\OrderHelper::create_order(1, $product);
+		$order->set_status('completed');
+		$order->set_total(100);
 		$order->save();
 
 		$download = new \WC_Customer_Download();
-		$download->set_user_id( $this->user );
-		$download->set_order_id( $order->get_id() );
-		$download->set_product_id( $product->get_id() );
-		$download->set_download_id( $prod_download->get_id() );
+		$download->set_user_id($this->user);
+		$download->set_order_id($order->get_id());
+		$download->set_product_id($product->get_id());
+		$download->set_download_id($prod_download->get_id());
 		$download->save();
 
 		$object = new \WC_Customer_Download_Log();
-		$object->set_permission_id( $download->get_id() );
-		$object->set_user_id( $this->user );
-		$object->set_user_ip_address( '1.2.3.4' );
+		$object->set_permission_id($download->get_id());
+		$object->set_user_id($this->user);
+		$object->set_user_ip_address('1.2.3.4');
 		$id = $object->save();
 
 		$object = new \WC_Customer_Download_Log();
-		$object->set_permission_id( $download->get_id() );
-		$object->set_user_id( $this->user );
-		$object->set_user_ip_address( '54.2.1.3' );
+		$object->set_permission_id($download->get_id());
+		$object->set_user_id($this->user);
+		$object->set_user_ip_address('54.2.1.3');
 		$id = $object->save();
 
 		// Save a second log for the same IP -- only one result for this IP should be returned.
 		$object = new \WC_Customer_Download_Log();
-		$object->set_permission_id( $download->get_id() );
-		$object->set_user_id( $this->user );
-		$object->set_user_ip_address( '54.2.1.3' );
+		$object->set_permission_id($download->get_id());
+		$object->set_user_id($this->user);
+		$object->set_user_ip_address('54.2.1.3');
 		$id = $object->save();
 
 		$object = new \WC_Customer_Download_Log();
-		$object->set_permission_id( $download->get_id() );
-		$object->set_user_id( $this->user );
-		$object->set_user_ip_address( '54.5.1.7' );
+		$object->set_permission_id($download->get_id());
+		$object->set_user_id($this->user);
+		$object->set_user_ip_address('54.5.1.7');
 		$id = $object->save();
 
-		$request = new \WP_REST_Request( 'GET', $this->endpoint . '/download-ips' );
+		$request = new \WP_REST_Request('GET', $this->endpoint . '/download-ips');
 		$request->set_query_params(
 			array(
 				'match' => '54',
 			)
 		);
 
-		$response  = $this->server->dispatch( $request );
+		$response  = $this->server->dispatch($request);
 		$addresses = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( 2, count( $addresses ) );
+		$this->assertEquals(200, $response->get_status());
+		$this->assertEquals(2, count($addresses));
 
-		$this->assertEquals( '54.2.1.3', $addresses[0]['user_ip_address'] );
-		$this->assertEquals( '54.5.1.7', $addresses[1]['user_ip_address'] );
+		$this->assertEquals('54.2.1.3', $addresses[0]['user_ip_address']);
+		$this->assertEquals('54.5.1.7', $addresses[1]['user_ip_address']);
 	}
 }

unit-tests/Tests/Version4/Settings.php

Spacing +207 added lines, -207 removed lines

@@ -8,7 +8,7 @@ 
 
 namespace WooCommerce\RestApi\UnitTests\Tests\Version4;
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 use \WP_REST_Request;
 use \WC_REST_Unit_Test_Case;
@@ -28,7 +28,7 @@ 
 	 *
 	 * @param object $factory Factory object.
 	 */
-	public static function wpSetUpBeforeClass( $factory ) {
+	public static function wpSetUpBeforeClass($factory) {
 		self::$user = $factory->user->create(
 			array(
 				'role' => 'administrator',
@@ -41,7 +41,7 @@ 
 	 */
 	public function setUp() {
 		parent::setUp();
-		wp_set_current_user( self::$user );
+		wp_set_current_user(self::$user);
 		SettingsHelper::register();
 		$this->zones = array();
 	}
@@ -53,9 +53,9 @@ 
 	 */
 	public function test_register_routes() {
 		$routes = $this->server->get_routes();
-		$this->assertArrayHasKey( '/wc/v4/settings', $routes );
-		$this->assertArrayHasKey( '/wc/v4/settings/(?P<group_id>[\w-]+)', $routes );
-		$this->assertArrayHasKey( '/wc/v4/settings/(?P<group_id>[\w-]+)/(?P<id>[\w-]+)', $routes );
+		$this->assertArrayHasKey('/wc/v4/settings', $routes);
+		$this->assertArrayHasKey('/wc/v4/settings/(?P<group_id>[\w-]+)', $routes);
+		$this->assertArrayHasKey('/wc/v4/settings/(?P<group_id>[\w-]+)/(?P<id>[\w-]+)', $routes);
 	}
 
 	/**
@@ -64,10 +64,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_groups() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings'));
 		$data     = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
+		$this->assertEquals(200, $response->get_status());
 
 		$this->assertContains(
 			array(
@@ -75,11 +75,11 @@ 
 				'label'       => 'Test extension',
 				'parent_id'   => '',
 				'description' => 'My awesome test settings.',
-				'sub_groups'  => array( 'sub-test' ),
+				'sub_groups'  => array('sub-test'),
 				'_links'      => array(
 					'options' => array(
 						array(
-							'href' => rest_url( '/wc/v4/settings/test' ),
+							'href' => rest_url('/wc/v4/settings/test'),
 						),
 					),
 				),
@@ -97,7 +97,7 @@ 
 				'_links'      => array(
 					'options' => array(
 						array(
-							'href' => rest_url( '/wc/v4/settings/sub-test' ),
+							'href' => rest_url('/wc/v4/settings/sub-test'),
 						),
 					),
 				),
@@ -112,10 +112,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_groups_without_permission() {
-		wp_set_current_user( 0 );
+		wp_set_current_user(0);
 
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings' ) );
-		$this->assertEquals( 401, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings'));
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -125,10 +125,10 @@ 
 	 * @covers WC_Rest_Settings_Controller::get_items
 	 */
 	public function test_get_groups_none_registered() {
-		remove_all_filters( 'woocommerce_settings_groups' );
+		remove_all_filters('woocommerce_settings_groups');
 
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings' ) );
-		$this->assertEquals( 500, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings'));
+		$this->assertEquals(500, $response->get_status());
 
 		SettingsHelper::register();
 	}
@@ -139,16 +139,16 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_group_schema() {
-		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v4/settings' );
-		$response   = $this->server->dispatch( $request );
+		$request    = new WP_REST_Request('OPTIONS', '/wc/v4/settings');
+		$response   = $this->server->dispatch($request);
 		$data       = $response->get_data();
 		$properties = $data['schema']['properties'];
-		$this->assertEquals( 5, count( $properties ) );
-		$this->assertArrayHasKey( 'id', $properties );
-		$this->assertArrayHasKey( 'parent_id', $properties );
-		$this->assertArrayHasKey( 'label', $properties );
-		$this->assertArrayHasKey( 'description', $properties );
-		$this->assertArrayHasKey( 'sub_groups', $properties );
+		$this->assertEquals(5, count($properties));
+		$this->assertArrayHasKey('id', $properties);
+		$this->assertArrayHasKey('parent_id', $properties);
+		$this->assertArrayHasKey('label', $properties);
+		$this->assertArrayHasKey('description', $properties);
+		$this->assertArrayHasKey('sub_groups', $properties);
 	}
 
 	/**
@@ -157,21 +157,21 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_setting_schema() {
-		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v4/settings/test/woocommerce_shop_page_display' );
-		$response   = $this->server->dispatch( $request );
+		$request    = new WP_REST_Request('OPTIONS', '/wc/v4/settings/test/woocommerce_shop_page_display');
+		$response   = $this->server->dispatch($request);
 		$data       = $response->get_data();
 		$properties = $data['schema']['properties'];
-		$this->assertEquals( 10, count( $properties ) );
-		$this->assertArrayHasKey( 'id', $properties );
-		$this->assertArrayHasKey( 'label', $properties );
-		$this->assertArrayHasKey( 'description', $properties );
-		$this->assertArrayHasKey( 'value', $properties );
-		$this->assertArrayHasKey( 'default', $properties );
-		$this->assertArrayHasKey( 'tip', $properties );
-		$this->assertArrayHasKey( 'placeholder', $properties );
-		$this->assertArrayHasKey( 'type', $properties );
-		$this->assertArrayHasKey( 'options', $properties );
-		$this->assertArrayHasKey( 'group_id', $properties );
+		$this->assertEquals(10, count($properties));
+		$this->assertArrayHasKey('id', $properties);
+		$this->assertArrayHasKey('label', $properties);
+		$this->assertArrayHasKey('description', $properties);
+		$this->assertArrayHasKey('value', $properties);
+		$this->assertArrayHasKey('default', $properties);
+		$this->assertArrayHasKey('tip', $properties);
+		$this->assertArrayHasKey('placeholder', $properties);
+		$this->assertArrayHasKey('type', $properties);
+		$this->assertArrayHasKey('options', $properties);
+		$this->assertArrayHasKey('group_id', $properties);
 	}
 
 	/**
@@ -181,23 +181,23 @@ 
 	 */
 	public function test_get_group() {
 		// test route callback receiving an empty group id
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/' ) );
-		$this->assertEquals( 404, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/'));
+		$this->assertEquals(404, $response->get_status());
 
 		// test getting a group that does not exist
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/not-real' ) );
-		$this->assertEquals( 404, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/not-real'));
+		$this->assertEquals(404, $response->get_status());
 
 		// test getting the 'invalid' group
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/invalid' ) );
-		$this->assertEquals( 404, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/invalid'));
+		$this->assertEquals(404, $response->get_status());
 
 		// test getting a valid group with settings attached to it
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/test' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/test'));
 		$data     = $response->get_data();
-		$this->assertEquals( 1, count( $data ) );
-		$this->assertEquals( 'woocommerce_shop_page_display', $data[0]['id'] );
-		$this->assertEmpty( $data[0]['value'] );
+		$this->assertEquals(1, count($data));
+		$this->assertEquals('woocommerce_shop_page_display', $data[0]['id']);
+		$this->assertEmpty($data[0]['value']);
 	}
 
 	/**
@@ -206,10 +206,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_group_without_permission() {
-		wp_set_current_user( 0 );
+		wp_set_current_user(0);
 
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/coupon-data' ) );
-		$this->assertEquals( 401, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/coupon-data'));
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -219,46 +219,46 @@ 
 	 */
 	public function test_update_setting() {
 		// test defaults first
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/test/woocommerce_shop_page_display' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/test/woocommerce_shop_page_display'));
 		$data     = $response->get_data();
-		$this->assertEquals( '', $data['value'] );
+		$this->assertEquals('', $data['value']);
 
 		// test updating shop display setting
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'test', 'woocommerce_shop_page_display' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'test', 'woocommerce_shop_page_display'));
 		$request->set_body_params(
 			array(
 				'value' => 'both',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'both', $data['value'] );
-		$this->assertEquals( 'both', get_option( 'woocommerce_shop_page_display' ) );
+		$this->assertEquals('both', $data['value']);
+		$this->assertEquals('both', get_option('woocommerce_shop_page_display'));
 
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'test', 'woocommerce_shop_page_display' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'test', 'woocommerce_shop_page_display'));
 		$request->set_body_params(
 			array(
 				'value' => 'subcategories',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'subcategories', $data['value'] );
-		$this->assertEquals( 'subcategories', get_option( 'woocommerce_shop_page_display' ) );
+		$this->assertEquals('subcategories', $data['value']);
+		$this->assertEquals('subcategories', get_option('woocommerce_shop_page_display'));
 
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'test', 'woocommerce_shop_page_display' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'test', 'woocommerce_shop_page_display'));
 		$request->set_body_params(
 			array(
 				'value' => '',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( '', $data['value'] );
-		$this->assertEquals( '', get_option( 'woocommerce_shop_page_display' ) );
+		$this->assertEquals('', $data['value']);
+		$this->assertEquals('', get_option('woocommerce_shop_page_display'));
 	}
 
 	/**
@@ -268,12 +268,12 @@ 
 	 */
 	public function test_update_settings() {
 		// test defaults first
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/test' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/test'));
 		$data     = $response->get_data();
-		$this->assertEquals( '', $data[0]['value'] );
+		$this->assertEquals('', $data[0]['value']);
 
 		// test setting both at once
-		$request = new WP_REST_Request( 'POST', '/wc/v4/settings/test/batch' );
+		$request = new WP_REST_Request('POST', '/wc/v4/settings/test/batch');
 		$request->set_body_params(
 			array(
 				'update' => array(
@@ -284,14 +284,14 @@ 
 				),
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'both', $data['update'][0]['value'] );
-		$this->assertEquals( 'both', get_option( 'woocommerce_shop_page_display' ) );
+		$this->assertEquals('both', $data['update'][0]['value']);
+		$this->assertEquals('both', get_option('woocommerce_shop_page_display'));
 
 		// test updating one, but making sure the other value stays the same
-		$request = new WP_REST_Request( 'POST', '/wc/v4/settings/test/batch' );
+		$request = new WP_REST_Request('POST', '/wc/v4/settings/test/batch');
 		$request->set_body_params(
 			array(
 				'update' => array(
@@ -302,10 +302,10 @@ 
 				),
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
-		$this->assertEquals( 'subcategories', $data['update'][0]['value'] );
-		$this->assertEquals( 'subcategories', get_option( 'woocommerce_shop_page_display' ) );
+		$this->assertEquals('subcategories', $data['update'][0]['value']);
+		$this->assertEquals('subcategories', get_option('woocommerce_shop_page_display'));
 	}
 
 	/**
@@ -315,26 +315,26 @@ 
 	 */
 	public function test_get_setting() {
 		// test getting an invalid setting from a group that does not exist
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/not-real/woocommerce_shop_page_display' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/not-real/woocommerce_shop_page_display'));
 		$data     = $response->get_data();
-		$this->assertEquals( 404, $response->get_status() );
+		$this->assertEquals(404, $response->get_status());
 
 		// test getting an invalid setting from a group that does exist
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/invalid/invalid' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/invalid/invalid'));
 		$data     = $response->get_data();
-		$this->assertEquals( 404, $response->get_status() );
+		$this->assertEquals(404, $response->get_status());
 
 		// test getting a valid setting
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/test/woocommerce_shop_page_display' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/test/woocommerce_shop_page_display'));
 		$data     = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
+		$this->assertEquals(200, $response->get_status());
 
-		$this->assertEquals( 'woocommerce_shop_page_display', $data['id'] );
-		$this->assertEquals( 'Shop page display', $data['label'] );
-		$this->assertEquals( '', $data['default'] );
-		$this->assertEquals( 'select', $data['type'] );
-		$this->assertEquals( '', $data['value'] );
+		$this->assertEquals('woocommerce_shop_page_display', $data['id']);
+		$this->assertEquals('Shop page display', $data['label']);
+		$this->assertEquals('', $data['default']);
+		$this->assertEquals('select', $data['type']);
+		$this->assertEquals('', $data['value']);
 	}
 
 	/**
@@ -343,10 +343,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_setting_without_permission() {
-		wp_set_current_user( 0 );
+		wp_set_current_user(0);
 
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/test/woocommerce_shop_page_display' ) );
-		$this->assertEquals( 401, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/test/woocommerce_shop_page_display'));
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -355,8 +355,8 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_setting_empty_setting_id() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/test/' ) );
-		$this->assertEquals( 404, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/test/'));
+		$this->assertEquals(404, $response->get_status());
 	}
 
 	/**
@@ -365,8 +365,8 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_setting_invalid_setting_id() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/test/invalid' ) );
-		$this->assertEquals( 404, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/test/invalid'));
+		$this->assertEquals(404, $response->get_status());
 	}
 
 	/**
@@ -376,21 +376,21 @@ 
 	 */
 	public function test_get_setting_invalid_setting_type() {
 		// $controller = $this->getMock( 'WC_Rest_Setting_Options_Controller', array( 'get_group_settings', 'is_setting_type_valid' ) );
-		$controller = $this->getMockBuilder( 'WC_Rest_Setting_Options_Controller' )->setMethods( array( 'get_group_settings', 'is_setting_type_valid' ) )->getMock();
+		$controller = $this->getMockBuilder('WC_Rest_Setting_Options_Controller')->setMethods(array('get_group_settings', 'is_setting_type_valid'))->getMock();
 
 		$controller
-			->expects( $this->any() )
-			->method( 'get_group_settings' )
-			->will( $this->returnValue( SettingsHelper::register_test_settings( array() ) ) );
+			->expects($this->any())
+			->method('get_group_settings')
+			->will($this->returnValue(SettingsHelper::register_test_settings(array())));
 
 		$controller
-			->expects( $this->any() )
-			->method( 'is_setting_type_valid' )
-			->will( $this->returnValue( false ) );
+			->expects($this->any())
+			->method('is_setting_type_valid')
+			->will($this->returnValue(false));
 
-		$result = $controller->get_setting( 'test', 'woocommerce_shop_page_display' );
+		$result = $controller->get_setting('test', 'woocommerce_shop_page_display');
 
-		$this->assertIsWPError( $result );
+		$this->assertIsWPError($result);
 	}
 
 	/**
@@ -399,16 +399,16 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_update_setting_without_permission() {
-		wp_set_current_user( 0 );
+		wp_set_current_user(0);
 
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'test', 'woocommerce_shop_page_display' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'test', 'woocommerce_shop_page_display'));
 		$request->set_body_params(
 			array(
 				'value' => 'subcategories',
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 401, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(401, $response->get_status());
 	}
 
 
@@ -418,9 +418,9 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_update_settings_without_permission() {
-		wp_set_current_user( 0 );
+		wp_set_current_user(0);
 
-		$request = new WP_REST_Request( 'POST', '/wc/v4/settings/test/batch' );
+		$request = new WP_REST_Request('POST', '/wc/v4/settings/test/batch');
 		$request->set_body_params(
 			array(
 				'update' => array(
@@ -431,8 +431,8 @@ 
 				),
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 401, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -442,14 +442,14 @@ 
 	 * @covers WC_Rest_Setting_Options_Controller::update_item
 	 */
 	public function test_update_setting_bad_setting_id() {
-		$request = new WP_REST_Request( 'PUT', '/wc/v4/settings/test/invalid' );
+		$request = new WP_REST_Request('PUT', '/wc/v4/settings/test/invalid');
 		$request->set_body_params(
 			array(
 				'value' => 'test',
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 404, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(404, $response->get_status());
 	}
 
 	/**
@@ -459,9 +459,9 @@ 
 	 */
 	public function test_classic_settings() {
 		// Make sure the group is properly registered
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/products' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/products'));
 		$data     = $response->get_data();
-		$this->assertTrue( is_array( $data ) );
+		$this->assertTrue(is_array($data));
 		$this->assertContains(
 			array(
 				'id'          => 'woocommerce_downloads_require_login',
@@ -474,12 +474,12 @@ 
 				'_links'      => array(
 					'self'       => array(
 						array(
-							'href' => rest_url( '/wc/v4/settings/products/woocommerce_downloads_require_login' ),
+							'href' => rest_url('/wc/v4/settings/products/woocommerce_downloads_require_login'),
 						),
 					),
 					'collection' => array(
 						array(
-							'href' => rest_url( '/wc/v4/settings/products' ),
+							'href' => rest_url('/wc/v4/settings/products'),
 						),
 					),
 				),
@@ -488,23 +488,23 @@ 
 		);
 
 		// test get single
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/products/woocommerce_dimension_unit' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/products/woocommerce_dimension_unit'));
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'cm', $data['default'] );
+		$this->assertEquals('cm', $data['default']);
 
 		// test update
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'products', 'woocommerce_dimension_unit' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'products', 'woocommerce_dimension_unit'));
 		$request->set_body_params(
 			array(
 				'value' => 'yd',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'yd', $data['value'] );
-		$this->assertEquals( 'yd', get_option( 'woocommerce_dimension_unit' ) );
+		$this->assertEquals('yd', $data['value']);
+		$this->assertEquals('yd', get_option('woocommerce_dimension_unit'));
 	}
 
 	/**
@@ -513,10 +513,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_email_settings() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/email_new_order' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/email_new_order'));
 		$settings = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
+		$this->assertEquals(200, $response->get_status());
 
 		$this->assertContains(
 			array(
@@ -530,12 +530,12 @@ 
 				'_links'      => array(
 					'self'       => array(
 						array(
-							'href' => rest_url( '/wc/v4/settings/email_new_order/recipient' ),
+							'href' => rest_url('/wc/v4/settings/email_new_order/recipient'),
 						),
 					),
 					'collection' => array(
 						array(
-							'href' => rest_url( '/wc/v4/settings/email_new_order' ),
+							'href' => rest_url('/wc/v4/settings/email_new_order'),
 						),
 					),
 				),
@@ -544,7 +544,7 @@ 
 		);
 
 		// test get single
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/email_new_order/subject' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/email_new_order/subject'));
 		$setting  = $response->get_data();
 
 		$this->assertEquals(
@@ -562,13 +562,13 @@ 
 		);
 
 		// test update
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'email_new_order', 'subject' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'email_new_order', 'subject'));
 		$request->set_body_params(
 			array(
 				'value' => 'This is my subject',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
 
 		$this->assertEquals(
@@ -586,29 +586,29 @@ 
 		);
 
 		// test updating another subject and making sure it works with a "similar" id
-		$request  = new WP_REST_Request( 'GET', sprintf( '/wc/v4/settings/%s/%s', 'email_customer_new_account', 'subject' ) );
-		$response = $this->server->dispatch( $request );
+		$request  = new WP_REST_Request('GET', sprintf('/wc/v4/settings/%s/%s', 'email_customer_new_account', 'subject'));
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
 
-		$this->assertEmpty( $setting['value'] );
+		$this->assertEmpty($setting['value']);
 
 		// test update
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'email_customer_new_account', 'subject' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'email_customer_new_account', 'subject'));
 		$request->set_body_params(
 			array(
 				'value' => 'This is my new subject',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
 
-		$this->assertEquals( 'This is my new subject', $setting['value'] );
+		$this->assertEquals('This is my new subject', $setting['value']);
 
 		// make sure the other is what we left it
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/email_new_order/subject' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/email_new_order/subject'));
 		$setting  = $response->get_data();
 
-		$this->assertEquals( 'This is my subject', $setting['value'] );
+		$this->assertEquals('This is my subject', $setting['value']);
 	}
 
 	/**
@@ -618,34 +618,34 @@ 
 	 */
 	public function test_validation_checkbox() {
 		// test bogus value
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'email_cancelled_order', 'enabled' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'email_cancelled_order', 'enabled'));
 		$request->set_body_params(
 			array(
 				'value' => 'not_yes_or_no',
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 400, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(400, $response->get_status());
 
 		// test yes
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'email_cancelled_order', 'enabled' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'email_cancelled_order', 'enabled'));
 		$request->set_body_params(
 			array(
 				'value' => 'yes',
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 200, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(200, $response->get_status());
 
 		// test no
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'email_cancelled_order', 'enabled' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'email_cancelled_order', 'enabled'));
 		$request->set_body_params(
 			array(
 				'value' => 'no',
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 200, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(200, $response->get_status());
 	}
 
 	/**
@@ -655,24 +655,24 @@ 
 	 */
 	public function test_validation_radio() {
 		// not a valid option
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'shipping', 'woocommerce_ship_to_destination' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'shipping', 'woocommerce_ship_to_destination'));
 		$request->set_body_params(
 			array(
 				'value' => 'billing2',
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 400, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(400, $response->get_status());
 
 		// valid
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'shipping', 'woocommerce_ship_to_destination' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'shipping', 'woocommerce_ship_to_destination'));
 		$request->set_body_params(
 			array(
 				'value' => 'billing',
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 200, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(200, $response->get_status());
 	}
 
 	/**
@@ -681,19 +681,19 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_validation_multiselect() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', sprintf( '/wc/v4/settings/%s/%s', 'general', 'woocommerce_specific_allowed_countries' ) ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', sprintf('/wc/v4/settings/%s/%s', 'general', 'woocommerce_specific_allowed_countries')));
 		$setting  = $response->get_data();
-		$this->assertEmpty( $setting['value'] );
+		$this->assertEmpty($setting['value']);
 
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'general', 'woocommerce_specific_allowed_countries' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'general', 'woocommerce_specific_allowed_countries'));
 		$request->set_body_params(
 			array(
-				'value' => array( 'AX', 'DZ', 'MMM' ),
+				'value' => array('AX', 'DZ', 'MMM'),
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( array( 'AX', 'DZ' ), $setting['value'] );
+		$this->assertEquals(array('AX', 'DZ'), $setting['value']);
 	}
 
 	/**
@@ -702,30 +702,30 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_validation_select() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', sprintf( '/wc/v4/settings/%s/%s', 'products', 'woocommerce_weight_unit' ) ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', sprintf('/wc/v4/settings/%s/%s', 'products', 'woocommerce_weight_unit')));
 		$setting  = $response->get_data();
-		$this->assertEquals( 'kg', $setting['value'] );
+		$this->assertEquals('kg', $setting['value']);
 
 		// invalid
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'products', 'woocommerce_weight_unit' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'products', 'woocommerce_weight_unit'));
 		$request->set_body_params(
 			array(
 				'value' => 'pounds', // invalid, should be lbs
 			)
 		);
-		$response = $this->server->dispatch( $request );
-		$this->assertEquals( 400, $response->get_status() );
+		$response = $this->server->dispatch($request);
+		$this->assertEquals(400, $response->get_status());
 
 		// valid
-		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v4/settings/%s/%s', 'products', 'woocommerce_weight_unit' ) );
+		$request = new WP_REST_Request('PUT', sprintf('/wc/v4/settings/%s/%s', 'products', 'woocommerce_weight_unit'));
 		$request->set_body_params(
 			array(
 				'value' => 'lbs', // invalid, should be lbs
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( 'lbs', $setting['value'] );
+		$this->assertEquals('lbs', $setting['value']);
 	}
 
 	/**
@@ -736,12 +736,12 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_woocommerce_default_country() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/general/woocommerce_default_country' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/general/woocommerce_default_country'));
 		$setting  = $response->get_data();
 
-		$this->assertEquals( 'select', $setting['type'] );
-		$this->assertArrayHasKey( 'GB', $setting['options'] );
-		$this->assertArrayHasKey( 'US:OR', $setting['options'] );
+		$this->assertEquals('select', $setting['type']);
+		$this->assertArrayHasKey('GB', $setting['options']);
+		$this->assertArrayHasKey('US:OR', $setting['options']);
 	}
 
 	/**
@@ -750,33 +750,33 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_woocommerce_store_address() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/general/woocommerce_store_address' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/general/woocommerce_store_address'));
 		$setting  = $response->get_data();
-		$this->assertEquals( 'text', $setting['type'] );
+		$this->assertEquals('text', $setting['type']);
 
 		// Repalce the old value with something uniquely new
 		$old_value = $setting['value'];
-		$new_value = $old_value . ' ' . rand( 1000, 9999 );
-		$request   = new WP_REST_Request( 'PUT', '/wc/v4/settings/general/woocommerce_store_address' );
+		$new_value = $old_value . ' ' . rand(1000, 9999);
+		$request   = new WP_REST_Request('PUT', '/wc/v4/settings/general/woocommerce_store_address');
 		$request->set_body_params(
 			array(
 				'value' => $new_value,
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( $new_value, $setting['value'] );
+		$this->assertEquals($new_value, $setting['value']);
 
 		// Put the original value back
-		$request = new WP_REST_Request( 'PUT', '/wc/v4/settings/general/woocommerce_store_address' );
+		$request = new WP_REST_Request('PUT', '/wc/v4/settings/general/woocommerce_store_address');
 		$request->set_body_params(
 			array(
 				'value' => $old_value,
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( $old_value, $setting['value'] );
+		$this->assertEquals($old_value, $setting['value']);
 	}
 
 	/**
@@ -785,33 +785,33 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_woocommerce_store_address_2() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/general/woocommerce_store_address_2' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/general/woocommerce_store_address_2'));
 		$setting  = $response->get_data();
-		$this->assertEquals( 'text', $setting['type'] );
+		$this->assertEquals('text', $setting['type']);
 
 		// Repalce the old value with something uniquely new
 		$old_value = $setting['value'];
-		$new_value = $old_value . ' ' . rand( 1000, 9999 );
-		$request   = new WP_REST_Request( 'PUT', '/wc/v4/settings/general/woocommerce_store_address_2' );
+		$new_value = $old_value . ' ' . rand(1000, 9999);
+		$request   = new WP_REST_Request('PUT', '/wc/v4/settings/general/woocommerce_store_address_2');
 		$request->set_body_params(
 			array(
 				'value' => $new_value,
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( $new_value, $setting['value'] );
+		$this->assertEquals($new_value, $setting['value']);
 
 		// Put the original value back
-		$request = new WP_REST_Request( 'PUT', '/wc/v4/settings/general/woocommerce_store_address_2' );
+		$request = new WP_REST_Request('PUT', '/wc/v4/settings/general/woocommerce_store_address_2');
 		$request->set_body_params(
 			array(
 				'value' => $old_value,
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( $old_value, $setting['value'] );
+		$this->assertEquals($old_value, $setting['value']);
 	}
 
 	/**
@@ -820,33 +820,33 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_woocommerce_store_city() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/general/woocommerce_store_city' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/general/woocommerce_store_city'));
 		$setting  = $response->get_data();
-		$this->assertEquals( 'text', $setting['type'] );
+		$this->assertEquals('text', $setting['type']);
 
 		// Repalce the old value with something uniquely new
 		$old_value = $setting['value'];
-		$new_value = $old_value . ' ' . rand( 1000, 9999 );
-		$request   = new WP_REST_Request( 'PUT', '/wc/v4/settings/general/woocommerce_store_city' );
+		$new_value = $old_value . ' ' . rand(1000, 9999);
+		$request   = new WP_REST_Request('PUT', '/wc/v4/settings/general/woocommerce_store_city');
 		$request->set_body_params(
 			array(
 				'value' => $new_value,
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( $new_value, $setting['value'] );
+		$this->assertEquals($new_value, $setting['value']);
 
 		// Put the original value back
-		$request = new WP_REST_Request( 'PUT', '/wc/v4/settings/general/woocommerce_store_city' );
+		$request = new WP_REST_Request('PUT', '/wc/v4/settings/general/woocommerce_store_city');
 		$request->set_body_params(
 			array(
 				'value' => $old_value,
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( $old_value, $setting['value'] );
+		$this->assertEquals($old_value, $setting['value']);
 	}
 
 	/**
@@ -855,32 +855,32 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_woocommerce_store_postcode() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/settings/general/woocommerce_store_postcode' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/settings/general/woocommerce_store_postcode'));
 		$setting  = $response->get_data();
-		$this->assertEquals( 'text', $setting['type'] );
+		$this->assertEquals('text', $setting['type']);
 
 		// Repalce the old value with something uniquely new
 		$old_value = $setting['value'];
-		$new_value = $old_value . ' ' . rand( 1000, 9999 );
-		$request   = new WP_REST_Request( 'PUT', '/wc/v4/settings/general/woocommerce_store_postcode' );
+		$new_value = $old_value . ' ' . rand(1000, 9999);
+		$request   = new WP_REST_Request('PUT', '/wc/v4/settings/general/woocommerce_store_postcode');
 		$request->set_body_params(
 			array(
 				'value' => $new_value,
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( $new_value, $setting['value'] );
+		$this->assertEquals($new_value, $setting['value']);
 
 		// Put the original value back
-		$request = new WP_REST_Request( 'PUT', '/wc/v4/settings/general/woocommerce_store_postcode' );
+		$request = new WP_REST_Request('PUT', '/wc/v4/settings/general/woocommerce_store_postcode');
 		$request->set_body_params(
 			array(
 				'value' => $old_value,
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$setting  = $response->get_data();
-		$this->assertEquals( $old_value, $setting['value'] );
+		$this->assertEquals($old_value, $setting['value']);
 	}
 }

unit-tests/Tests/Version4/ShippingMethods.php

Spacing +25 added lines, -25 removed lines

@@ -8,7 +8,7 @@ 
 
 namespace WooCommerce\RestApi\UnitTests\Tests\Version4;
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 use \WP_REST_Request;
 use \WC_REST_Unit_Test_Case;
@@ -27,7 +27,7 @@ 
 	 *
 	 * @param object $factory Factory object.
 	 */
-	public static function wpSetUpBeforeClass( $factory ) {
+	public static function wpSetUpBeforeClass($factory) {
 		self::$user = $factory->user->create(
 			array(
 				'role' => 'administrator',
@@ -40,7 +40,7 @@ 
 	 */
 	public function setUp() {
 		parent::setUp();
-		wp_set_current_user( self::$user );
+		wp_set_current_user(self::$user);
 		$this->zones = array();
 	}
 
@@ -51,8 +51,8 @@ 
 	 */
 	public function test_register_routes() {
 		$routes = $this->server->get_routes();
-		$this->assertArrayHasKey( '/wc/v4/shipping_methods', $routes );
-		$this->assertArrayHasKey( '/wc/v4/shipping_methods/(?P<id>[\w-]+)', $routes );
+		$this->assertArrayHasKey('/wc/v4/shipping_methods', $routes);
+		$this->assertArrayHasKey('/wc/v4/shipping_methods/(?P<id>[\w-]+)', $routes);
 	}
 
 	/**
@@ -61,10 +61,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_shipping_methods() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/shipping_methods' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/shipping_methods'));
 		$methods  = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
+		$this->assertEquals(200, $response->get_status());
 		$this->assertContains(
 			array(
 				'id'          => 'free_shipping',
@@ -73,12 +73,12 @@ 
 				'_links'      => array(
 					'self'       => array(
 						array(
-							'href' => rest_url( '/wc/v4/shipping_methods/free_shipping' ),
+							'href' => rest_url('/wc/v4/shipping_methods/free_shipping'),
 						),
 					),
 					'collection' => array(
 						array(
-							'href' => rest_url( '/wc/v4/shipping_methods' ),
+							'href' => rest_url('/wc/v4/shipping_methods'),
 						),
 					),
 				),
@@ -93,9 +93,9 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_shipping_methods_without_permission() {
-		wp_set_current_user( 0 );
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/shipping_methods' ) );
-		$this->assertEquals( 401, $response->get_status() );
+		wp_set_current_user(0);
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/shipping_methods'));
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -104,10 +104,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_shipping_method() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/shipping_methods/local_pickup' ) );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/shipping_methods/local_pickup'));
 		$method   = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
+		$this->assertEquals(200, $response->get_status());
 		$this->assertEquals(
 			array(
 				'id'          => 'local_pickup',
@@ -124,10 +124,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_shipping_method_without_permission() {
-		wp_set_current_user( 0 );
+		wp_set_current_user(0);
 
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/shipping_methods/local_pickup' ) );
-		$this->assertEquals( 401, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/shipping_methods/local_pickup'));
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -136,8 +136,8 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_get_shipping_method_invalid_id() {
-		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v4/shipping_methods/fake_method' ) );
-		$this->assertEquals( 404, $response->get_status() );
+		$response = $this->server->dispatch(new WP_REST_Request('GET', '/wc/v4/shipping_methods/fake_method'));
+		$this->assertEquals(404, $response->get_status());
 	}
 
 	/**
@@ -146,14 +146,14 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_shipping_method_schema() {
-		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v4/shipping_methods' );
-		$response   = $this->server->dispatch( $request );
+		$request    = new WP_REST_Request('OPTIONS', '/wc/v4/shipping_methods');
+		$response   = $this->server->dispatch($request);
 		$data       = $response->get_data();
 		$properties = $data['schema']['properties'];
 
-		$this->assertEquals( 3, count( $properties ) );
-		$this->assertArrayHasKey( 'id', $properties );
-		$this->assertArrayHasKey( 'title', $properties );
-		$this->assertArrayHasKey( 'description', $properties );
+		$this->assertEquals(3, count($properties));
+		$this->assertArrayHasKey('id', $properties);
+		$this->assertArrayHasKey('title', $properties);
+		$this->assertArrayHasKey('description', $properties);
 	}
 }

unit-tests/Tests/Version4/ProductReviews.php

Spacing +86 added lines, -86 removed lines

@@ -7,7 +7,7 @@ 
 
 namespace WooCommerce\RestApi\UnitTests\Tests\Version4;
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 use \WooCommerce\RestApi\UnitTests\AbstractRestApiTest;
 use \WooCommerce\RestApi\UnitTests\Helpers\ProductHelper;
@@ -35,17 +35,17 @@ 
 	 * @var array Keys are property names, values are supported context.
 	 */
 	protected $properties = [
-		'id'                   => array( 'view', 'edit' ),
-		'date_created'         => array( 'view', 'edit' ),
-		'date_created_gmt'     => array( 'view', 'edit' ),
-		'product_id'           => array( 'view', 'edit' ),
-		'status'               => array( 'view', 'edit' ),
-		'reviewer'             => array( 'view', 'edit' ),
-		'reviewer_email'       => array( 'view', 'edit' ),
-		'review'               => array( 'view', 'edit' ),
-		'rating'               => array( 'view', 'edit' ),
-		'verified'             => array( 'view', 'edit' ),
-		'reviewer_avatar_urls' => array( 'view', 'edit' ),
+		'id'                   => array('view', 'edit'),
+		'date_created'         => array('view', 'edit'),
+		'date_created_gmt'     => array('view', 'edit'),
+		'product_id'           => array('view', 'edit'),
+		'status'               => array('view', 'edit'),
+		'reviewer'             => array('view', 'edit'),
+		'reviewer_email'       => array('view', 'edit'),
+		'review'               => array('view', 'edit'),
+		'rating'               => array('view', 'edit'),
+		'verified'             => array('view', 'edit'),
+		'reviewer_avatar_urls' => array('view', 'edit'),
 	];
 
 	/**
@@ -61,8 +61,8 @@ 
 			'rating'         => '5',
 			'product_id'     => $product->get_id(),
 		];
-		$response = $this->do_request( '/wc/v4/products/reviews', 'POST', $data );
-		$this->assertExpectedResponse( $response, 201, $data );
+		$response = $this->do_request('/wc/v4/products/reviews', 'POST', $data);
+		$this->assertExpectedResponse($response, 201, $data);
 		$this->assertEquals(
 			array(
 				'id'                   => $response->data['id'],
@@ -86,20 +86,20 @@ 
 	 */
 	public function test_read() {
 		$product = ProductHelper::create_simple_product();
-		for ( $i = 0; $i < 10; $i++ ) {
-			$review_id = ProductHelper::create_product_review( $product->get_id() );
+		for ($i = 0; $i < 10; $i++) {
+			$review_id = ProductHelper::create_product_review($product->get_id());
 		}
 
 		// Invalid.
-		$response = $this->do_request( '/wc/v4/products/0/reviews' );
-		$this->assertExpectedResponse( $response, 404 );
+		$response = $this->do_request('/wc/v4/products/0/reviews');
+		$this->assertExpectedResponse($response, 404);
 
 		// Collections.
-		$response        = $this->do_request( '/wc/v4/products/reviews' );
+		$response        = $this->do_request('/wc/v4/products/reviews');
 		$product_reviews = $response->data;
 
-		$this->assertExpectedResponse( $response, 200 );
-		$this->assertEquals( 10, count( $product_reviews ) );
+		$this->assertExpectedResponse($response, 200);
+		$this->assertEquals(10, count($product_reviews));
 		$this->assertContains(
 			array(
 				'id'                   => $review_id,
@@ -116,18 +116,18 @@ 
 				'_links'               => array(
 					'self'       => array(
 						array(
-							'href' => rest_url( '/wc/v4/products/reviews/' . $review_id ),
+							'href' => rest_url('/wc/v4/products/reviews/' . $review_id),
 						),
 					),
 					'collection' => array(
 						array(
-							'href' => rest_url( '/wc/v4/products/reviews' ),
+							'href' => rest_url('/wc/v4/products/reviews'),
 						),
 					),
 					'up'         => array(
 						array(
 							'embeddable' => true,
-							'href'       => rest_url( '/wc/v4/products/' . $product->get_id() ),
+							'href'       => rest_url('/wc/v4/products/' . $product->get_id()),
 						),
 					),
 				),
@@ -142,27 +142,27 @@ 
 	 */
 	public function test_update() {
 		$product           = ProductHelper::create_simple_product();
-		$product_review_id = ProductHelper::create_product_review( $product->get_id() );
+		$product_review_id = ProductHelper::create_product_review($product->get_id());
 
-		$response          = $this->do_request( '/wc/v4/products/reviews/' . $product_review_id );
-		$this->assertEquals( 200, $response->status );
-		$this->assertEquals( "<p>Review content here</p>\n", $response->data['review'] );
-		$this->assertEquals( 'admin', $response->data['reviewer'] );
-		$this->assertEquals( 'woo@woo.local', $response->data['reviewer_email'] );
-		$this->assertEquals( 0, $response->data['rating'] );
+		$response          = $this->do_request('/wc/v4/products/reviews/' . $product_review_id);
+		$this->assertEquals(200, $response->status);
+		$this->assertEquals("<p>Review content here</p>\n", $response->data['review']);
+		$this->assertEquals('admin', $response->data['reviewer']);
+		$this->assertEquals('woo@woo.local', $response->data['reviewer_email']);
+		$this->assertEquals(0, $response->data['rating']);
 
-		$data     = [
+		$data = [
 			'review'         => 'Hello world - updated.',
 			'reviewer'       => 'Justin',
 			'reviewer_email' => 'woo2@woo.local',
 			'rating'         => 3,
 		];
-		$response = $this->do_request( '/wc/v4/products/reviews/' . $product_review_id, 'PUT', $data );
+		$response = $this->do_request('/wc/v4/products/reviews/' . $product_review_id, 'PUT', $data);
 
-		$this->assertExpectedResponse( $response, 200, $data );
+		$this->assertExpectedResponse($response, 200, $data);
 
-		foreach ( $this->get_properties( 'view' ) as $property ) {
-			$this->assertArrayHasKey( $property, $response->data );
+		foreach ($this->get_properties('view') as $property) {
+			$this->assertArrayHasKey($property, $response->data);
 		}
 	}
 
@@ -172,23 +172,23 @@ 
 	 */
 	public function test_delete() {
 		// Invalid.
-		$result = $this->do_request( '/wc/v4/products/reviews/0', 'DELETE', [ 'force' => true ] );
-		$this->assertEquals( 404, $result->status );
+		$result = $this->do_request('/wc/v4/products/reviews/0', 'DELETE', ['force' => true]);
+		$this->assertEquals(404, $result->status);
 
 		// Valid.
 		$product           = ProductHelper::create_simple_product();
-		$product_review_id = ProductHelper::create_product_review( $product->get_id() );
-		$result            = $this->do_request( '/wc/v4/products/reviews/' . $product_review_id, 'DELETE', [ 'force' => true ] );
-		$this->assertEquals( 200, $result->status );
+		$product_review_id = ProductHelper::create_product_review($product->get_id());
+		$result            = $this->do_request('/wc/v4/products/reviews/' . $product_review_id, 'DELETE', ['force' => true]);
+		$this->assertEquals(200, $result->status);
 	}
 
 	/**
 	 * Test get/read using this method.
 	 */
 	public function test_guest_read() {
-		wp_set_current_user( 0 );
-		$result = $this->do_request( '/wc/v4/products/reviews' );
-		$this->assertEquals( 401, $result->status );
+		wp_set_current_user(0);
+		$result = $this->do_request('/wc/v4/products/reviews');
+		$this->assertEquals(401, $result->status);
 	}
 
 	/**
@@ -198,12 +198,12 @@ 
 	 */
 	public function test_get_product_review() {
 		$product           = ProductHelper::create_simple_product();
-		$product_review_id = ProductHelper::create_product_review( $product->get_id() );
+		$product_review_id = ProductHelper::create_product_review($product->get_id());
 
-		$response = $this->server->dispatch( new \WP_REST_Request( 'GET', '/wc/v4/products/reviews/' . $product_review_id ) );
+		$response = $this->server->dispatch(new \WP_REST_Request('GET', '/wc/v4/products/reviews/' . $product_review_id));
 		$data     = $response->get_data();
 
-		$this->assertEquals( 200, $response->get_status() );
+		$this->assertEquals(200, $response->get_status());
 		$this->assertEquals(
 			array(
 				'id'                   => $product_review_id,
@@ -229,8 +229,8 @@ 
 	 */
 	public function test_get_product_review_invalid_id() {
 		$product  = ProductHelper::create_simple_product();
-		$response = $this->server->dispatch( new \WP_REST_Request( 'GET', '/wc/v4/products/reviews/0' ) );
-		$this->assertEquals( 404, $response->get_status() );
+		$response = $this->server->dispatch(new \WP_REST_Request('GET', '/wc/v4/products/reviews/0'));
+		$this->assertEquals(404, $response->get_status());
 	}
 
 	/**
@@ -242,43 +242,43 @@ 
 		$product = ProductHelper::create_simple_product();
 
 		// missing review
-		$request = new \WP_REST_Request( 'POST', '/wc/v4/products/reviews' );
+		$request = new \WP_REST_Request('POST', '/wc/v4/products/reviews');
 		$request->set_body_params(
 			array(
 				'reviewer'       => 'Admin',
 				'reviewer_email' => 'woo@woo.local',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 400, $response->get_status() );
+		$this->assertEquals(400, $response->get_status());
 
 		// Missing reviewer.
-		$request = new \WP_REST_Request( 'POST', '/wc/v4/products/reviews' );
+		$request = new \WP_REST_Request('POST', '/wc/v4/products/reviews');
 		$request->set_body_params(
 			array(
 				'review'         => 'Hello world.',
 				'reviewer_email' => 'woo@woo.local',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 400, $response->get_status() );
+		$this->assertEquals(400, $response->get_status());
 
 		// missing reviewer_email
-		$request = new \WP_REST_Request( 'POST', '/wc/v4/products/reviews' );
+		$request = new \WP_REST_Request('POST', '/wc/v4/products/reviews');
 		$request->set_body_params(
 			array(
 				'review'   => 'Hello world.',
 				'reviewer' => 'Admin',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 400, $response->get_status() );
+		$this->assertEquals(400, $response->get_status());
 	}
 
 	/**
@@ -287,11 +287,11 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_update_product_review_without_permission() {
-		wp_set_current_user( 0 );
+		wp_set_current_user(0);
 		$product           = ProductHelper::create_simple_product();
-		$product_review_id = ProductHelper::create_product_review( $product->get_id() );
+		$product_review_id = ProductHelper::create_product_review($product->get_id());
 
-		$request = new \WP_REST_Request( 'PUT', '/wc/v4/products/reviews/' . $product_review_id );
+		$request = new \WP_REST_Request('PUT', '/wc/v4/products/reviews/' . $product_review_id);
 		$request->set_body_params(
 			array(
 				'review'         => 'Hello world.',
@@ -299,10 +299,10 @@ 
 				'reviewer_email' => 'woo@woo.dev',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 401, $response->get_status() );
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -311,10 +311,10 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_update_product_review_invalid_id() {
-		wp_set_current_user( $this->user );
+		wp_set_current_user($this->user);
 		$product = ProductHelper::create_simple_product();
 
-		$request = new \WP_REST_Request( 'PUT', '/wc/v4/products/reviews/0' );
+		$request = new \WP_REST_Request('PUT', '/wc/v4/products/reviews/0');
 		$request->set_body_params(
 			array(
 				'review'         => 'Hello world.',
@@ -322,10 +322,10 @@ 
 				'reviewer_email' => 'woo@woo.dev',
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 404, $response->get_status() );
+		$this->assertEquals(404, $response->get_status());
 	}
 
 	/**
@@ -334,14 +334,14 @@ 
 	 * @since 3.5.0
 	 */
 	public function test_delete_product_without_permission() {
-		wp_set_current_user( 0 );
+		wp_set_current_user(0);
 		$product           = ProductHelper::create_simple_product();
-		$product_review_id = ProductHelper::create_product_review( $product->get_id() );
+		$product_review_id = ProductHelper::create_product_review($product->get_id());
 
-		$request  = new \WP_REST_Request( 'DELETE', '/wc/v4/products/reviews/' . $product_review_id );
-		$response = $this->server->dispatch( $request );
+		$request  = new \WP_REST_Request('DELETE', '/wc/v4/products/reviews/' . $product_review_id);
+		$response = $this->server->dispatch($request);
 
-		$this->assertEquals( 401, $response->get_status() );
+		$this->assertEquals(401, $response->get_status());
 	}
 
 	/**
@@ -352,12 +352,12 @@ 
 	public function test_product_reviews_batch() {
 		$product = ProductHelper::create_simple_product();
 
-		$review_1_id = ProductHelper::create_product_review( $product->get_id() );
-		$review_2_id = ProductHelper::create_product_review( $product->get_id() );
-		$review_3_id = ProductHelper::create_product_review( $product->get_id() );
-		$review_4_id = ProductHelper::create_product_review( $product->get_id() );
+		$review_1_id = ProductHelper::create_product_review($product->get_id());
+		$review_2_id = ProductHelper::create_product_review($product->get_id());
+		$review_3_id = ProductHelper::create_product_review($product->get_id());
+		$review_4_id = ProductHelper::create_product_review($product->get_id());
 
-		$request = new \WP_REST_Request( 'POST', '/wc/v4/products/reviews/batch' );
+		$request = new \WP_REST_Request('POST', '/wc/v4/products/reviews/batch');
 		$request->set_body_params(
 			array(
 				'update' => array(
@@ -380,21 +380,21 @@ 
 				),
 			)
 		);
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'Updated review.', $data['update'][0]['review'] );
-		$this->assertEquals( 'New review.', $data['create'][0]['review'] );
-		$this->assertEquals( $review_2_id, $data['delete'][0]['previous']['id'] );
-		$this->assertEquals( $review_3_id, $data['delete'][1]['previous']['id'] );
+		$this->assertEquals('Updated review.', $data['update'][0]['review']);
+		$this->assertEquals('New review.', $data['create'][0]['review']);
+		$this->assertEquals($review_2_id, $data['delete'][0]['previous']['id']);
+		$this->assertEquals($review_3_id, $data['delete'][1]['previous']['id']);
 
-		$request = new \WP_REST_Request( 'GET', '/wc/v4/products/reviews' );
-		$request->set_param( 'product', $product->get_id() );
+		$request = new \WP_REST_Request('GET', '/wc/v4/products/reviews');
+		$request->set_param('product', $product->get_id());
 
-		$response = $this->server->dispatch( $request );
+		$response = $this->server->dispatch($request);
 		$data     = $response->get_data();
 
-		$this->assertEquals( 3, count( $data ) );
+		$this->assertEquals(3, count($data));
 	}
 }
 

unit-tests/AbstractReportsTest.php

Spacing +9 added lines, -9 removed lines

@@ -5,7 +5,7 @@ 
 
 namespace WooCommerce\RestApi\UnitTests;
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 use \WooCommerce\RestApi\UnitTests\Bootstrap;
 use \WC_REST_Unit_Test_Case;
@@ -27,7 +27,7 @@ 
 	 *
 	 * @param object $factory Factory object.
 	 */
-	public static function wpSetUpBeforeClass( $factory ) {
+	public static function wpSetUpBeforeClass($factory) {
 		self::$user = $factory->user->create(
 			array(
 				'role' => 'administrator',
@@ -39,19 +39,19 @@ 
 	 * Setup test reports categories data.
 	 */
 	public function setUp() {
-		if ( ! class_exists( '\WC_Admin_Reports_Sync' ) ) {
-			$this->markTestSkipped( 'Skipping reports tests - WC_Admin_Reports_Sync class not found.' );
+		if ( ! class_exists('\WC_Admin_Reports_Sync')) {
+			$this->markTestSkipped('Skipping reports tests - WC_Admin_Reports_Sync class not found.');
 			return;
 		}
 
 		parent::setUp();
-		wp_set_current_user( self::$user );
+		wp_set_current_user(self::$user);
 
 		global $wpdb;
-		$wpdb->query( "DELETE FROM $wpdb->prefix" . \WC_Admin_Reports_Orders_Stats_Data_Store::TABLE_NAME ); // @codingStandardsIgnoreLine.
-		$wpdb->query( "DELETE FROM $wpdb->prefix" . \WC_Admin_Reports_Products_Data_Store::TABLE_NAME ); // @codingStandardsIgnoreLine.
-		$wpdb->query( "DELETE FROM $wpdb->prefix" . \WC_Admin_Reports_Coupons_Data_Store::TABLE_NAME ); // @codingStandardsIgnoreLine.
-		$wpdb->query( "DELETE FROM $wpdb->prefix" . \WC_Admin_Reports_Customers_Data_Store::TABLE_NAME ); // @codingStandardsIgnoreLine.
+		$wpdb->query("DELETE FROM $wpdb->prefix" . \WC_Admin_Reports_Orders_Stats_Data_Store::TABLE_NAME); // @codingStandardsIgnoreLine.
+		$wpdb->query("DELETE FROM $wpdb->prefix" . \WC_Admin_Reports_Products_Data_Store::TABLE_NAME); // @codingStandardsIgnoreLine.
+		$wpdb->query("DELETE FROM $wpdb->prefix" . \WC_Admin_Reports_Coupons_Data_Store::TABLE_NAME); // @codingStandardsIgnoreLine.
+		$wpdb->query("DELETE FROM $wpdb->prefix" . \WC_Admin_Reports_Customers_Data_Store::TABLE_NAME); // @codingStandardsIgnoreLine.
 	}
 
 }

woocommerce-rest-api.php

Spacing +6 added lines, -6 removed lines

@@ -12,9 +12,9 @@ 
  * @package WooCommerce/RestApi
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
-if ( version_compare( PHP_VERSION, '5.6.0', '<' ) ) {
+if (version_compare(PHP_VERSION, '5.6.0', '<')) {
 	return;
 }
 
@@ -27,11 +27,11 @@ 
 /**
  * This callback registers this version of the API with WooCommerce.
  */
-$register_callback = function() use ( $version, $init_callback ) {
-	if ( ! is_callable( array( wc()->api, 'register' ) ) ) {
+$register_callback = function() use ($version, $init_callback) {
+	if ( ! is_callable(array(wc()->api, 'register'))) {
 		return;
 	}
-	wc()->api->register( $version, $init_callback );
+	wc()->api->register($version, $init_callback);
 };
 
-add_action( 'woocommerce_loaded', $register_callback );
+add_action('woocommerce_loaded', $register_callback);

src/RestApi/Version3/class-wc-rest-data-currencies-controller.php

Spacing +46 added lines, -46 removed lines

@@ -8,7 +8,7 @@ 
  * @since   3.5.0
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * REST API Data Currencies controller class.
@@ -39,36 +39,36 @@ 
 			$this->namespace, '/' . $this->rest_base, array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_items' ),
-					'permission_callback' => array( $this, 'get_items_permissions_check' ),
+					'callback'            => array($this, 'get_items'),
+					'permission_callback' => array($this, 'get_items_permissions_check'),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 		register_rest_route(
 			$this->namespace, '/' . $this->rest_base . '/current', array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_current_item' ),
-					'permission_callback' => array( $this, 'get_item_permissions_check' ),
+					'callback'            => array($this, 'get_current_item'),
+					'permission_callback' => array($this, 'get_item_permissions_check'),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 		register_rest_route(
 			$this->namespace, '/' . $this->rest_base . '/(?P<currency>[\w-]{3})', array(
 				array(
 					'methods'             => WP_REST_Server::READABLE,
-					'callback'            => array( $this, 'get_item' ),
-					'permission_callback' => array( $this, 'get_item_permissions_check' ),
+					'callback'            => array($this, 'get_item'),
+					'permission_callback' => array($this, 'get_item_permissions_check'),
 					'args'                => array(
 						'location' => array(
-							'description' => __( 'ISO4217 currency code.', 'woocommerce' ),
+							'description' => __('ISO4217 currency code.', 'woocommerce'),
 							'type'        => 'string',
 						),
 					),
 				),
-				'schema' => array( $this, 'get_public_item_schema' ),
+				'schema' => array($this, 'get_public_item_schema'),
 			)
 		);
 	}
@@ -80,18 +80,18 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return array|mixed Response data, ready for insertion into collection data.
 	 */
-	public function get_currency( $code = false, $request ) {
+	public function get_currency($code = false, $request) {
 		$currencies = get_woocommerce_currencies();
 		$data       = array();
 
-		if ( ! array_key_exists( $code, $currencies ) ) {
+		if ( ! array_key_exists($code, $currencies)) {
 			return false;
 		}
 
 		$currency = array(
 			'code'   => $code,
-			'name'   => $currencies[ $code ],
-			'symbol' => get_woocommerce_currency_symbol( $code ),
+			'name'   => $currencies[$code],
+			'symbol' => get_woocommerce_currency_symbol($code),
 		);
 
 		return $currency;
@@ -103,15 +103,15 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function get_items( $request ) {
+	public function get_items($request) {
 		$currencies = get_woocommerce_currencies();
-		foreach ( array_keys( $currencies ) as $code ) {
-			$currency = $this->get_currency( $code, $request );
-			$response = $this->prepare_item_for_response( $currency, $request );
-			$data[]   = $this->prepare_response_for_collection( $response );
+		foreach (array_keys($currencies) as $code) {
+			$currency = $this->get_currency($code, $request);
+			$response = $this->prepare_item_for_response($currency, $request);
+			$data[]   = $this->prepare_response_for_collection($response);
 		}
 
-		return rest_ensure_response( $data );
+		return rest_ensure_response($data);
 	}
 
 	/**
@@ -120,12 +120,12 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function get_item( $request ) {
-		$data = $this->get_currency( strtoupper( $request['currency'] ), $request );
-		if ( empty( $data ) ) {
-			return new WP_Error( 'woocommerce_rest_data_invalid_currency', __( 'There are no currencies matching these parameters.', 'woocommerce' ), array( 'status' => 404 ) );
+	public function get_item($request) {
+		$data = $this->get_currency(strtoupper($request['currency']), $request);
+		if (empty($data)) {
+			return new WP_Error('woocommerce_rest_data_invalid_currency', __('There are no currencies matching these parameters.', 'woocommerce'), array('status' => 404));
 		}
-		return $this->prepare_item_for_response( $data, $request );
+		return $this->prepare_item_for_response($data, $request);
 	}
 
 	/**
@@ -134,9 +134,9 @@ 
 	 * @param  WP_REST_Request $request Request data.
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function get_current_item( $request ) {
-		$currency = get_option( 'woocommerce_currency' );
-		return $this->prepare_item_for_response( $this->get_currency( $currency, $request ), $request );
+	public function get_current_item($request) {
+		$currency = get_option('woocommerce_currency');
+		return $this->prepare_item_for_response($this->get_currency($currency, $request), $request);
 	}
 
 	/**
@@ -146,12 +146,12 @@ 
 	 * @param WP_REST_Request $request Request object.
 	 * @return WP_REST_Response $response Response data.
 	 */
-	public function prepare_item_for_response( $item, $request ) {
-		$data     = $this->add_additional_fields_to_object( $item, $request );
-		$data     = $this->filter_response_by_context( $data, 'view' );
-		$response = rest_ensure_response( $data );
+	public function prepare_item_for_response($item, $request) {
+		$data     = $this->add_additional_fields_to_object($item, $request);
+		$data     = $this->filter_response_by_context($data, 'view');
+		$response = rest_ensure_response($data);
 
-		$response->add_links( $this->prepare_links( $item ) );
+		$response->add_links($this->prepare_links($item));
 
 		/**
 		 * Filter currency returned from the API.
@@ -160,7 +160,7 @@ 
 		 * @param array            $item     Currency data.
 		 * @param WP_REST_Request  $request  Request used to generate the response.
 		 */
-		return apply_filters( 'woocommerce_rest_prepare_data_currency', $response, $item, $request );
+		return apply_filters('woocommerce_rest_prepare_data_currency', $response, $item, $request);
 	}
 
 	/**
@@ -169,14 +169,14 @@ 
 	 * @param object $item Data object.
 	 * @return array Links for the given currency.
 	 */
-	protected function prepare_links( $item ) {
-		$code  = strtoupper( $item['code'] );
+	protected function prepare_links($item) {
+		$code  = strtoupper($item['code']);
 		$links = array(
 			'self'       => array(
-				'href' => rest_url( sprintf( '/%s/%s/%s', $this->namespace, $this->rest_base, $code ) ),
+				'href' => rest_url(sprintf('/%s/%s/%s', $this->namespace, $this->rest_base, $code)),
 			),
 			'collection' => array(
-				'href' => rest_url( sprintf( '/%s/%s', $this->namespace, $this->rest_base ) ),
+				'href' => rest_url(sprintf('/%s/%s', $this->namespace, $this->rest_base)),
 			),
 		);
 
@@ -197,25 +197,25 @@ 
 			'properties' => array(
 				'code'   => array(
 					'type'        => 'string',
-					'description' => __( 'ISO4217 currency code.', 'woocommerce' ),
-					'context'     => array( 'view' ),
+					'description' => __('ISO4217 currency code.', 'woocommerce'),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'name'   => array(
 					'type'        => 'string',
-					'description' => __( 'Full name of currency.', 'woocommerce' ),
-					'context'     => array( 'view' ),
+					'description' => __('Full name of currency.', 'woocommerce'),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 				'symbol' => array(
 					'type'        => 'string',
-					'description' => __( 'Currency symbol.', 'woocommerce' ),
-					'context'     => array( 'view' ),
+					'description' => __('Currency symbol.', 'woocommerce'),
+					'context'     => array('view'),
 					'readonly'    => true,
 				),
 			),
 		);
 
-		return $this->add_additional_fields_schema( $schema );
+		return $this->add_additional_fields_schema($schema);
 	}
 }

src/RestApi/Version3/class-wc-rest-system-status-tools-controller.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
  * @since   3.0.0
  */
 
-defined( 'ABSPATH' ) || exit;
+defined('ABSPATH') || exit;
 
 /**
  * System status tools controller.