woocommerce / woocommerce-gateway-stripe

tests/phpunit/test-wc-stripe-sub-renewal.php

Spacing +68 added lines, -68 removed lines

@@ -34,17 +34,17 @@ 
 	public function setUp() {
 		parent::setUp();
 
-		$this->wc_stripe_subs_compat = $this->getMockBuilder( 'WC_Stripe_Subs_Compat' )
+		$this->wc_stripe_subs_compat = $this->getMockBuilder('WC_Stripe_Subs_Compat')
 			->disableOriginalConstructor()
-			->setMethods( array( 'prepare_order_source', 'has_subscription', 'ensure_subscription_has_customer_id' ) )
+			->setMethods(array('prepare_order_source', 'has_subscription', 'ensure_subscription_has_customer_id'))
 			->getMock();
 
 		// Mocked in order to get metadata[payment_type] = recurring in the HTTP request.
 		$this->wc_stripe_subs_compat
-			->expects( $this->any() )
-			->method( 'has_subscription' )
+			->expects($this->any())
+			->method('has_subscription')
 			->will(
-				$this->returnValue( true )
+				$this->returnValue(true)
 			);
 
 		$this->statement_descriptor = 'This is a statement descriptor.';
@@ -62,7 +62,7 @@ 
 	public function tearDown() {
 		parent::tearDown();
 
-		delete_option( 'woocommerce_stripe_settings' );
+		delete_option('woocommerce_stripe_settings');
 	}
 
 	/**
@@ -79,22 +79,22 @@ 
 		// Arrange: Some variables we'll use later.
 		$renewal_order                 = WC_Helper_Order::create_order();
 		$amount                        = 20; // WC Subs sends an amount to be used, instead of using the order amount.
-		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount( $amount );
-		$currency                      = strtolower( $renewal_order->get_currency() );
+		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount($amount);
+		$currency                      = strtolower($renewal_order->get_currency());
 		$customer                      = 'cus_123abc';
 		$source                        = 'src_123abc';
-		$statement_descriptor          = WC_Stripe_Helper::clean_statement_descriptor( $this->statement_descriptor );
+		$statement_descriptor          = WC_Stripe_Helper::clean_statement_descriptor($this->statement_descriptor);
 		$should_retry                  = false;
 		$previous_error                = false;
 		$payments_intents_api_endpoint = 'https://api.stripe.com/v1/payment_intents';
 		$urls_used                     = array();
 
-		$renewal_order->set_payment_method( 'stripe' );
+		$renewal_order->set_payment_method('stripe');
 
 		// Arrange: Mock prepare_order_source() so that we have a customer and source.
 		$this->wc_stripe_subs_compat
-			->expects( $this->any() )
-			->method( 'prepare_order_source' )
+			->expects($this->any())
+			->method('prepare_order_source')
 			->will(
 				$this->returnValue(
 					(object) array(
@@ -108,7 +108,7 @@ 
 
 		// Arrange: Add filter that will return a mocked HTTP response for the payment_intent call.
 		// Note: There are assertions in the callback function.
-		$pre_http_request_response_callback = function( $preempt, $request_args, $url ) use (
+		$pre_http_request_response_callback = function($preempt, $request_args, $url) use (
 			$renewal_order,
 			$stripe_amount,
 			$currency,
@@ -119,35 +119,35 @@ 
 			&$urls_used
 		) {
 			// Add all urls to array so we can later make assertions about which endpoints were used.
-			array_push( $urls_used, $url );
+			array_push($urls_used, $url);
 
 			// Continue without mocking the request if it's not the endpoint we care about.
-			if ( $payments_intents_api_endpoint !== $url ) {
+			if ($payments_intents_api_endpoint !== $url) {
 				return false;
 			}
 
 			// Assert: the request method is POST.
-			$this->assertArrayHasKey( 'method', $request_args );
-			$this->assertSame( 'POST', $request_args['method'] );
+			$this->assertArrayHasKey('method', $request_args);
+			$this->assertSame('POST', $request_args['method']);
 
 			// Assert: the request has a body.
-			$this->assertArrayHasKey( 'body', $request_args );
+			$this->assertArrayHasKey('body', $request_args);
 
 			// Assert: the request body contains these values.
 			$expected_request_body_values = array(
 				'source'               => $source,
 				'amount'               => $stripe_amount,
 				'currency'             => $currency,
-				'payment_method_types' => array( 'card' ),
+				'payment_method_types' => array('card'),
 				'customer'             => $customer,
 				'off_session'          => 'true',
 				'confirm'              => 'true',
 				'confirmation_method'  => 'automatic',
 				'statement_descriptor' => $statement_descriptor,
 			);
-			foreach ( $expected_request_body_values as $key => $value ) {
-				$this->assertArrayHasKey( $key, $request_args['body'] );
-				$this->assertSame( $value, $request_args['body'][ $key ] );
+			foreach ($expected_request_body_values as $key => $value) {
+				$this->assertArrayHasKey($key, $request_args['body']);
+				$this->assertSame($value, $request_args['body'][$key]);
 			}
 
 			// Assert: the request body contains these keys, without checking for their value.
@@ -155,8 +155,8 @@ 
 				'description',
 				'metadata',
 			);
-			foreach ( $expected_request_body_keys as $key ) {
-				$this->assertArrayHasKey( $key, $request_args['body'] );
+			foreach ($expected_request_body_keys as $key) {
+				$this->assertArrayHasKey($key, $request_args['body']);
 			}
 
 			// Assert: the body metadata has these values.
@@ -165,9 +165,9 @@ 
 				'order_id'     => $order_id,
 				'payment_type' => 'recurring',
 			);
-			foreach ( $expected_metadata_values as $key => $value ) {
-				$this->assertArrayHasKey( $key, $request_args['body']['metadata'] );
-				$this->assertSame( $value, $request_args['body']['metadata'][ $key ] );
+			foreach ($expected_metadata_values as $key => $value) {
+				$this->assertArrayHasKey($key, $request_args['body']['metadata']);
+				$this->assertSame($value, $request_args['body']['metadata'][$key]);
 			}
 
 			// Assert: the body metadata has these keys, without checking for their value.
@@ -176,8 +176,8 @@ 
 				'customer_email',
 				'site_url',
 			);
-			foreach ( $expected_metadata_keys as $key ) {
-				$this->assertArrayHasKey( $key, $request_args['body']['metadata'] );
+			foreach ($expected_metadata_keys as $key) {
+				$this->assertArrayHasKey($key, $request_args['body']['metadata']);
 			}
 
 			// Assert: the request body does not contains these keys.
@@ -186,15 +186,15 @@ 
 				'capture_method', // The default ('automatic') is what we want in this case, so we leave it off.
 				'expand[]',
 			);
-			foreach ( $expected_missing_request_body_keys as $key ) {
-				$this->assertArrayNotHasKey( $key, $request_args['body'] );
+			foreach ($expected_missing_request_body_keys as $key) {
+				$this->assertArrayNotHasKey($key, $request_args['body']);
 			}
 
 			// Arrange: return dummy content as the response.
 			return array(
 				'headers'  => array(),
 				// Too bad we aren't dynamically setting things 'cus_123abc' when using this file.
-				'body'     => file_get_contents( 'tests/phpunit/dummy-data/subscription_renewal_response_success.json' ),
+				'body'     => file_get_contents('tests/phpunit/dummy-data/subscription_renewal_response_success.json'),
 				'response' => array(
 					'code'    => 200,
 					'message' => 'OK',
@@ -204,48 +204,48 @@ 
 			);
 		};
 
-		add_filter( 'pre_http_request', $pre_http_request_response_callback, 10, 3 );
+		add_filter('pre_http_request', $pre_http_request_response_callback, 10, 3);
 
 		// Arrange: Make sure to check that an action we care about was called
 		// by hooking into it.
 		$mock_action_process_payment = new MockAction();
 		add_action(
 			'wc_gateway_stripe_process_payment',
-			[ &$mock_action_process_payment, 'action' ]
+			[&$mock_action_process_payment, 'action']
 		);
 
 		// Act: call process_subscription_payment().
 		// We need to use `wc_stripe_subs_compat` here because we mocked this class earlier.
-		$result = $this->wc_stripe_subs_compat->process_subscription_payment( 20, $renewal_order, $should_retry, $previous_error );
+		$result = $this->wc_stripe_subs_compat->process_subscription_payment(20, $renewal_order, $should_retry, $previous_error);
 
 		// Assert: nothing was returned.
-		$this->assertEquals( $result, null );
+		$this->assertEquals($result, null);
 
 		// Assert that we saved the payment intent to the order.
-		$order = wc_get_order( $renewal_order->get_id() );
-		$this->assertNotFalse( $order );
-		$order_data = $order->get_meta( '_stripe_intent_id'	);
+		$order = wc_get_order($renewal_order->get_id());
+		$this->assertNotFalse($order);
+		$order_data = $order->get_meta('_stripe_intent_id');
 
-		$this->assertEquals( $order_data, 'pi_123abc' );
+		$this->assertEquals($order_data, 'pi_123abc');
 
 		// Transaction ID was saved to order.
 		$order_transaction_id = $order->get_transaction_id();
-		$this->assertEquals( $order_transaction_id, 'ch_123abc' );
+		$this->assertEquals($order_transaction_id, 'ch_123abc');
 
 		// Assert: the order was marked as processing (this is done in process_response()).
-		$this->assertEquals( $order->get_status(), 'processing' );
+		$this->assertEquals($order->get_status(), 'processing');
 
 		// Assert: called payment intents.
-		$this->assertTrue( in_array( $payments_intents_api_endpoint, $urls_used ) );
+		$this->assertTrue(in_array($payments_intents_api_endpoint, $urls_used));
 
 		// Assert: Our hook was called once.
-		$this->assertEquals( 1, $mock_action_process_payment->get_call_count() );
+		$this->assertEquals(1, $mock_action_process_payment->get_call_count());
 
 		// Assert: Only our hook was called.
-		$this->assertEquals( array( 'wc_gateway_stripe_process_payment' ), $mock_action_process_payment->get_tags() );
+		$this->assertEquals(array('wc_gateway_stripe_process_payment'), $mock_action_process_payment->get_tags());
 
 		// Clean up.
-		remove_filter( 'pre_http_request', array( $this, 'pre_http_request_response_success' ) );
+		remove_filter('pre_http_request', array($this, 'pre_http_request_response_success'));
 	}
 
 	/**
@@ -262,8 +262,8 @@ 
 		// Arrange: Some variables we'll use later.
 		$renewal_order                 = WC_Helper_Order::create_order();
 		$amount                        = 20;
-		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount( $amount );
-		$currency                      = strtolower( $renewal_order->get_currency() );
+		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount($amount);
+		$currency                      = strtolower($renewal_order->get_currency());
 		$customer                      = 'cus_123abc';
 		$source                        = 'src_123abc';
 		$should_retry                  = false;
@@ -273,8 +273,8 @@ 
 
 		// Arrange: Mock prepare_order_source() so that we have a customer and source.
 		$this->wc_stripe_subs_compat
-			->expects( $this->any() )
-			->method( 'prepare_order_source' )
+			->expects($this->any())
+			->method('prepare_order_source')
 			->will(
 				$this->returnValue(
 					(object) array(
@@ -287,7 +287,7 @@ 
 			);
 
 		// Arrange: Add filter that will return a mocked HTTP response for the payment_intent call.
-		$pre_http_request_response_callback = function( $preempt, $request_args, $url ) use (
+		$pre_http_request_response_callback = function($preempt, $request_args, $url) use (
 			$renewal_order,
 			$stripe_amount,
 			$currency,
@@ -297,10 +297,10 @@ 
 			&$urls_used
 		) {
 			// Add all urls to array so we can later make assertions about which endpoints were used.
-			array_push( $urls_used, $url );
+			array_push($urls_used, $url);
 
 			// Continue without mocking the request if it's not the endpoint we care about.
-			if ( $payments_intents_api_endpoint !== $url ) {
+			if ($payments_intents_api_endpoint !== $url) {
 				return false;
 			}
 
@@ -308,7 +308,7 @@ 
 			return array(
 				'headers'  => array(),
 				// Too bad we aren't dynamically setting things 'cus_123abc' when using this file.
-				'body'     => file_get_contents( 'tests/phpunit/dummy-data/subscription_renewal_response_authentication_required.json' ),
+				'body'     => file_get_contents('tests/phpunit/dummy-data/subscription_renewal_response_authentication_required.json'),
 				'response' => array(
 					'code'    => 402,
 					'message' => 'Payment Required',
@@ -317,48 +317,48 @@ 
 				'filename' => null,
 			);
 		};
-		add_filter( 'pre_http_request', $pre_http_request_response_callback, 10, 3 );
+		add_filter('pre_http_request', $pre_http_request_response_callback, 10, 3);
 
 		// Arrange: Make sure to check that an action we care about was called
 		// by hooking into it.
 		$mock_action_process_payment = new MockAction();
 		add_action(
 			'wc_gateway_stripe_process_payment_authentication_required',
-			[ &$mock_action_process_payment, 'action' ]
+			[&$mock_action_process_payment, 'action']
 		);
 
 		// Act: call process_subscription_payment().
 		// We need to use `wc_stripe_subs_compat` here because we mocked this class earlier.
-		$result = $this->wc_stripe_subs_compat->process_subscription_payment( 20, $renewal_order, $should_retry, $previous_error );
+		$result = $this->wc_stripe_subs_compat->process_subscription_payment(20, $renewal_order, $should_retry, $previous_error);
 
 		// Assert: nothing was returned.
-		$this->assertEquals( $result, null );
+		$this->assertEquals($result, null);
 
 		// Assert that we saved the payment intent to the order.
-		$order                = wc_get_order( $renewal_order->get_id() );
-		$this->assertNotFalse( $order );
-		$order_data           = $order->get_meta( '_stripe_intent_id' );
+		$order                = wc_get_order($renewal_order->get_id());
+		$this->assertNotFalse($order);
+		$order_data           = $order->get_meta('_stripe_intent_id');
 		$order_transaction_id = $order->get_transaction_id();
 
 		// Intent was saved to order even though there was an error in the response body.
-		$this->assertEquals( $order_data, 'pi_123abc' );
+		$this->assertEquals($order_data, 'pi_123abc');
 
 		// Transaction ID was saved to order.
-		$this->assertEquals( $order_transaction_id, 'ch_123abc' );
+		$this->assertEquals($order_transaction_id, 'ch_123abc');
 
 		// Assert: the order was marked as failed.
-		$this->assertEquals( $order->get_status(), 'failed' );
+		$this->assertEquals($order->get_status(), 'failed');
 
 		// Assert: called payment intents.
-		$this->assertTrue( in_array( $payments_intents_api_endpoint, $urls_used ) );
+		$this->assertTrue(in_array($payments_intents_api_endpoint, $urls_used));
 
 		// Assert: Our hook was called once.
-		$this->assertEquals( 1, $mock_action_process_payment->get_call_count() );
+		$this->assertEquals(1, $mock_action_process_payment->get_call_count());
 
 		// Assert: Only our hook was called.
-		$this->assertEquals( array( 'wc_gateway_stripe_process_payment_authentication_required' ), $mock_action_process_payment->get_tags() );
+		$this->assertEquals(array('wc_gateway_stripe_process_payment_authentication_required'), $mock_action_process_payment->get_tags());
 
 		// Clean up.
-		remove_filter( 'pre_http_request', array( $this, 'pre_http_request_response_success' ) );
+		remove_filter('pre_http_request', array($this, 'pre_http_request_response_success'));
 	}
 }

tests/phpunit/helpers/class-wc-helper-product.php

Spacing +61 added lines, -61 removed lines

@@ -17,10 +17,10 @@ 
 	 *
 	 * @param int $product_id ID to delete.
 	 */
-	public static function delete_product( $product_id ) {
-		$product = wc_get_product( $product_id );
-		if ( $product ) {
-			$product->delete( true );
+	public static function delete_product($product_id) {
+		$product = wc_get_product($product_id);
+		if ($product) {
+			$product->delete(true);
 		}
 	}
 
@@ -31,7 +31,7 @@ 
 	 * @param bool $save Save or return object.
 	 * @return WC_Product_Simple
 	 */
-	public static function create_simple_product( $save = true ) {
+	public static function create_simple_product($save = true) {
 		$product = new WC_Product_Simple();
 		$product->set_props(
 			[
@@ -48,9 +48,9 @@ 
 			]
 		);
 
-		if ( $save ) {
+		if ($save) {
 			$product->save();
-			return wc_get_product( $product->get_id() );
+			return wc_get_product($product->get_id());
 		} else {
 			return $product;
 		}
@@ -75,7 +75,7 @@ 
 		);
 		$product->save();
 
-		return wc_get_product( $product->get_id() );
+		return wc_get_product($product->get_id());
 	}
 
 	/**
@@ -94,10 +94,10 @@ 
 				'sku'  => 'DUMMY GROUPED SKU',
 			]
 		);
-		$product->set_children( [ $simple_product_1->get_id(), $simple_product_2->get_id() ] );
+		$product->set_children([$simple_product_1->get_id(), $simple_product_2->get_id()]);
 		$product->save();
 
-		return wc_get_product( $product->get_id() );
+		return wc_get_product($product->get_id());
 	}
 
 	/**
@@ -119,36 +119,36 @@ 
 		$attributes = [];
 
 		$attribute      = new WC_Product_Attribute();
-		$attribute_data = self::create_attribute( 'size', [ 'small', 'large', 'huge' ] );
-		$attribute->set_id( $attribute_data['attribute_id'] );
-		$attribute->set_name( $attribute_data['attribute_taxonomy'] );
-		$attribute->set_options( $attribute_data['term_ids'] );
-		$attribute->set_position( 1 );
-		$attribute->set_visible( true );
-		$attribute->set_variation( true );
+		$attribute_data = self::create_attribute('size', ['small', 'large', 'huge']);
+		$attribute->set_id($attribute_data['attribute_id']);
+		$attribute->set_name($attribute_data['attribute_taxonomy']);
+		$attribute->set_options($attribute_data['term_ids']);
+		$attribute->set_position(1);
+		$attribute->set_visible(true);
+		$attribute->set_variation(true);
 		$attributes[] = $attribute;
 
 		$attribute      = new WC_Product_Attribute();
-		$attribute_data = self::create_attribute( 'colour', [ 'red', 'blue' ] );
-		$attribute->set_id( $attribute_data['attribute_id'] );
-		$attribute->set_name( $attribute_data['attribute_taxonomy'] );
-		$attribute->set_options( $attribute_data['term_ids'] );
-		$attribute->set_position( 1 );
-		$attribute->set_visible( true );
-		$attribute->set_variation( true );
+		$attribute_data = self::create_attribute('colour', ['red', 'blue']);
+		$attribute->set_id($attribute_data['attribute_id']);
+		$attribute->set_name($attribute_data['attribute_taxonomy']);
+		$attribute->set_options($attribute_data['term_ids']);
+		$attribute->set_position(1);
+		$attribute->set_visible(true);
+		$attribute->set_variation(true);
 		$attributes[] = $attribute;
 
 		$attribute      = new WC_Product_Attribute();
-		$attribute_data = self::create_attribute( 'number', [ '0', '1', '2' ] );
-		$attribute->set_id( $attribute_data['attribute_id'] );
-		$attribute->set_name( $attribute_data['attribute_taxonomy'] );
-		$attribute->set_options( $attribute_data['term_ids'] );
-		$attribute->set_position( 1 );
-		$attribute->set_visible( true );
-		$attribute->set_variation( true );
+		$attribute_data = self::create_attribute('number', ['0', '1', '2']);
+		$attribute->set_id($attribute_data['attribute_id']);
+		$attribute->set_name($attribute_data['attribute_taxonomy']);
+		$attribute->set_options($attribute_data['term_ids']);
+		$attribute->set_position(1);
+		$attribute->set_visible(true);
+		$attribute->set_variation(true);
 		$attributes[] = $attribute;
 
-		$product->set_attributes( $attributes );
+		$product->set_attributes($attributes);
 		$product->save();
 
 		$variation_1 = new WC_Product_Variation();
@@ -159,7 +159,7 @@ 
 				'regular_price' => 10,
 			]
 		);
-		$variation_1->set_attributes( [ 'pa_size' => 'small' ] );
+		$variation_1->set_attributes(['pa_size' => 'small']);
 		$variation_1->save();
 
 		$variation_2 = new WC_Product_Variation();
@@ -170,7 +170,7 @@ 
 				'regular_price' => 15,
 			]
 		);
-		$variation_2->set_attributes( [ 'pa_size' => 'large' ] );
+		$variation_2->set_attributes(['pa_size' => 'large']);
 		$variation_2->save();
 
 		$variation_3 = new WC_Product_Variation();
@@ -207,7 +207,7 @@ 
 		);
 		$variation_4->save();
 
-		return wc_get_product( $product->get_id() );
+		return wc_get_product($product->get_id());
 	}
 
 	/**
@@ -219,30 +219,30 @@ 
 	 * @param array(string) $terms          Terms to create for the attribute.
 	 * @return array
 	 */
-	public static function create_attribute( $raw_name = 'size', $terms = [ 'small' ] ) {
+	public static function create_attribute($raw_name = 'size', $terms = ['small']) {
 		global $wpdb, $wc_product_attributes;
 
 		// Make sure caches are clean.
-		delete_transient( 'wc_attribute_taxonomies' );
-		if ( is_callable( [ 'WC_Cache_Helper', 'invalidate_cache_group' ] ) ) {
+		delete_transient('wc_attribute_taxonomies');
+		if (is_callable(['WC_Cache_Helper', 'invalidate_cache_group'])) {
 			WC_Cache_Helper::invalidate_cache_group('woocommerce-attributes');
 		}
 
 		// These are exported as labels, so convert the label to a name if possible first.
-		$attribute_labels = wp_list_pluck( wc_get_attribute_taxonomies(), 'attribute_label', 'attribute_name' );
-		$attribute_name   = array_search( $raw_name, $attribute_labels, true );
+		$attribute_labels = wp_list_pluck(wc_get_attribute_taxonomies(), 'attribute_label', 'attribute_name');
+		$attribute_name   = array_search($raw_name, $attribute_labels, true);
 
-		if ( ! $attribute_name ) {
-			$attribute_name = wc_sanitize_taxonomy_name( $raw_name );
+		if ( ! $attribute_name) {
+			$attribute_name = wc_sanitize_taxonomy_name($raw_name);
 		}
 
-		$attribute_id = wc_attribute_taxonomy_id_by_name( $attribute_name );
+		$attribute_id = wc_attribute_taxonomy_id_by_name($attribute_name);
 
-		if ( ! $attribute_id ) {
-			$taxonomy_name = wc_attribute_taxonomy_name( $attribute_name );
+		if ( ! $attribute_id) {
+			$taxonomy_name = wc_attribute_taxonomy_name($attribute_name);
 
 			// Degister taxonomy which other tests may have created...
-			unregister_taxonomy( $taxonomy_name );
+			unregister_taxonomy($taxonomy_name);
 
 			$attribute_id = wc_create_attribute(
 				[
@@ -257,7 +257,7 @@ 
 			// Register as taxonomy.
 			register_taxonomy(
 				$taxonomy_name,
-				apply_filters( 'woocommerce_taxonomy_objects_' . $taxonomy_name, [ 'product' ] ),
+				apply_filters('woocommerce_taxonomy_objects_' . $taxonomy_name, ['product']),
 				apply_filters(
 					'woocommerce_taxonomy_args_' . $taxonomy_name,
 					[
@@ -275,12 +275,12 @@ 
 			// Set product attributes global.
 			$wc_product_attributes = [];
 
-			foreach ( wc_get_attribute_taxonomies() as $taxonomy ) {
-				$wc_product_attributes[ wc_attribute_taxonomy_name( $taxonomy->attribute_name ) ] = $taxonomy;
+			foreach (wc_get_attribute_taxonomies() as $taxonomy) {
+				$wc_product_attributes[wc_attribute_taxonomy_name($taxonomy->attribute_name)] = $taxonomy;
 			}
 		}
 
-		$attribute = wc_get_attribute( $attribute_id );
+		$attribute = wc_get_attribute($attribute_id);
 		$return    = [
 			'attribute_name'     => $attribute->name,
 			'attribute_taxonomy' => $attribute->slug,
@@ -288,11 +288,11 @@ 
 			'term_ids'           => [],
 		];
 
-		foreach ( $terms as $term ) {
-			$result = term_exists( $term, $attribute->slug );
+		foreach ($terms as $term) {
+			$result = term_exists($term, $attribute->slug);
 
-			if ( ! $result ) {
-				$result               = wp_insert_term( $term, $attribute->slug );
+			if ( ! $result) {
+				$result               = wp_insert_term($term, $attribute->slug);
 				$return['term_ids'][] = $result['term_id'];
 			} else {
 				$return['term_ids'][] = $result['term_id'];
@@ -309,13 +309,13 @@ 
 	 *
 	 * @since 2.3
 	 */
-	public static function delete_attribute( $attribute_id ) {
+	public static function delete_attribute($attribute_id) {
 		global $wpdb;
 
-		$attribute_id = absint( $attribute_id );
+		$attribute_id = absint($attribute_id);
 
 		$wpdb->query(
-			$wpdb->prepare( "DELETE FROM {$wpdb->prefix}woocommerce_attribute_taxonomies WHERE attribute_id = %d", $attribute_id )
+			$wpdb->prepare("DELETE FROM {$wpdb->prefix}woocommerce_attribute_taxonomies WHERE attribute_id = %d", $attribute_id)
 		);
 	}
 
@@ -327,7 +327,7 @@ 
 	 * @param string $review_content string Content to use for the product review.
 	 * @return integer Product Review ID.
 	 */
-	public static function create_product_review( $product_id, $review_content = 'Review content here' ) {
+	public static function create_product_review($product_id, $review_content = 'Review content here') {
 		$data = [
 			'comment_post_ID'      => $product_id,
 			'comment_author'       => 'admin',
@@ -338,7 +338,7 @@ 
 			'comment_approved'     => 1,
 			'comment_type'         => 'review',
 		];
-		return wp_insert_comment( $data );
+		return wp_insert_comment($data);
 	}
 
 	/**
@@ -347,7 +347,7 @@ 
 	 *
 	 * @param int $id ID to update.
 	 */
-	public static function save_post_test_update_meta_data_direct( $id ) {
-		update_post_meta( $id, '_test2', 'world' );
+	public static function save_post_test_update_meta_data_direct($id) {
+		update_post_meta($id, '_test2', 'world');
 	}
 }