woocommerce / woocommerce-rest-api

unit-tests/Tests/Version3/reports-customers-totals.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 		$users_count     = count_users();
 		$total_customers = 0;
 
-		foreach ( $users_count['avail_roles'] as $role => $total ) {
+		foreach ( $users_count[ 'avail_roles' ] as $role => $total ) {
 			if ( in_array( $role, array( 'administrator', 'shop_manager' ), true ) ) {
 				continue;
 			}
@@ -109,7 +109,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v3/reports/customers/totals' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( 3, count( $properties ) );
 		$this->assertArrayHasKey( 'slug', $properties );

unit-tests/Tests/Version2/shipping-zones.php

Spacing +50 added lines, -50 removed lines

@@ -26,7 +26,7 @@ 
 				'role' => 'administrator',
 			)
 		);
-		$this->zones    = array();
+		$this->zones = array();
 	}
 
 	/**
@@ -43,7 +43,7 @@ 
 		$zone->set_locations( $locations );
 		$zone->save();
 
-		$this->zones[] = $zone;
+		$this->zones[ ] = $zone;
 
 		return $zone;
 	}
@@ -76,13 +76,13 @@ 
 		$this->assertEquals( count( $data ), 1 );
 		$this->assertContains(
 			array(
-				'id'     => $data[0]['id'],
+				'id'     => $data[ 0 ][ 'id' ],
 				'name'   => 'Locations not covered by your other zones',
 				'order'  => 0,
 				'_links' => array(
 					'self'        => array(
 						array(
-							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[0]['id'] ),
+							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[ 0 ][ 'id' ] ),
 						),
 					),
 					'collection'  => array(
@@ -92,7 +92,7 @@ 
 					),
 					'describedby' => array(
 						array(
-							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[0]['id'] . '/locations' ),
+							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[ 0 ][ 'id' ] . '/locations' ),
 						),
 					),
 				),
@@ -110,13 +110,13 @@ 
 		$this->assertEquals( count( $data ), 2 );
 		$this->assertContains(
 			array(
-				'id'     => $data[1]['id'],
+				'id'     => $data[ 1 ][ 'id' ],
 				'name'   => 'Zone 1',
 				'order'  => 0,
 				'_links' => array(
 					'self'        => array(
 						array(
-							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[1]['id'] ),
+							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[ 1 ][ 'id' ] ),
 						),
 					),
 					'collection'  => array(
@@ -126,7 +126,7 @@ 
 					),
 					'describedby' => array(
 						array(
-							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[1]['id'] . '/locations' ),
+							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[ 1 ][ 'id' ] . '/locations' ),
 						),
 					),
 				),
@@ -169,10 +169,10 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/shipping/zones' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 		$this->assertEquals( 3, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );
-		$this->assertTrue( $properties['id']['readonly'] );
+		$this->assertTrue( $properties[ 'id' ][ 'readonly' ] );
 		$this->assertArrayHasKey( 'name', $properties );
 		$this->assertArrayHasKey( 'order', $properties );
 	}
@@ -197,13 +197,13 @@ 
 		$this->assertEquals( 201, $response->get_status() );
 		$this->assertEquals(
 			array(
-				'id'     => $data['id'],
+				'id'     => $data[ 'id' ],
 				'name'   => 'Test Zone',
 				'order'  => 1,
 				'_links' => array(
 					'self'        => array(
 						array(
-							'href' => rest_url( '/wc/v2/shipping/zones/' . $data['id'] ),
+							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[ 'id' ] ),
 						),
 					),
 					'collection'  => array(
@@ -213,7 +213,7 @@ 
 					),
 					'describedby' => array(
 						array(
-							'href' => rest_url( '/wc/v2/shipping/zones/' . $data['id'] . '/locations' ),
+							'href' => rest_url( '/wc/v2/shipping/zones/' . $data[ 'id' ] . '/locations' ),
 						),
 					),
 				),
@@ -577,16 +577,16 @@ 
 		foreach ( $method->get_instance_form_fields() as $id => $field ) {
 			$data = array(
 				'id'          => $id,
-				'label'       => $field['title'],
-				'description' => ( empty( $field['description'] ) ? '' : $field['description'] ),
-				'type'        => $field['type'],
+				'label'       => $field[ 'title' ],
+				'description' => ( empty( $field[ 'description' ] ) ? '' : $field[ 'description' ] ),
+				'type'        => $field[ 'type' ],
 				'value'       => $method->instance_settings[ $id ],
-				'default'     => ( empty( $field['default'] ) ? '' : $field['default'] ),
-				'tip'         => ( empty( $field['description'] ) ? '' : $field['description'] ),
-				'placeholder' => ( empty( $field['placeholder'] ) ? '' : $field['placeholder'] ),
+				'default'     => ( empty( $field[ 'default' ] ) ? '' : $field[ 'default' ] ),
+				'tip'         => ( empty( $field[ 'description' ] ) ? '' : $field[ 'description' ] ),
+				'placeholder' => ( empty( $field[ 'placeholder' ] ) ? '' : $field[ 'placeholder' ] ),
 			);
-			if ( ! empty( $field['options'] ) ) {
-				$data['options'] = $field['options'];
+			if ( ! empty( $field[ 'options' ] ) ) {
+				$data[ 'options' ] = $field[ 'options' ];
 			}
 			$settings[ $id ] = $data;
 		}
@@ -596,7 +596,7 @@ 
 		$expected = array(
 			'id'                 => $instance_id,
 			'instance_id'        => $instance_id,
-			'title'              => $method->instance_settings['title'],
+			'title'              => $method->instance_settings[ 'title' ],
 			'order'              => $method->method_order,
 			'enabled'            => ( 'yes' === $method->enabled ),
 			'method_id'          => $method->id,
@@ -679,12 +679,12 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertArrayHasKey( 'title', $data['settings'] );
-		$this->assertEquals( 'Flat rate', $data['settings']['title']['value'] );
-		$this->assertArrayHasKey( 'tax_status', $data['settings'] );
-		$this->assertEquals( 'taxable', $data['settings']['tax_status']['value'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '0', $data['settings']['cost']['value'] );
+		$this->assertArrayHasKey( 'title', $data[ 'settings' ] );
+		$this->assertEquals( 'Flat rate', $data[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertArrayHasKey( 'tax_status', $data[ 'settings' ] );
+		$this->assertEquals( 'taxable', $data[ 'settings' ][ 'tax_status' ][ 'value' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '0', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 
 		// Update a single value
 		$request = new WP_REST_Request( 'POST', '/wc/v2/shipping/zones/' . $zone->get_id() . '/methods/' . $instance_id );
@@ -698,12 +698,12 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertArrayHasKey( 'title', $data['settings'] );
-		$this->assertEquals( 'Flat rate', $data['settings']['title']['value'] );
-		$this->assertArrayHasKey( 'tax_status', $data['settings'] );
-		$this->assertEquals( 'taxable', $data['settings']['tax_status']['value'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '5', $data['settings']['cost']['value'] );
+		$this->assertArrayHasKey( 'title', $data[ 'settings' ] );
+		$this->assertEquals( 'Flat rate', $data[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertArrayHasKey( 'tax_status', $data[ 'settings' ] );
+		$this->assertEquals( 'taxable', $data[ 'settings' ][ 'tax_status' ][ 'value' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '5', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 
 		// Test multiple settings
 		$request = new WP_REST_Request( 'POST', '/wc/v2/shipping/zones/' . $zone->get_id() . '/methods/' . $instance_id );
@@ -718,12 +718,12 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertArrayHasKey( 'title', $data['settings'] );
-		$this->assertEquals( 'Flat rate', $data['settings']['title']['value'] );
-		$this->assertArrayHasKey( 'tax_status', $data['settings'] );
-		$this->assertEquals( 'none', $data['settings']['tax_status']['value'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '10', $data['settings']['cost']['value'] );
+		$this->assertArrayHasKey( 'title', $data[ 'settings' ] );
+		$this->assertEquals( 'Flat rate', $data[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertArrayHasKey( 'tax_status', $data[ 'settings' ] );
+		$this->assertEquals( 'none', $data[ 'settings' ][ 'tax_status' ][ 'value' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '10', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 
 		// Test bogus
 		$request = new WP_REST_Request( 'POST', '/wc/v2/shipping/zones/' . $zone->get_id() . '/methods/' . $instance_id );
@@ -739,8 +739,8 @@ 
 		$this->assertEquals( 400, $response->get_status() );
 
 		// Test other parameters
-		$this->assertTrue( $data['enabled'] );
-		$this->assertEquals( 1, $data['order'] );
+		$this->assertTrue( $data[ 'enabled' ] );
+		$this->assertEquals( 1, $data[ 'order' ] );
 
 		$request = new WP_REST_Request( 'POST', '/wc/v2/shipping/zones/' . $zone->get_id() . '/methods/' . $instance_id );
 		$request->set_body_params(
@@ -752,10 +752,10 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertFalse( $data['enabled'] );
-		$this->assertEquals( 2, $data['order'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '10', $data['settings']['cost']['value'] );
+		$this->assertFalse( $data[ 'enabled' ] );
+		$this->assertEquals( 2, $data[ 'order' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '10', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 	}
 
 	/**
@@ -776,10 +776,10 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertFalse( $data['enabled'] );
-		$this->assertEquals( 2, $data['order'] );
-		$this->assertArrayHasKey( 'cost', $data['settings'] );
-		$this->assertEquals( '0', $data['settings']['cost']['value'] );
+		$this->assertFalse( $data[ 'enabled' ] );
+		$this->assertEquals( 2, $data[ 'order' ] );
+		$this->assertArrayHasKey( 'cost', $data[ 'settings' ] );
+		$this->assertEquals( '0', $data[ 'settings' ][ 'cost' ][ 'value' ] );
 	}
 
 	/**

unit-tests/Tests/Version2/settings.php

Spacing +44 added lines, -44 removed lines

@@ -122,7 +122,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/settings' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 		$this->assertEquals( 5, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );
 		$this->assertArrayHasKey( 'parent_id', $properties );
@@ -140,7 +140,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/settings/test/woocommerce_shop_page_display' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 		$this->assertEquals( 9, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );
 		$this->assertArrayHasKey( 'label', $properties );
@@ -177,8 +177,8 @@ 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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( 'woocommerce_shop_page_display', $data[ 0 ][ 'id' ] );
+		$this->assertEmpty( $data[ 0 ][ 'value' ] );
 	}
 
 	/**
@@ -204,7 +204,7 @@ 
 		// test defaults first
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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/v2/settings/%s/%s', 'test', 'woocommerce_shop_page_display' ) );
@@ -216,7 +216,7 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'both', $data['value'] );
+		$this->assertEquals( 'both', $data[ 'value' ] );
 		$this->assertEquals( 'both', get_option( 'woocommerce_shop_page_display' ) );
 
 		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v2/settings/%s/%s', 'test', 'woocommerce_shop_page_display' ) );
@@ -228,7 +228,7 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'subcategories', $data['value'] );
+		$this->assertEquals( 'subcategories', $data[ 'value' ] );
 		$this->assertEquals( 'subcategories', get_option( 'woocommerce_shop_page_display' ) );
 
 		$request = new WP_REST_Request( 'PUT', sprintf( '/wc/v2/settings/%s/%s', 'test', 'woocommerce_shop_page_display' ) );
@@ -240,7 +240,7 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( '', $data['value'] );
+		$this->assertEquals( '', $data[ 'value' ] );
 		$this->assertEquals( '', get_option( 'woocommerce_shop_page_display' ) );
 	}
 
@@ -255,7 +255,7 @@ 
 		// test defaults first
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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/v2/settings/test/batch' );
@@ -272,7 +272,7 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'both', $data['update'][0]['value'] );
+		$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
@@ -289,7 +289,7 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
-		$this->assertEquals( 'subcategories', $data['update'][0]['value'] );
+		$this->assertEquals( 'subcategories', $data[ 'update' ][ 0 ][ 'value' ] );
 		$this->assertEquals( 'subcategories', get_option( 'woocommerce_shop_page_display' ) );
 	}
 
@@ -317,11 +317,11 @@ 
 
 		$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' ] );
 	}
 
 	/**
@@ -484,7 +484,7 @@ 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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/v2/settings/%s/%s', 'products', 'woocommerce_dimension_unit' ) );
@@ -496,7 +496,7 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'yd', $data['value'] );
+		$this->assertEquals( 'yd', $data[ 'value' ] );
 		$this->assertEquals( 'yd', get_option( 'woocommerce_dimension_unit' ) );
 	}
 
@@ -583,7 +583,7 @@ 
 		$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/v2/settings/%s/%s', 'email_customer_new_account', 'subject' ) );
@@ -595,13 +595,13 @@ 
 		$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/v2/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' ] );
 	}
 
 	/**
@@ -682,7 +682,7 @@ 
 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', sprintf( '/wc/v2/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/v2/settings/%s/%s', 'general', 'woocommerce_specific_allowed_countries' ) );
 		$request->set_body_params(
@@ -692,7 +692,7 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$setting  = $response->get_data();
-		$this->assertEquals( array( 'AX', 'DZ' ), $setting['value'] );
+		$this->assertEquals( array( 'AX', 'DZ' ), $setting[ 'value' ] );
 	}
 
 	/**
@@ -705,7 +705,7 @@ 
 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', sprintf( '/wc/v2/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/v2/settings/%s/%s', 'products', 'woocommerce_weight_unit' ) );
@@ -726,7 +726,7 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$setting  = $response->get_data();
-		$this->assertEquals( 'lbs', $setting['value'] );
+		$this->assertEquals( 'lbs', $setting[ 'value' ] );
 	}
 
 	/**
@@ -741,9 +741,9 @@ 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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' ] );
 	}
 
 	/**
@@ -755,10 +755,10 @@ 
 		wp_set_current_user( $this->user );
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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'];
+		$old_value = $setting[ 'value' ];
 		$new_value = $old_value . ' ' . rand( 1000, 9999 );
 		$request   = new WP_REST_Request( 'PUT', '/wc/v2/settings/general/woocommerce_store_address' );
 		$request->set_body_params(
@@ -768,7 +768,7 @@ 
 		);
 		$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/v2/settings/general/woocommerce_store_address' );
@@ -779,7 +779,7 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$setting  = $response->get_data();
-		$this->assertEquals( $old_value, $setting['value'] );
+		$this->assertEquals( $old_value, $setting[ 'value' ] );
 	}
 
 	/**
@@ -791,10 +791,10 @@ 
 		wp_set_current_user( $this->user );
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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'];
+		$old_value = $setting[ 'value' ];
 		$new_value = $old_value . ' ' . rand( 1000, 9999 );
 		$request   = new WP_REST_Request( 'PUT', '/wc/v2/settings/general/woocommerce_store_address_2' );
 		$request->set_body_params(
@@ -804,7 +804,7 @@ 
 		);
 		$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/v2/settings/general/woocommerce_store_address_2' );
@@ -815,7 +815,7 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$setting  = $response->get_data();
-		$this->assertEquals( $old_value, $setting['value'] );
+		$this->assertEquals( $old_value, $setting[ 'value' ] );
 	}
 
 	/**
@@ -827,10 +827,10 @@ 
 		wp_set_current_user( $this->user );
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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'];
+		$old_value = $setting[ 'value' ];
 		$new_value = $old_value . ' ' . rand( 1000, 9999 );
 		$request   = new WP_REST_Request( 'PUT', '/wc/v2/settings/general/woocommerce_store_city' );
 		$request->set_body_params(
@@ -840,7 +840,7 @@ 
 		);
 		$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/v2/settings/general/woocommerce_store_city' );
@@ -851,7 +851,7 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$setting  = $response->get_data();
-		$this->assertEquals( $old_value, $setting['value'] );
+		$this->assertEquals( $old_value, $setting[ 'value' ] );
 	}
 
 	/**
@@ -863,10 +863,10 @@ 
 		wp_set_current_user( $this->user );
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/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'];
+		$old_value = $setting[ 'value' ];
 		$new_value = $old_value . ' ' . rand( 1000, 9999 );
 		$request   = new WP_REST_Request( 'PUT', '/wc/v2/settings/general/woocommerce_store_postcode' );
 		$request->set_body_params(
@@ -876,7 +876,7 @@ 
 		);
 		$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/v2/settings/general/woocommerce_store_postcode' );
@@ -887,6 +887,6 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$setting  = $response->get_data();
-		$this->assertEquals( $old_value, $setting['value'] );
+		$this->assertEquals( $old_value, $setting[ 'value' ] );
 	}
 }

unit-tests/Tests/Version2/product-variations.php

Spacing +40 added lines, -40 removed lines

@@ -45,8 +45,8 @@ 
 		$variations = $response->get_data();
 		$this->assertEquals( 200, $response->get_status() );
 		$this->assertEquals( 2, count( $variations ) );
-		$this->assertEquals( 'DUMMY SKU VARIABLE LARGE', $variations[0]['sku'] );
-		$this->assertEquals( 'size', $variations[0]['attributes'][0]['name'] );
+		$this->assertEquals( 'DUMMY SKU VARIABLE LARGE', $variations[ 0 ][ 'sku' ] );
+		$this->assertEquals( 'size', $variations[ 0 ][ 'attributes' ][ 0 ][ 'name' ] );
 	}
 
 	/**
@@ -70,14 +70,14 @@ 
 		wp_set_current_user( $this->user );
 		$product      = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper::create_variation_product();
 		$children     = $product->get_children();
-		$variation_id = $children[0];
+		$variation_id = $children[ 0 ];
 
 		$response  = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id ) );
 		$variation = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( $variation_id, $variation['id'] );
-		$this->assertEquals( 'size', $variation['attributes'][0]['name'] );
+		$this->assertEquals( $variation_id, $variation[ 'id' ] );
+		$this->assertEquals( 'size', $variation[ 'attributes' ][ 0 ][ 'name' ] );
 	}
 
 	/**
@@ -89,7 +89,7 @@ 
 		wp_set_current_user( 0 );
 		$product      = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper::create_variation_product();
 		$children     = $product->get_children();
-		$variation_id = $children[0];
+		$variation_id = $children[ 0 ];
 		$response     = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id ) );
 		$this->assertEquals( 401, $response->get_status() );
 	}
@@ -103,7 +103,7 @@ 
 		wp_set_current_user( $this->user );
 		$product      = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper::create_variation_product();
 		$children     = $product->get_children();
-		$variation_id = $children[0];
+		$variation_id = $children[ 0 ];
 
 		$request = new WP_REST_Request( 'DELETE', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id );
 		$request->set_param( 'force', true );
@@ -124,7 +124,7 @@ 
 		wp_set_current_user( 0 );
 		$product      = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper::create_variation_product();
 		$children     = $product->get_children();
-		$variation_id = $children[0];
+		$variation_id = $children[ 0 ];
 
 		$request = new WP_REST_Request( 'DELETE', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id );
 		$request->set_param( 'force', true );
@@ -155,15 +155,15 @@ 
 		wp_set_current_user( $this->user );
 		$product      = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper::create_variation_product();
 		$children     = $product->get_children();
-		$variation_id = $children[0];
+		$variation_id = $children[ 0 ];
 
 		$response  = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id ) );
 		$variation = $response->get_data();
 
-		$this->assertEquals( 'DUMMY SKU VARIABLE SMALL', $variation['sku'] );
-		$this->assertEquals( 10, $variation['regular_price'] );
-		$this->assertEmpty( $variation['sale_price'] );
-		$this->assertEquals( 'small', $variation['attributes'][0]['option'] );
+		$this->assertEquals( 'DUMMY SKU VARIABLE SMALL', $variation[ 'sku' ] );
+		$this->assertEquals( 10, $variation[ 'regular_price' ] );
+		$this->assertEmpty( $variation[ 'sale_price' ] );
+		$this->assertEquals( 'small', $variation[ 'attributes' ][ 0 ][ 'option' ] );
 
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id );
 		$request->set_body_params(
@@ -187,15 +187,15 @@ 
 		$response  = $this->server->dispatch( $request );
 		$variation = $response->get_data();
 
-		$this->assertTrue( isset( $variation['description'] ), print_r( $variation, true ) );
-		$this->assertContains( 'O_O', $variation['description'], print_r( $variation, true ) );
-		$this->assertEquals( '8', $variation['price'], print_r( $variation, true ) );
-		$this->assertEquals( '8', $variation['sale_price'], print_r( $variation, true ) );
-		$this->assertEquals( '10', $variation['regular_price'], print_r( $variation, true ) );
-		$this->assertEquals( 'FIXED-SKU', $variation['sku'], print_r( $variation, true ) );
-		$this->assertEquals( 'medium', $variation['attributes'][0]['option'], print_r( $variation, true ) );
-		$this->assertContains( 'Dr1Bczxq4q', $variation['image']['src'], print_r( $variation, true ) );
-		$this->assertContains( 'test upload image', $variation['image']['alt'], print_r( $variation, true ) );
+		$this->assertTrue( isset( $variation[ 'description' ] ), print_r( $variation, true ) );
+		$this->assertContains( 'O_O', $variation[ 'description' ], print_r( $variation, true ) );
+		$this->assertEquals( '8', $variation[ 'price' ], print_r( $variation, true ) );
+		$this->assertEquals( '8', $variation[ 'sale_price' ], print_r( $variation, true ) );
+		$this->assertEquals( '10', $variation[ 'regular_price' ], print_r( $variation, true ) );
+		$this->assertEquals( 'FIXED-SKU', $variation[ 'sku' ], print_r( $variation, true ) );
+		$this->assertEquals( 'medium', $variation[ 'attributes' ][ 0 ][ 'option' ], print_r( $variation, true ) );
+		$this->assertContains( 'Dr1Bczxq4q', $variation[ 'image' ][ 'src' ], print_r( $variation, true ) );
+		$this->assertContains( 'test upload image', $variation[ 'image' ][ 'alt' ], print_r( $variation, true ) );
 	}
 
 	/**
@@ -207,7 +207,7 @@ 
 		wp_set_current_user( 0 );
 		$product      = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper::create_variation_product();
 		$children     = $product->get_children();
-		$variation_id = $children[0];
+		$variation_id = $children[ 0 ];
 
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id );
 		$request->set_body_params(
@@ -267,12 +267,12 @@ 
 		$response  = $this->server->dispatch( $request );
 		$variation = $response->get_data();
 
-		$this->assertContains( 'A medium size.', $variation['description'] );
-		$this->assertEquals( '12', $variation['price'] );
-		$this->assertEquals( '12', $variation['regular_price'] );
-		$this->assertTrue( $variation['purchasable'] );
-		$this->assertEquals( 'DUMMY SKU VARIABLE MEDIUM', $variation['sku'] );
-		$this->assertEquals( 'medium', $variation['attributes'][0]['option'] );
+		$this->assertContains( 'A medium size.', $variation[ 'description' ] );
+		$this->assertEquals( '12', $variation[ 'price' ] );
+		$this->assertEquals( '12', $variation[ 'regular_price' ] );
+		$this->assertTrue( $variation[ 'purchasable' ] );
+		$this->assertEquals( 'DUMMY SKU VARIABLE MEDIUM', $variation[ 'sku' ] );
+		$this->assertEquals( 'medium', $variation[ 'attributes' ][ 0 ][ 'option' ] );
 
 		$response   = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/products/' . $product->get_id() . '/variations' ) );
 		$variations = $response->get_data();
@@ -318,7 +318,7 @@ 
 			array(
 				'update' => array(
 					array(
-						'id'          => $children[0],
+						'id'          => $children[ 0 ],
 						'description' => 'Updated description.',
 						'image'       => array(
 							'position' => 0,
@@ -328,7 +328,7 @@ 
 					),
 				),
 				'delete' => array(
-					$children[1],
+					$children[ 1 ],
 				),
 				'create' => array(
 					array(
@@ -348,10 +348,10 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertContains( 'Updated description.', $data['update'][0]['description'] );
-		$this->assertEquals( 'DUMMY SKU VARIABLE MEDIUM', $data['create'][0]['sku'] );
-		$this->assertEquals( 'medium', $data['create'][0]['attributes'][0]['option'] );
-		$this->assertEquals( $children[1], $data['delete'][0]['id'] );
+		$this->assertContains( 'Updated description.', $data[ 'update' ][ 0 ][ 'description' ] );
+		$this->assertEquals( 'DUMMY SKU VARIABLE MEDIUM', $data[ 'create' ][ 0 ][ 'sku' ] );
+		$this->assertEquals( 'medium', $data[ 'create' ][ 0 ][ 'attributes' ][ 0 ][ 'option' ] );
+		$this->assertEquals( $children[ 1 ], $data[ 'delete' ][ 0 ][ 'id' ] );
 
 		$request  = new WP_REST_Request( 'GET', '/wc/v2/products/' . $product->get_id() . '/variations' );
 		$response = $this->server->dispatch( $request );
@@ -371,7 +371,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/products/' . $product->get_id() . '/variations' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( 37, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );
@@ -424,7 +424,7 @@ 
 		$product->save();
 
 		$children     = $product->get_children();
-		$variation_id = $children[0];
+		$variation_id = $children[ 0 ];
 
 		// Set stock to true.
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id );
@@ -438,7 +438,7 @@ 
 		$variation = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( true, $variation['manage_stock'] );
+		$this->assertEquals( true, $variation[ 'manage_stock' ] );
 
 		// Set stock to false.
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/products/' . $product->get_id() . '/variations/' . $variation_id );
@@ -452,7 +452,7 @@ 
 		$variation = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( false, $variation['manage_stock'] );
+		$this->assertEquals( false, $variation[ 'manage_stock' ] );
 
 		// Set stock to false but parent is managing stock.
 		$product->set_manage_stock( true );
@@ -468,6 +468,6 @@ 
 		$variation = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( 'parent', $variation['manage_stock'] );
+		$this->assertEquals( 'parent', $variation[ 'manage_stock' ] );
 	}
 }

unit-tests/Tests/Version2/coupons.php

Spacing +17 added lines, -17 removed lines

@@ -194,13 +194,13 @@ 
 		$this->assertEquals( 201, $response->get_status() );
 		$this->assertEquals(
 			array(
-				'id'                          => $data['id'],
+				'id'                          => $data[ 'id' ],
 				'code'                        => 'test',
 				'amount'                      => '5.00',
-				'date_created'                => $data['date_created'],
-				'date_created_gmt'            => $data['date_created_gmt'],
-				'date_modified'               => $data['date_modified'],
-				'date_modified_gmt'           => $data['date_modified_gmt'],
+				'date_created'                => $data[ 'date_created' ],
+				'date_created_gmt'            => $data[ 'date_created_gmt' ],
+				'date_modified'               => $data[ 'date_modified' ],
+				'date_modified_gmt'           => $data[ 'date_modified_gmt' ],
 				'discount_type'               => 'fixed_product',
 				'description'                 => 'Test',
 				'date_expires'                => null,
@@ -280,9 +280,9 @@ 
 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/coupons/' . $coupon->get_id() ) );
 		$data     = $response->get_data();
-		$this->assertEquals( 'This is a dummy coupon', $data['description'] );
-		$this->assertEquals( 'fixed_cart', $data['discount_type'] );
-		$this->assertEquals( '1.00', $data['amount'] );
+		$this->assertEquals( 'This is a dummy coupon', $data[ 'description' ] );
+		$this->assertEquals( 'fixed_cart', $data[ 'discount_type' ] );
+		$this->assertEquals( '1.00', $data[ 'amount' ] );
 
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/coupons/' . $coupon->get_id() );
 		$request->set_body_params(
@@ -294,9 +294,9 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( '10.00', $data['amount'] );
-		$this->assertEquals( 'New description', $data['description'] );
-		$this->assertEquals( 'fixed_cart', $data['discount_type'] );
+		$this->assertEquals( '10.00', $data[ 'amount' ] );
+		$this->assertEquals( 'New description', $data[ 'description' ] );
+		$this->assertEquals( 'fixed_cart', $data[ 'discount_type' ] );
 	}
 
 	/**
@@ -416,11 +416,11 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( '5.15', $data['update'][0]['amount'] );
-		$this->assertEquals( '11.00', $data['create'][0]['amount'] );
-		$this->assertEquals( 'new-coupon', $data['create'][0]['code'] );
-		$this->assertEquals( $coupon_2->get_id(), $data['delete'][0]['id'] );
-		$this->assertEquals( $coupon_3->get_id(), $data['delete'][1]['id'] );
+		$this->assertEquals( '5.15', $data[ 'update' ][ 0 ][ 'amount' ] );
+		$this->assertEquals( '11.00', $data[ 'create' ][ 0 ][ 'amount' ] );
+		$this->assertEquals( 'new-coupon', $data[ 'create' ][ 0 ][ 'code' ] );
+		$this->assertEquals( $coupon_2->get_id(), $data[ 'delete' ][ 0 ][ 'id' ] );
+		$this->assertEquals( $coupon_3->get_id(), $data[ 'delete' ][ 1 ][ 'id' ] );
 
 		$request  = new WP_REST_Request( 'GET', '/wc/v2/coupons' );
 		$response = $this->server->dispatch( $request );
@@ -438,7 +438,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/coupons' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( 27, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );

unit-tests/Tests/Version2/orders.php

Spacing +51 added lines, -51 removed lines

@@ -46,7 +46,7 @@ 
 
 		// Create 10 orders.
 		for ( $i = 0; $i < 10; $i++ ) {
-			$this->orders[] = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\OrderHelper::create_order( $this->user );
+			$this->orders[ ] = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\OrderHelper::create_order( $this->user );
 		}
 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/orders' ) );
@@ -63,7 +63,7 @@ 
 	 */
 	public function test_get_items_without_permission() {
 		wp_set_current_user( 0 );
-		$this->orders[] = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\OrderHelper::create_order();
+		$this->orders[ ] = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\OrderHelper::create_order();
 		$response       = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/orders' ) );
 		$this->assertEquals( 401, $response->get_status() );
 	}
@@ -78,18 +78,18 @@ 
 		$order->add_meta_data( 'key', 'value' );
 		$order->add_meta_data( 'key2', 'value2' );
 		$order->save();
-		$this->orders[] = $order;
+		$this->orders[ ] = $order;
 		$response       = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/orders/' . $order->get_id() ) );
 		$data           = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( $order->get_id(), $data['id'] );
+		$this->assertEquals( $order->get_id(), $data[ 'id' ] );
 
 		// Test meta data is set.
-		$this->assertEquals( 'key', $data['meta_data'][0]->key );
-		$this->assertEquals( 'value', $data['meta_data'][0]->value );
-		$this->assertEquals( 'key2', $data['meta_data'][1]->key );
-		$this->assertEquals( 'value2', $data['meta_data'][1]->value );
+		$this->assertEquals( 'key', $data[ 'meta_data' ][ 0 ]->key );
+		$this->assertEquals( 'value', $data[ 'meta_data' ][ 0 ]->value );
+		$this->assertEquals( 'key2', $data[ 'meta_data' ][ 1 ]->key );
+		$this->assertEquals( 'value2', $data[ 'meta_data' ][ 1 ]->value );
 	}
 
 	/**
@@ -99,7 +99,7 @@ 
 	public function test_get_item_without_permission() {
 		wp_set_current_user( 0 );
 		$order          = \Automattic\WooCommerce\RestApi\UnitTests\Helpers\OrderHelper::create_order();
-		$this->orders[] = $order;
+		$this->orders[ ] = $order;
 		$response       = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/orders/' . $order->get_id() ) );
 		$this->assertEquals( 401, $response->get_status() );
 	}
@@ -182,32 +182,32 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
-		$order    = wc_get_order( $data['id'] );
+		$order    = wc_get_order( $data[ 'id' ] );
 		$this->assertEquals( 201, $response->get_status() );
-		$this->assertEquals( $order->get_payment_method(), $data['payment_method'] );
-		$this->assertEquals( $order->get_payment_method_title(), $data['payment_method_title'] );
-		$this->assertEquals( $order->get_billing_first_name(), $data['billing']['first_name'] );
-		$this->assertEquals( $order->get_billing_last_name(), $data['billing']['last_name'] );
-		$this->assertEquals( '', $data['billing']['company'] );
-		$this->assertEquals( $order->get_billing_address_1(), $data['billing']['address_1'] );
-		$this->assertEquals( $order->get_billing_address_2(), $data['billing']['address_2'] );
-		$this->assertEquals( $order->get_billing_city(), $data['billing']['city'] );
-		$this->assertEquals( $order->get_billing_state(), $data['billing']['state'] );
-		$this->assertEquals( $order->get_billing_postcode(), $data['billing']['postcode'] );
-		$this->assertEquals( $order->get_billing_country(), $data['billing']['country'] );
-		$this->assertEquals( $order->get_billing_email(), $data['billing']['email'] );
-		$this->assertEquals( $order->get_billing_phone(), $data['billing']['phone'] );
-		$this->assertEquals( $order->get_shipping_first_name(), $data['shipping']['first_name'] );
-		$this->assertEquals( $order->get_shipping_last_name(), $data['shipping']['last_name'] );
-		$this->assertEquals( '', $data['shipping']['company'] );
-		$this->assertEquals( $order->get_shipping_address_1(), $data['shipping']['address_1'] );
-		$this->assertEquals( $order->get_shipping_address_2(), $data['shipping']['address_2'] );
-		$this->assertEquals( $order->get_shipping_city(), $data['shipping']['city'] );
-		$this->assertEquals( $order->get_shipping_state(), $data['shipping']['state'] );
-		$this->assertEquals( $order->get_shipping_postcode(), $data['shipping']['postcode'] );
-		$this->assertEquals( $order->get_shipping_country(), $data['shipping']['country'] );
-		$this->assertEquals( 1, count( $data['line_items'] ) );
-		$this->assertEquals( 1, count( $data['shipping_lines'] ) );
+		$this->assertEquals( $order->get_payment_method(), $data[ 'payment_method' ] );
+		$this->assertEquals( $order->get_payment_method_title(), $data[ 'payment_method_title' ] );
+		$this->assertEquals( $order->get_billing_first_name(), $data[ 'billing' ][ 'first_name' ] );
+		$this->assertEquals( $order->get_billing_last_name(), $data[ 'billing' ][ 'last_name' ] );
+		$this->assertEquals( '', $data[ 'billing' ][ 'company' ] );
+		$this->assertEquals( $order->get_billing_address_1(), $data[ 'billing' ][ 'address_1' ] );
+		$this->assertEquals( $order->get_billing_address_2(), $data[ 'billing' ][ 'address_2' ] );
+		$this->assertEquals( $order->get_billing_city(), $data[ 'billing' ][ 'city' ] );
+		$this->assertEquals( $order->get_billing_state(), $data[ 'billing' ][ 'state' ] );
+		$this->assertEquals( $order->get_billing_postcode(), $data[ 'billing' ][ 'postcode' ] );
+		$this->assertEquals( $order->get_billing_country(), $data[ 'billing' ][ 'country' ] );
+		$this->assertEquals( $order->get_billing_email(), $data[ 'billing' ][ 'email' ] );
+		$this->assertEquals( $order->get_billing_phone(), $data[ 'billing' ][ 'phone' ] );
+		$this->assertEquals( $order->get_shipping_first_name(), $data[ 'shipping' ][ 'first_name' ] );
+		$this->assertEquals( $order->get_shipping_last_name(), $data[ 'shipping' ][ 'last_name' ] );
+		$this->assertEquals( '', $data[ 'shipping' ][ 'company' ] );
+		$this->assertEquals( $order->get_shipping_address_1(), $data[ 'shipping' ][ 'address_1' ] );
+		$this->assertEquals( $order->get_shipping_address_2(), $data[ 'shipping' ][ 'address_2' ] );
+		$this->assertEquals( $order->get_shipping_city(), $data[ 'shipping' ][ 'city' ] );
+		$this->assertEquals( $order->get_shipping_state(), $data[ 'shipping' ][ 'state' ] );
+		$this->assertEquals( $order->get_shipping_postcode(), $data[ 'shipping' ][ 'postcode' ] );
+		$this->assertEquals( $order->get_shipping_country(), $data[ 'shipping' ][ 'country' ] );
+		$this->assertEquals( 1, count( $data[ 'line_items' ] ) );
+		$this->assertEquals( 1, count( $data[ 'shipping_lines' ] ) );
 	}
 
 	/**
@@ -265,13 +265,13 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
-		$order    = wc_get_order( $data['id'] );
+		$order    = wc_get_order( $data[ 'id' ] );
 		$this->assertEquals( 201, $response->get_status() );
-		$this->assertEquals( $order->get_payment_method(), $data['payment_method'] );
+		$this->assertEquals( $order->get_payment_method(), $data[ 'payment_method' ] );
 		$this->assertEquals( $order->get_payment_method_title(), 'Sanitize this' );
 
 		// Test when updating order.
-		$request = new WP_REST_Request( 'PUT', '/wc/v3/orders/' . $data['id'] );
+		$request = new WP_REST_Request( 'PUT', '/wc/v3/orders/' . $data[ 'id' ] );
 		$request->set_body_params(
 			array(
 				'payment_method'       => 'bacs',
@@ -280,9 +280,9 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
-		$order    = wc_get_order( $data['id'] );
+		$order    = wc_get_order( $data[ 'id' ] );
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( $order->get_payment_method(), $data['payment_method'] );
+		$this->assertEquals( $order->get_payment_method(), $data[ 'payment_method' ] );
 		$this->assertEquals( $order->get_payment_method_title(), 'Sanitize this too' );
 	}
 
@@ -366,9 +366,9 @@ 
 		$data     = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( 'test-update', $data['payment_method'] );
-		$this->assertEquals( 'Fish', $data['billing']['first_name'] );
-		$this->assertEquals( 'Face', $data['billing']['last_name'] );
+		$this->assertEquals( 'test-update', $data[ 'payment_method' ] );
+		$this->assertEquals( 'Fish', $data[ 'billing' ][ 'first_name' ] );
+		$this->assertEquals( 'Face', $data[ 'billing' ][ 'last_name' ] );
 	}
 
 	/**
@@ -408,7 +408,7 @@ 
 		$data     = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertTrue( empty( $data['fee_lines'] ) );
+		$this->assertTrue( empty( $data[ 'fee_lines' ] ) );
 	}
 
 	/**
@@ -446,8 +446,8 @@ 
 		$data     = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertCount( 1, $data['coupon_lines'] );
-		$this->assertEquals( '45.00', $data['total'] );
+		$this->assertCount( 1, $data[ 'coupon_lines' ] );
+		$this->assertEquals( '45.00', $data[ 'total' ] );
 	}
 
 	/**
@@ -493,8 +493,8 @@ 
 		$data     = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertTrue( empty( $data['coupon_lines'] ) );
-		$this->assertEquals( '50.00', $data['total'] );
+		$this->assertTrue( empty( $data[ 'coupon_lines' ] ) );
+		$this->assertEquals( '50.00', $data[ 'total' ] );
 	}
 
 	/**
@@ -607,9 +607,9 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'updated', $data['update'][0]['payment_method'] );
-		$this->assertEquals( $order2->get_id(), $data['delete'][0]['id'] );
-		$this->assertEquals( $order3->get_id(), $data['delete'][1]['id'] );
+		$this->assertEquals( 'updated', $data[ 'update' ][ 0 ][ 'payment_method' ] );
+		$this->assertEquals( $order2->get_id(), $data[ 'delete' ][ 0 ][ 'id' ] );
+		$this->assertEquals( $order3->get_id(), $data[ 'delete' ][ 1 ][ 'id' ] );
 
 		$request  = new WP_REST_Request( 'GET', '/wc/v2/orders' );
 		$response = $this->server->dispatch( $request );
@@ -627,7 +627,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/orders/' . $order->get_id() );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( 42, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );

unit-tests/Tests/Version2/customers.php

Spacing +48 added lines, -48 removed lines

@@ -144,11 +144,11 @@ 
 		$this->assertEquals( 201, $response->get_status() );
 		$this->assertEquals(
 			array(
-				'id'                 => $data['id'],
-				'date_created'       => $data['date_created'],
-				'date_created_gmt'   => $data['date_created_gmt'],
-				'date_modified'      => $data['date_modified'],
-				'date_modified_gmt'  => $data['date_modified_gmt'],
+				'id'                 => $data[ 'id' ],
+				'date_created'       => $data[ 'date_created' ],
+				'date_created_gmt'   => $data[ 'date_created_gmt' ],
+				'date_modified'      => $data[ 'date_modified' ],
+				'date_modified_gmt'  => $data[ 'date_modified_gmt' ],
 				'email'              => 'create_customer_test@woo.local',
 				'first_name'         => '',
 				'last_name'          => '',
@@ -182,7 +182,7 @@ 
 				'meta_data'          => array(),
 				'orders_count'       => 0,
 				'total_spent'        => '0.00',
-				'avatar_url'         => $data['avatar_url'],
+				'avatar_url'         => $data[ 'avatar_url' ],
 			),
 			$data
 		);
@@ -212,11 +212,11 @@ 
 		$this->assertEquals( 201, $response->get_status() );
 		$this->assertEquals(
 			array(
-				'id'                 => $data['id'],
-				'date_created'       => $data['date_created'],
-				'date_created_gmt'   => $data['date_created_gmt'],
-				'date_modified'      => $data['date_modified'],
-				'date_modified_gmt'  => $data['date_modified_gmt'],
+				'id'                 => $data[ 'id' ],
+				'date_created'       => $data[ 'date_created' ],
+				'date_created_gmt'   => $data[ 'date_created_gmt' ],
+				'date_modified'      => $data[ 'date_modified' ],
+				'date_modified_gmt'  => $data[ 'date_modified_gmt' ],
 				'email'              => 'create_customer_test2@woo.local',
 				'first_name'         => 'Test',
 				'last_name'          => 'McTestFace',
@@ -250,7 +250,7 @@ 
 				'meta_data'          => array(),
 				'orders_count'       => 0,
 				'total_spent'        => '0.00',
-				'avatar_url'         => $data['avatar_url'],
+				'avatar_url'         => $data[ 'avatar_url' ],
 			),
 			$data
 		);
@@ -302,11 +302,11 @@ 
 
 		$this->assertEquals(
 			array(
-				'id'                 => $data['id'],
-				'date_created'       => $data['date_created'],
-				'date_created_gmt'   => $data['date_created_gmt'],
-				'date_modified'      => $data['date_modified'],
-				'date_modified_gmt'  => $data['date_modified_gmt'],
+				'id'                 => $data[ 'id' ],
+				'date_created'       => $data[ 'date_created' ],
+				'date_created_gmt'   => $data[ 'date_created_gmt' ],
+				'date_modified'      => $data[ 'date_modified' ],
+				'date_modified_gmt'  => $data[ 'date_modified_gmt' ],
 				'email'              => 'get_customer_test@woo.local',
 				'first_name'         => 'Justin',
 				'billing'            => array(
@@ -340,7 +340,7 @@ 
 				'username'           => 'get_customer_test',
 				'orders_count'       => 0,
 				'total_spent'        => '0.00',
-				'avatar_url'         => $data['avatar_url'],
+				'avatar_url'         => $data[ 'avatar_url' ],
 			),
 			$data
 		);
@@ -380,8 +380,8 @@ 
 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/customers/' . $customer->get_id() ) );
 		$data     = $response->get_data();
-		$this->assertEquals( 'update_customer_test', $data['username'] );
-		$this->assertEquals( 'update_customer_test@woo.local', $data['email'] );
+		$this->assertEquals( 'update_customer_test', $data[ 'username' ] );
+		$this->assertEquals( 'update_customer_test@woo.local', $data[ 'email' ] );
 
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/customers/' . $customer->get_id() );
 		$request->set_body_params(
@@ -393,8 +393,8 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'updated_email@woo.local', $data['email'] );
-		$this->assertEquals( 'UpdatedTest', $data['first_name'] );
+		$this->assertEquals( 'updated_email@woo.local', $data[ 'email' ] );
+		$this->assertEquals( 'UpdatedTest', $data[ 'first_name' ] );
 	}
 
 	/**
@@ -500,11 +500,11 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'McTest', $data['update'][0]['last_name'] );
-		$this->assertEquals( 'newuser', $data['create'][0]['username'] );
-		$this->assertEmpty( $data['create'][0]['last_name'] );
-		$this->assertEquals( $customer_2->get_id(), $data['delete'][0]['id'] );
-		$this->assertEquals( $customer_3->get_id(), $data['delete'][1]['id'] );
+		$this->assertEquals( 'McTest', $data[ 'update' ][ 0 ][ 'last_name' ] );
+		$this->assertEquals( 'newuser', $data[ 'create' ][ 0 ][ 'username' ] );
+		$this->assertEmpty( $data[ 'create' ][ 0 ][ 'last_name' ] );
+		$this->assertEquals( $customer_2->get_id(), $data[ 'delete' ][ 0 ][ 'id' ] );
+		$this->assertEquals( $customer_3->get_id(), $data[ 'delete' ][ 1 ][ 'id' ] );
 
 		$request  = new WP_REST_Request( 'GET', '/wc/v2/customers' );
 		$response = $this->server->dispatch( $request );
@@ -523,7 +523,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/customers' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( 18, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );
@@ -541,26 +541,26 @@ 
 		$this->assertArrayHasKey( 'total_spent', $properties );
 		$this->assertArrayHasKey( 'avatar_url', $properties );
 		$this->assertArrayHasKey( 'billing', $properties );
-		$this->assertArrayHasKey( 'first_name', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'last_name', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'company', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'address_1', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'address_2', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'city', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'state', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'postcode', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'country', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'email', $properties['billing']['properties'] );
-		$this->assertArrayHasKey( 'phone', $properties['billing']['properties'] );
+		$this->assertArrayHasKey( 'first_name', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'last_name', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'company', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'address_1', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'address_2', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'city', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'state', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'postcode', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'country', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'email', $properties[ 'billing' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'phone', $properties[ 'billing' ][ 'properties' ] );
 		$this->assertArrayHasKey( 'shipping', $properties );
-		$this->assertArrayHasKey( 'first_name', $properties['shipping']['properties'] );
-		$this->assertArrayHasKey( 'last_name', $properties['shipping']['properties'] );
-		$this->assertArrayHasKey( 'company', $properties['shipping']['properties'] );
-		$this->assertArrayHasKey( 'address_1', $properties['shipping']['properties'] );
-		$this->assertArrayHasKey( 'address_2', $properties['shipping']['properties'] );
-		$this->assertArrayHasKey( 'city', $properties['shipping']['properties'] );
-		$this->assertArrayHasKey( 'state', $properties['shipping']['properties'] );
-		$this->assertArrayHasKey( 'postcode', $properties['shipping']['properties'] );
-		$this->assertArrayHasKey( 'country', $properties['shipping']['properties'] );
+		$this->assertArrayHasKey( 'first_name', $properties[ 'shipping' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'last_name', $properties[ 'shipping' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'company', $properties[ 'shipping' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'address_1', $properties[ 'shipping' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'address_2', $properties[ 'shipping' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'city', $properties[ 'shipping' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'state', $properties[ 'shipping' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'postcode', $properties[ 'shipping' ][ 'properties' ] );
+		$this->assertArrayHasKey( 'country', $properties[ 'shipping' ][ 'properties' ] );
 	}
 }

unit-tests/Tests/Version2/products.php

Spacing +51 added lines, -51 removed lines

@@ -52,10 +52,10 @@ 
 		$this->assertEquals( 200, $response->get_status() );
 
 		$this->assertEquals( 2, count( $products ) );
-		$this->assertEquals( 'Dummy Product', $products[0]['name'] );
-		$this->assertEquals( 'DUMMY SKU', $products[0]['sku'] );
-		$this->assertEquals( 'Dummy External Product', $products[1]['name'] );
-		$this->assertEquals( 'DUMMY EXTERNAL SKU', $products[1]['sku'] );
+		$this->assertEquals( 'Dummy Product', $products[ 0 ][ 'name' ] );
+		$this->assertEquals( 'DUMMY SKU', $products[ 0 ][ 'sku' ] );
+		$this->assertEquals( 'Dummy External Product', $products[ 1 ][ 'name' ] );
+		$this->assertEquals( 'DUMMY EXTERNAL SKU', $products[ 1 ][ 'sku' ] );
 	}
 
 	/**
@@ -166,9 +166,9 @@ 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/products/' . $product->get_id() ) );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'DUMMY SKU', $data['sku'] );
-		$this->assertEquals( 10, $data['regular_price'] );
-		$this->assertEmpty( $data['sale_price'] );
+		$this->assertEquals( 'DUMMY SKU', $data[ 'sku' ] );
+		$this->assertEquals( 10, $data[ 'regular_price' ] );
+		$this->assertEmpty( $data[ 'sale_price' ] );
 
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/products/' . $product->get_id() );
 		$request->set_body_params(
@@ -188,13 +188,13 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertContains( 'Testing', $data['description'] );
-		$this->assertEquals( '8', $data['price'] );
-		$this->assertEquals( '8', $data['sale_price'] );
-		$this->assertEquals( '10', $data['regular_price'] );
-		$this->assertEquals( 'FIXED-SKU', $data['sku'] );
-		$this->assertContains( 'Dr1Bczxq4q', $data['images'][0]['src'] );
-		$this->assertContains( 'test upload image', $data['images'][0]['alt'] );
+		$this->assertContains( 'Testing', $data[ 'description' ] );
+		$this->assertEquals( '8', $data[ 'price' ] );
+		$this->assertEquals( '8', $data[ 'sale_price' ] );
+		$this->assertEquals( '10', $data[ 'regular_price' ] );
+		$this->assertEquals( 'FIXED-SKU', $data[ 'sku' ] );
+		$this->assertContains( 'Dr1Bczxq4q', $data[ 'images' ][ 0 ][ 'src' ] );
+		$this->assertContains( 'test upload image', $data[ 'images' ][ 0 ][ 'alt' ] );
 		$product->delete( true );
 
 		// test variable product (variations are tested in product-variations.php).
@@ -203,7 +203,7 @@ 
 		$data     = $response->get_data();
 
 		foreach ( array( 'small', 'large' ) as $term_name ) {
-			$this->assertContains( $term_name, $data['attributes'][0]['options'] );
+			$this->assertContains( $term_name, $data[ 'attributes' ][ 0 ][ 'options' ] );
 		}
 
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/products/' . $product->get_id() );
@@ -235,8 +235,8 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( array( 'small' ), $data['attributes'][0]['options'] );
-		$this->assertEquals( array( 'red', 'yellow' ), $data['attributes'][1]['options'] );
+		$this->assertEquals( array( 'small' ), $data[ 'attributes' ][ 0 ][ 'options' ] );
+		$this->assertEquals( array( 'red', 'yellow' ), $data[ 'attributes' ][ 1 ][ 'options' ] );
 		$product->delete( true );
 
 		// test external product.
@@ -244,8 +244,8 @@ 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/products/' . $product->get_id() ) );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'Buy external product', $data['button_text'] );
-		$this->assertEquals( 'http://woocommerce.com', $data['external_url'] );
+		$this->assertEquals( 'Buy external product', $data[ 'button_text' ] );
+		$this->assertEquals( 'http://woocommerce.com', $data[ 'external_url' ] );
 
 		$request = new WP_REST_Request( 'PUT', '/wc/v2/products/' . $product->get_id() );
 		$request->set_body_params(
@@ -257,8 +257,8 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'Test API Update', $data['button_text'] );
-		$this->assertEquals( 'http://automattic.com', $data['external_url'] );
+		$this->assertEquals( 'Test API Update', $data[ 'button_text' ] );
+		$this->assertEquals( 'http://automattic.com', $data[ 'external_url' ] );
 	}
 
 	/**
@@ -313,7 +313,7 @@ 
 		);
 		$response          = $this->server->dispatch( $request );
 		$data              = $response->get_data();
-		$shipping_class_id = $data['id'];
+		$shipping_class_id = $data[ 'id' ];
 
 		// Create simple.
 		$request = new WP_REST_Request( 'POST', '/wc/v2/products' );
@@ -329,13 +329,13 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( '10', $data['price'] );
-		$this->assertEquals( '10', $data['regular_price'] );
-		$this->assertTrue( $data['purchasable'] );
-		$this->assertEquals( 'DUMMY SKU SIMPLE API', $data['sku'] );
-		$this->assertEquals( 'Test Simple Product', $data['name'] );
-		$this->assertEquals( 'simple', $data['type'] );
-		$this->assertEquals( $shipping_class_id, $data['shipping_class_id'] );
+		$this->assertEquals( '10', $data[ 'price' ] );
+		$this->assertEquals( '10', $data[ 'regular_price' ] );
+		$this->assertTrue( $data[ 'purchasable' ] );
+		$this->assertEquals( 'DUMMY SKU SIMPLE API', $data[ 'sku' ] );
+		$this->assertEquals( 'Test Simple Product', $data[ 'name' ] );
+		$this->assertEquals( 'simple', $data[ 'type' ] );
+		$this->assertEquals( $shipping_class_id, $data[ 'shipping_class_id' ] );
 
 		// Create external.
 		$request = new WP_REST_Request( 'POST', '/wc/v2/products' );
@@ -352,14 +352,14 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( '10', $data['price'] );
-		$this->assertEquals( '10', $data['regular_price'] );
-		$this->assertFalse( $data['purchasable'] );
-		$this->assertEquals( 'DUMMY SKU EXTERNAL API', $data['sku'] );
-		$this->assertEquals( 'Test External Product', $data['name'] );
-		$this->assertEquals( 'external', $data['type'] );
-		$this->assertEquals( 'Test Button', $data['button_text'] );
-		$this->assertEquals( 'https://wordpress.org', $data['external_url'] );
+		$this->assertEquals( '10', $data[ 'price' ] );
+		$this->assertEquals( '10', $data[ 'regular_price' ] );
+		$this->assertFalse( $data[ 'purchasable' ] );
+		$this->assertEquals( 'DUMMY SKU EXTERNAL API', $data[ 'sku' ] );
+		$this->assertEquals( 'Test External Product', $data[ 'name' ] );
+		$this->assertEquals( 'external', $data[ 'type' ] );
+		$this->assertEquals( 'Test Button', $data[ 'button_text' ] );
+		$this->assertEquals( 'https://wordpress.org', $data[ 'external_url' ] );
 
 		// Create variable.
 		$request = new WP_REST_Request( 'POST', '/wc/v2/products' );
@@ -385,10 +385,10 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertEquals( 'DUMMY SKU VARIABLE API', $data['sku'] );
-		$this->assertEquals( 'Test Variable Product', $data['name'] );
-		$this->assertEquals( 'variable', $data['type'] );
-		$this->assertEquals( array( 'small', 'medium' ), $data['attributes'][0]['options'] );
+		$this->assertEquals( 'DUMMY SKU VARIABLE API', $data[ 'sku' ] );
+		$this->assertEquals( 'Test Variable Product', $data[ 'name' ] );
+		$this->assertEquals( 'variable', $data[ 'type' ] );
+		$this->assertEquals( array( 'small', 'medium' ), $data[ 'attributes' ][ 0 ][ 'options' ] );
 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/products' ) );
 		$products = $response->get_data();
@@ -453,13 +453,13 @@ 
 		$response = $this->server->dispatch( $request );
 		$data     = $response->get_data();
 
-		$this->assertContains( 'Updated description.', $data['update'][0]['description'] );
-		$this->assertEquals( 'DUMMY SKU BATCH TEST 1', $data['create'][0]['sku'] );
-		$this->assertEquals( 'DUMMY SKU BATCH TEST 2', $data['create'][1]['sku'] );
-		$this->assertEquals( 'Test Button', $data['create'][0]['button_text'] );
-		$this->assertEquals( 'external', $data['create'][0]['type'] );
-		$this->assertEquals( 'simple', $data['create'][1]['type'] );
-		$this->assertEquals( $product_2->get_id(), $data['delete'][0]['id'] );
+		$this->assertContains( 'Updated description.', $data[ 'update' ][ 0 ][ 'description' ] );
+		$this->assertEquals( 'DUMMY SKU BATCH TEST 1', $data[ 'create' ][ 0 ][ 'sku' ] );
+		$this->assertEquals( 'DUMMY SKU BATCH TEST 2', $data[ 'create' ][ 1 ][ 'sku' ] );
+		$this->assertEquals( 'Test Button', $data[ 'create' ][ 0 ][ 'button_text' ] );
+		$this->assertEquals( 'external', $data[ 'create' ][ 0 ][ 'type' ] );
+		$this->assertEquals( 'simple', $data[ 'create' ][ 1 ][ 'type' ] );
+		$this->assertEquals( $product_2->get_id(), $data[ 'delete' ][ 0 ][ 'id' ] );
 
 		$request  = new WP_REST_Request( 'GET', '/wc/v2/products' );
 		$response = $this->server->dispatch( $request );
@@ -496,7 +496,7 @@ 
 
 		$this->assertEquals( 4, count( $products ) );
 		foreach ( $products as $product ) {
-			$this->assertEquals( 'publish', $product['status'] );
+			$this->assertEquals( 'publish', $product[ 'status' ] );
 		}
 
 		// Test filtering with status=draft.
@@ -507,7 +507,7 @@ 
 
 		$this->assertEquals( 4, count( $products ) );
 		foreach ( $products as $product ) {
-			$this->assertEquals( 'draft', $product['status'] );
+			$this->assertEquals( 'draft', $product[ 'status' ] );
 		}
 
 		// Test filtering with no filters - which should return 'any' (all 8).
@@ -529,7 +529,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/products/' . $product->get_id() );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 		$this->assertEquals( 65, count( $properties ) );
 	}
 }

unit-tests/Tests/Version2/payment-gateways.php

Spacing +26 added lines, -26 removed lines

@@ -155,9 +155,9 @@ 
 		$response = $this->server->dispatch( new WP_REST_Request( 'GET', '/wc/v2/payment_gateways/paypal' ) );
 		$paypal   = $response->get_data();
 
-		$this->assertEquals( 'PayPal', $paypal['settings']['title']['value'] );
-		$this->assertEquals( 'admin@example.org', $paypal['settings']['email']['value'] );
-		$this->assertEquals( 'no', $paypal['settings']['testmode']['value'] );
+		$this->assertEquals( 'PayPal', $paypal[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertEquals( 'admin@example.org', $paypal[ 'settings' ][ 'email' ][ 'value' ] );
+		$this->assertEquals( 'no', $paypal[ 'settings' ][ 'testmode' ][ 'value' ] );
 
 		// test updating single setting
 		$request = new WP_REST_Request( 'POST', '/wc/v2/payment_gateways/paypal' );
@@ -172,9 +172,9 @@ 
 		$paypal   = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( 'PayPal', $paypal['settings']['title']['value'] );
-		$this->assertEquals( 'woo@woo.local', $paypal['settings']['email']['value'] );
-		$this->assertEquals( 'no', $paypal['settings']['testmode']['value'] );
+		$this->assertEquals( 'PayPal', $paypal[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertEquals( 'woo@woo.local', $paypal[ 'settings' ][ 'email' ][ 'value' ] );
+		$this->assertEquals( 'no', $paypal[ 'settings' ][ 'testmode' ][ 'value' ] );
 
 		// test updating multiple settings
 		$request = new WP_REST_Request( 'POST', '/wc/v2/payment_gateways/paypal' );
@@ -190,9 +190,9 @@ 
 		$paypal   = $response->get_data();
 
 		$this->assertEquals( 200, $response->get_status() );
-		$this->assertEquals( 'PayPal - New Title', $paypal['settings']['title']['value'] );
-		$this->assertEquals( 'woo@woo.local', $paypal['settings']['email']['value'] );
-		$this->assertEquals( 'yes', $paypal['settings']['testmode']['value'] );
+		$this->assertEquals( 'PayPal - New Title', $paypal[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertEquals( 'woo@woo.local', $paypal[ 'settings' ][ 'email' ][ 'value' ] );
+		$this->assertEquals( 'yes', $paypal[ 'settings' ][ 'testmode' ][ 'value' ] );
 
 		// Test other parameters, and recheck settings
 		$request = new WP_REST_Request( 'POST', '/wc/v2/payment_gateways/paypal' );
@@ -205,11 +205,11 @@ 
 		$response = $this->server->dispatch( $request );
 		$paypal   = $response->get_data();
 
-		$this->assertFalse( $paypal['enabled'] );
-		$this->assertEquals( 2, $paypal['order'] );
-		$this->assertEquals( 'PayPal - New Title', $paypal['settings']['title']['value'] );
-		$this->assertEquals( 'woo@woo.local', $paypal['settings']['email']['value'] );
-		$this->assertEquals( 'yes', $paypal['settings']['testmode']['value'] );
+		$this->assertFalse( $paypal[ 'enabled' ] );
+		$this->assertEquals( 2, $paypal[ 'order' ] );
+		$this->assertEquals( 'PayPal - New Title', $paypal[ 'settings' ][ 'title' ][ 'value' ] );
+		$this->assertEquals( 'woo@woo.local', $paypal[ 'settings' ][ 'email' ][ 'value' ] );
+		$this->assertEquals( 'yes', $paypal[ 'settings' ][ 'testmode' ][ 'value' ] );
 
 		// test bogus
 		$request = new WP_REST_Request( 'POST', '/wc/v2/payment_gateways/paypal' );
@@ -233,7 +233,7 @@ 
 		);
 		$response = $this->server->dispatch( $request );
 		$paypal   = $response->get_data();
-		$this->assertEquals( 'authorization', $paypal['settings']['paymentaction']['value'] );
+		$this->assertEquals( 'authorization', $paypal[ 'settings' ][ 'paymentaction' ][ 'value' ] );
 	}
 
 	/**
@@ -284,7 +284,7 @@ 
 		$request    = new WP_REST_Request( 'OPTIONS', '/wc/v2/payment_gateways' );
 		$response   = $this->server->dispatch( $request );
 		$data       = $response->get_data();
-		$properties = $data['schema']['properties'];
+		$properties = $data[ 'schema' ][ 'properties' ];
 
 		$this->assertEquals( 8, count( $properties ) );
 		$this->assertArrayHasKey( 'id', $properties );
@@ -309,25 +309,25 @@ 
 		$gateway->init_form_fields();
 		foreach ( $gateway->form_fields as $id => $field ) {
 			// Make sure we at least have a title and type
-			if ( empty( $field['title'] ) || empty( $field['type'] ) ) {
+			if ( empty( $field[ 'title' ] ) || empty( $field[ 'type' ] ) ) {
 				continue;
 			}
 			// Ignore 'title' settings/fields -- they are UI only
-			if ( 'title' === $field['type'] ) {
+			if ( 'title' === $field[ 'type' ] ) {
 				continue;
 			}
 			$data = array(
 				'id'          => $id,
-				'label'       => empty( $field['label'] ) ? $field['title'] : $field['label'],
-				'description' => empty( $field['description'] ) ? '' : $field['description'],
-				'type'        => $field['type'],
+				'label'       => empty( $field[ 'label' ] ) ? $field[ 'title' ] : $field[ 'label' ],
+				'description' => empty( $field[ 'description' ] ) ? '' : $field[ 'description' ],
+				'type'        => $field[ 'type' ],
 				'value'       => $gateway->settings[ $id ],
-				'default'     => empty( $field['default'] ) ? '' : $field['default'],
-				'tip'         => empty( $field['description'] ) ? '' : $field['description'],
-				'placeholder' => empty( $field['placeholder'] ) ? '' : $field['placeholder'],
+				'default'     => empty( $field[ 'default' ] ) ? '' : $field[ 'default' ],
+				'tip'         => empty( $field[ 'description' ] ) ? '' : $field[ 'description' ],
+				'placeholder' => empty( $field[ 'placeholder' ] ) ? '' : $field[ 'placeholder' ],
 			);
-			if ( ! empty( $field['options'] ) ) {
-				$data['options'] = $field['options'];
+			if ( ! empty( $field[ 'options' ] ) ) {
+				$data[ 'options' ] = $field[ 'options' ];
 			}
 			$settings[ $id ] = $data;
 		}