Commit 66641977227 for woocommerce
commit 6664197722726c4b69109097ef485bba1cfdf395
Author: Vlad Olaru <vlad.olaru@automattic.com>
Date: Sat Nov 14 09:27:24 2026 +0200
[tests] Drop hook removals that tear-down already undoes (#70944)
test: Drop hook removals that tear-down already undoes
WordPress's test case restores its hook snapshot inside
parent::tearDown(). A remove_filter(), remove_action() or
remove_all_*() call in a test class's own tearDown(), right before
the parent call, is overwritten by that restore, so it does nothing.
A removal after the parent call runs on the restored snapshot, where
the test's own callbacks no longer exist, so it does nothing either.
Delete 125 such calls in tearDown() methods and the 28 tearDown()
methods left with only the parent call. Kept: removals of hooks that
WC_Unit_Test_Case::tearDown() can still reach before the restore
(cart fees, gettext, shipping filters), one after-parent removal of a
hook the class also adds at class level, and one whose hook name is a
constant.
One of the deleted calls was not a no-op: test-wc-cancel-unpaid-
orders.php cleared woocommerce_cancel_unpaid_orders after the parent
call, which stripped WooCommerce's own wc_cancel_unpaid_orders
callback from the next test.
Also drop a remove_all_filters() in DataRegeneratorTest whose hook
name had a leading space, so it never removed anything.
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
diff --git a/plugins/woocommerce/changelog/test-drop-noop-hook-removals b/plugins/woocommerce/changelog/test-drop-noop-hook-removals
new file mode 100644
index 00000000000..1faf21cc28f
--- /dev/null
+++ b/plugins/woocommerce/changelog/test-drop-noop-hook-removals
@@ -0,0 +1,3 @@
+Significance: patch
+Type: dev
+Comment: Remove hook removals in test tear-downs that the base test case already undoes; no production change.
diff --git a/plugins/woocommerce/tests/legacy/includes/wp-http-testcase.php b/plugins/woocommerce/tests/legacy/includes/wp-http-testcase.php
index 44ed55b82ce..891a30e3b31 100644
--- a/plugins/woocommerce/tests/legacy/includes/wp-http-testcase.php
+++ b/plugins/woocommerce/tests/legacy/includes/wp-http-testcase.php
@@ -201,9 +201,6 @@ abstract class WP_HTTP_TestCase extends WP_UnitTestCase {
parent::tearDown();
- remove_filter( 'pre_http_request', array( $this, 'http_request_listener' ) );
- remove_filter( 'wp_handle_sideload_overrides', array( $this, 'set_sample_image_unique_filename_callback' ) );
-
$this->skip_cache_next = false;
$this->sample_image_download_pending = false;
}
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/admin/class-wc-tests-admin-dashboard.php b/plugins/woocommerce/tests/legacy/unit-tests/admin/class-wc-tests-admin-dashboard.php
index b04065939f6..037eff57ff2 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/admin/class-wc-tests-admin-dashboard.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/admin/class-wc-tests-admin-dashboard.php
@@ -27,14 +27,6 @@ class WC_Tests_Admin_Dashboard extends WC_Unit_Test_Case {
add_filter( 'rest_pre_dispatch', array( $this, 'mock_rest_responses' ), 10, 3 );
}
- /**
- * Tear down.
- */
- public function tearDown(): void {
- parent::tearDown();
- remove_filter( 'rest_pre_dispatch', array( $this, 'mock_rest_responses' ), 10 );
- }
-
/**
* Test: recent reviews widget placeholder.
*/
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/core/test-wc-cancel-unpaid-orders.php b/plugins/woocommerce/tests/legacy/unit-tests/core/test-wc-cancel-unpaid-orders.php
index 167478e5d73..ea4d7c60f67 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/core/test-wc-cancel-unpaid-orders.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/core/test-wc-cancel-unpaid-orders.php
@@ -340,7 +340,5 @@ class WC_Tests_Cancel_Unpaid_Orders extends WC_Unit_Test_Case {
// Remove any filters that might have been added.
remove_all_filters( 'woocommerce_cancel_unpaid_orders_interval_minutes' );
- remove_all_filters( 'woocommerce_cancel_unpaid_orders' );
- remove_all_filters( 'woocommerce_cancel_unpaid_order' );
}
}
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/integrations/maxmind-geolocation/class-wc-tests-maxmind-integration.php b/plugins/woocommerce/tests/legacy/unit-tests/integrations/maxmind-geolocation/class-wc-tests-maxmind-integration.php
index c72777baf40..94e78b83c9a 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/integrations/maxmind-geolocation/class-wc-tests-maxmind-integration.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/integrations/maxmind-geolocation/class-wc-tests-maxmind-integration.php
@@ -39,9 +39,6 @@ class WC_Tests_MaxMind_Integration extends WC_Unit_Test_Case {
public function tearDown(): void {
unset( $GLOBALS['wp_filesystem'] );
- remove_filter( 'filesystem_method', array( $this, 'override_filesystem_method' ) );
- remove_filter( 'woocommerce_maxmind_geolocation_database_service', array( $this, 'override_integration_service' ) );
-
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/order-items/class-wc-tests-order-item.php b/plugins/woocommerce/tests/legacy/unit-tests/order-items/class-wc-tests-order-item.php
index 2fac5b8250b..e811e0b7fbb 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/order-items/class-wc-tests-order-item.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/order-items/class-wc-tests-order-item.php
@@ -55,8 +55,6 @@ class WC_Tests_Base_Order_Item extends WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
$this->disable_cogs_feature();
-
- remove_all_filters( 'woocommerce_calculated_order_item_cogs_value' );
}
/**
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/order/class-wc-tests-crud-orders.php b/plugins/woocommerce/tests/legacy/unit-tests/order/class-wc-tests-crud-orders.php
index 9b99c8b4aba..f6d41879207 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/order/class-wc-tests-crud-orders.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/order/class-wc-tests-crud-orders.php
@@ -17,16 +17,6 @@ use Automattic\WooCommerce\Testing\Tools\CodeHacking\Hacks\FunctionsMockerHack;
* @package WooCommerce\Tests\CRUD
*/
class WC_Tests_CRUD_Orders extends WC_Unit_Test_Case {
- /**
- * Tear down the test class.
- */
- public function tearDown(): void {
- parent::tearDown();
-
- remove_all_filters( 'wc_get_price_thousand_separator' );
- remove_all_filters( 'wc_get_price_decimal_separator' );
- }
-
/**
* Test: get_type
*/
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/api/reports-customers.php b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/api/reports-customers.php
index a7570b171b6..74e0c05b5cc 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/api/reports-customers.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/api/reports-customers.php
@@ -45,14 +45,6 @@ class WC_Admin_Tests_API_Reports_Customers extends WC_REST_Unit_Test_Case {
add_filter( 'woocommerce_analytics_report_should_use_cache', '__return_false' );
}
- /**
- * Clean up after tests.
- */
- public function tearDown(): void {
- remove_filter( 'woocommerce_analytics_report_should_use_cache', '__return_false' );
- parent::tearDown();
- }
-
/**
* Test route registration.
*
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/batch-queue.php b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/batch-queue.php
index 99edc803bd9..c4330ba5d25 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/batch-queue.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/batch-queue.php
@@ -73,7 +73,6 @@ class WC_Admin_Tests_Reports_Regenerate_Batching extends WC_REST_Unit_Test_Case
CustomersScheduler::set_queue( null );
OrdersScheduler::set_queue( null );
$this->queue->actions = array();
- remove_filter( 'woocommerce_admin_scheduler_batch_size', array( $this, 'filter_batch_size' ), 10, 3 );
}
/**
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/onboarding-tasks/tasks/shipping.php b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/onboarding-tasks/tasks/shipping.php
index 5e52403e304..e042010bb13 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/onboarding-tasks/tasks/shipping.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/onboarding-tasks/tasks/shipping.php
@@ -46,7 +46,6 @@ class WC_Admin_Tests_OnboardingTasks_Task_Shipping extends WC_Unit_Test_Case {
*/
public function tearDown(): void {
parent::tearDown();
- remove_filter( 'woocommerce_admin_features', array( $this, 'turn_on_smart_shipping_defaults_feature' ), 1 );
delete_option( OnboardingProfile::DATA_OPTION );
}
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/onboarding-tasks/tasks/woocommerce-payments.php b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/onboarding-tasks/tasks/woocommerce-payments.php
index cdca0bc9fda..75b18b7f625 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/onboarding-tasks/tasks/woocommerce-payments.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/onboarding-tasks/tasks/woocommerce-payments.php
@@ -38,14 +38,6 @@ class WC_Admin_Tests_OnboardingTasks_Task_WooCommerce_Payments extends WC_Unit_T
$this->fake_gateway = new Fake_WC_Payments_Gateway();
}
- /**
- * Teardown after each test.
- */
- public function tearDown(): void {
- parent::tearDown();
- remove_filter( 'woocommerce_payment_gateways', array( $this, 'inject_fake_gateway' ) );
- }
-
/**
* Test is_connected method when WooCommerce Payments is connected.
*/
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/payment-gateway-suggestions/evaluate-suggestion.php b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/payment-gateway-suggestions/evaluate-suggestion.php
index a0a50ac1cb7..de247d24969 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/payment-gateway-suggestions/evaluate-suggestion.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/payment-gateway-suggestions/evaluate-suggestion.php
@@ -39,7 +39,6 @@ class WC_Admin_Tests_PaymentGatewaySuggestions_EvaluateSuggestion extends WC_Uni
*/
public function tearDown(): void {
delete_option( self::MOCK_OPTION );
- remove_filter( 'woocommerce_logging_class', array( $this, 'override_wc_logger' ) );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/payment-gateway-suggestions/payment-gateway-suggestions.php b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/payment-gateway-suggestions/payment-gateway-suggestions.php
index bb2ef0f93f1..d088b227e82 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/payment-gateway-suggestions/payment-gateway-suggestions.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/features/payment-gateway-suggestions/payment-gateway-suggestions.php
@@ -46,7 +46,6 @@ class WC_Admin_Tests_PaymentGatewaySuggestions_Init extends WC_Unit_Test_Case {
*/
public function tearDown(): void {
PaymentGatewaySuggestions::delete_specs_transient();
- remove_all_filters( 'transient_woocommerce_admin_' . PaymentGatewaySuggestionsDataSourcePoller::ID . '_specs' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/remote-inbox-notifications/remote-inbox-notifications-engine.php b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/remote-inbox-notifications/remote-inbox-notifications-engine.php
index 11238bc0e6e..d0af9632d95 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/remote-inbox-notifications/remote-inbox-notifications-engine.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/remote-inbox-notifications/remote-inbox-notifications-engine.php
@@ -64,7 +64,6 @@ class WC_Admin_Tests_RemoteInboxNotifications_RemoteInboxNotificationsEngine ext
public function tearDown(): void {
parent::tearDown();
delete_transient( 'woocommerce_admin_' . RemoteInboxNotificationsDataSourcePoller::ID . '_specs' );
- remove_all_filters( 'transient_woocommerce_admin_' . RemoteInboxNotificationsDataSourcePoller::ID . '_specs' );
}
diff --git a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/wc-admin-assets.php b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/wc-admin-assets.php
index 6a1ef2f4a4a..ed3272389c1 100644
--- a/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/wc-admin-assets.php
+++ b/plugins/woocommerce/tests/legacy/unit-tests/woocommerce-admin/wc-admin-assets.php
@@ -22,15 +22,6 @@ class WC_Admin_Tests_WCAdminAssets extends WP_UnitTestCase {
add_filter( 'woocommerce_admin_features', array( $this, 'turn_on_minified_js_feature' ), 20, 1 );
}
- /**
- * Tear Down
- */
- public function tearDown(): void {
- remove_filter( 'woocommerce_admin_features', array( $this, 'turn_on_minified_js_feature' ), 20 );
-
- parent::tearDown();
- }
-
/**
* Filter to enable the minified-js feature.
*
diff --git a/plugins/woocommerce/tests/php/includes/abstracts/class-wc-abstract-product-test.php b/plugins/woocommerce/tests/php/includes/abstracts/class-wc-abstract-product-test.php
index 29c06badd54..d4835b54f20 100644
--- a/plugins/woocommerce/tests/php/includes/abstracts/class-wc-abstract-product-test.php
+++ b/plugins/woocommerce/tests/php/includes/abstracts/class-wc-abstract-product-test.php
@@ -17,7 +17,6 @@ class WC_Abstract_Product_Test extends WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
$this->disable_cogs_feature();
- remove_all_filters( 'woocommerce_get_cogs_total_value' );
self::disable_direct_product_attribute_lookup_updates();
}
diff --git a/plugins/woocommerce/tests/php/includes/admin/helper/class-wc-helper-subscriptions-api-test.php b/plugins/woocommerce/tests/php/includes/admin/helper/class-wc-helper-subscriptions-api-test.php
index 674a46bed24..eeb7dc48c5b 100644
--- a/plugins/woocommerce/tests/php/includes/admin/helper/class-wc-helper-subscriptions-api-test.php
+++ b/plugins/woocommerce/tests/php/includes/admin/helper/class-wc-helper-subscriptions-api-test.php
@@ -45,8 +45,6 @@ class WC_Helper_Subscriptions_API_Test extends \WC_Unit_Test_Case {
deactivate_plugins( self::INACTIVE_PLUGIN_SLUG . '/' . self::INACTIVE_PLUGIN_SLUG . '.php' );
switch_theme( $this->original_stylesheet );
wp_set_current_user( 0 );
- remove_all_filters( 'wp_die_ajax_handler' );
- remove_all_filters( 'wp_doing_ajax' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/includes/class-wc-cart-test.php b/plugins/woocommerce/tests/php/includes/class-wc-cart-test.php
index f7408e24286..7381d3d705d 100644
--- a/plugins/woocommerce/tests/php/includes/class-wc-cart-test.php
+++ b/plugins/woocommerce/tests/php/includes/class-wc-cart-test.php
@@ -47,8 +47,6 @@ class WC_Cart_Test extends \WC_Unit_Test_Case {
foreach ( array( 'shipping_method_counts', 'previous_shipping_methods', 'shipping_for_package_0', 'shipping_for_package_1', 'chosen_shipping_methods' ) as $key ) {
WC()->session->set( $key, null );
}
-
- remove_filter( 'woocommerce_add_to_cart_quantity', array( $this, 'capture_add_to_cart_quantity_filter_args' ), 10 );
}
/**
diff --git a/plugins/woocommerce/tests/php/includes/class-wc-order-note-test.php b/plugins/woocommerce/tests/php/includes/class-wc-order-note-test.php
index 67489b6fbf9..869c58672e2 100644
--- a/plugins/woocommerce/tests/php/includes/class-wc-order-note-test.php
+++ b/plugins/woocommerce/tests/php/includes/class-wc-order-note-test.php
@@ -56,7 +56,6 @@ class WC_Order_Note_Test extends \WC_Unit_Test_Case {
public function tearDown(): void {
$this->clean_up_cot_setup();
$this->toggle_cot_feature_and_usage( $this->prev_hpos_enabled );
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/includes/class-wc-orders-tracking-test.php b/plugins/woocommerce/tests/php/includes/class-wc-orders-tracking-test.php
index ff8ed1188e0..91d91d01d12 100644
--- a/plugins/woocommerce/tests/php/includes/class-wc-orders-tracking-test.php
+++ b/plugins/woocommerce/tests/php/includes/class-wc-orders-tracking-test.php
@@ -75,7 +75,6 @@ class WC_Orders_Tracking_Test extends \WC_Unit_Test_Case {
$this->clean_up_cot_setup();
$this->toggle_cot_feature_and_usage( $this->prev_hpos_enabled );
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/includes/class-wc-shipping-zones-test.php b/plugins/woocommerce/tests/php/includes/class-wc-shipping-zones-test.php
index 2fd5b6c3983..46b4308a055 100644
--- a/plugins/woocommerce/tests/php/includes/class-wc-shipping-zones-test.php
+++ b/plugins/woocommerce/tests/php/includes/class-wc-shipping-zones-test.php
@@ -27,7 +27,6 @@ class WC_Shipping_Zones_Test extends WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
- remove_filter( 'woocommerce_valid_location_types', array( $this, 'add_custom_location_type' ) );
WC_Helper_Shipping_Zones::remove_mock_zones();
}
diff --git a/plugins/woocommerce/tests/php/includes/data-stores/class-wc-customer-data-store-test.php b/plugins/woocommerce/tests/php/includes/data-stores/class-wc-customer-data-store-test.php
index e01990ff9d9..a57f6457c5b 100644
--- a/plugins/woocommerce/tests/php/includes/data-stores/class-wc-customer-data-store-test.php
+++ b/plugins/woocommerce/tests/php/includes/data-stores/class-wc-customer-data-store-test.php
@@ -33,15 +33,6 @@ class WC_Customer_Data_Store_CPT_Test extends WC_Unit_Test_Case {
add_filter( 'wc_allow_changing_orders_storage_while_sync_is_pending', '__return_true' );
}
- /**
- * Runs after each test.
- */
- public function tearDown(): void {
- remove_filter( 'wc_allow_changing_orders_storage_while_sync_is_pending', '__return_true' );
-
- parent::tearDown();
- }
-
/**
* Test that metadata cannot overwrite customer's column data.
*
diff --git a/plugins/woocommerce/tests/php/includes/data-stores/class-wc-order-item-data-store-test.php b/plugins/woocommerce/tests/php/includes/data-stores/class-wc-order-item-data-store-test.php
index 48b5bffe23f..4c3f7f6f107 100644
--- a/plugins/woocommerce/tests/php/includes/data-stores/class-wc-order-item-data-store-test.php
+++ b/plugins/woocommerce/tests/php/includes/data-stores/class-wc-order-item-data-store-test.php
@@ -35,9 +35,6 @@ class WC_Order_Item_Data_Store_Test extends WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
$this->disable_cogs_feature();
-
- remove_all_filters( 'woocommerce_save_order_item_cogs_value' );
- remove_all_filters( 'woocommerce_load_order_item_cogs_value' );
}
/**
diff --git a/plugins/woocommerce/tests/php/includes/data-stores/class-wc-product-data-store-cpt-test.php b/plugins/woocommerce/tests/php/includes/data-stores/class-wc-product-data-store-cpt-test.php
index 2c2c2203097..b03f49db272 100644
--- a/plugins/woocommerce/tests/php/includes/data-stores/class-wc-product-data-store-cpt-test.php
+++ b/plugins/woocommerce/tests/php/includes/data-stores/class-wc-product-data-store-cpt-test.php
@@ -14,8 +14,6 @@ class WC_Product_Data_Store_CPT_Test extends WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
$this->disable_cogs_feature();
- remove_all_filters( 'woocommerce_load_cogs_value' );
- remove_all_filters( 'woocommerce_save_cogs_value' );
}
/**
diff --git a/plugins/woocommerce/tests/php/includes/data-stores/class-wc-product-variations-data-store-cpt-test.php b/plugins/woocommerce/tests/php/includes/data-stores/class-wc-product-variations-data-store-cpt-test.php
index f8695a2b33a..54cdb6087d2 100644
--- a/plugins/woocommerce/tests/php/includes/data-stores/class-wc-product-variations-data-store-cpt-test.php
+++ b/plugins/woocommerce/tests/php/includes/data-stores/class-wc-product-variations-data-store-cpt-test.php
@@ -32,8 +32,6 @@ class WC_Product_Variation_Data_Store_CPT_Test extends WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
$this->disable_cogs_feature();
- remove_all_filters( 'woocommerce_load_product_cogs_is_additive_flag' );
- remove_all_filters( 'woocommerce_save_product_cogs_is_additive_flag' );
}
/**
diff --git a/plugins/woocommerce/tests/php/includes/emails/class-wc-email-header-template-test.php b/plugins/woocommerce/tests/php/includes/emails/class-wc-email-header-template-test.php
index d76cac1469f..dc1b96a1498 100644
--- a/plugins/woocommerce/tests/php/includes/emails/class-wc-email-header-template-test.php
+++ b/plugins/woocommerce/tests/php/includes/emails/class-wc-email-header-template-test.php
@@ -15,7 +15,6 @@ class WC_Email_Header_Template_Test extends \WC_Unit_Test_Case {
parent::tearDown();
delete_option( 'woocommerce_email_header_image' );
update_option( 'woocommerce_feature_email_improvements_enabled', 'no' );
- remove_all_filters( 'woocommerce_email_header_image_url' );
}
/**
diff --git a/plugins/woocommerce/tests/php/includes/importer/class-wc-product-csv-importer-test.php b/plugins/woocommerce/tests/php/includes/importer/class-wc-product-csv-importer-test.php
index ade454ce6a7..2b17d17fe2b 100644
--- a/plugins/woocommerce/tests/php/includes/importer/class-wc-product-csv-importer-test.php
+++ b/plugins/woocommerce/tests/php/includes/importer/class-wc-product-csv-importer-test.php
@@ -26,15 +26,6 @@ class WC_Product_CSV_Importer_Test extends \WC_Unit_Test_Case {
require_once $bootstrap->plugin_dir . '/includes/admin/importers/class-wc-product-csv-importer-controller.php';
}
- /**
- * Clean up after each test.
- */
- public function tearDown(): void {
- remove_all_filters( 'wc_get_price_decimal_separator' );
-
- parent::tearDown();
- }
-
/**
* @testdox An update import resolves relationship SKUs saved by earlier rows in the same import.
* @testWith [-1]
diff --git a/plugins/woocommerce/tests/php/includes/rest-api/Controllers/Version3/class-wc-rest-orders-controller-tests.php b/plugins/woocommerce/tests/php/includes/rest-api/Controllers/Version3/class-wc-rest-orders-controller-tests.php
index b438d379eeb..fc6ab0704f1 100644
--- a/plugins/woocommerce/tests/php/includes/rest-api/Controllers/Version3/class-wc-rest-orders-controller-tests.php
+++ b/plugins/woocommerce/tests/php/includes/rest-api/Controllers/Version3/class-wc-rest-orders-controller-tests.php
@@ -71,7 +71,6 @@ class WC_REST_Orders_Controller_Tests extends WC_REST_Unit_Test_Case {
unregister_post_type( 'shop_test' );
$this->toggle_cot_feature_and_usage( $this->cot_state );
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/includes/rest-api/Controllers/Version4/ShippingZones/class-wc-rest-shipping-zones-v4-controller-tests.php b/plugins/woocommerce/tests/php/includes/rest-api/Controllers/Version4/ShippingZones/class-wc-rest-shipping-zones-v4-controller-tests.php
index 6b1dbe19b9a..f8a066ef047 100644
--- a/plugins/woocommerce/tests/php/includes/rest-api/Controllers/Version4/ShippingZones/class-wc-rest-shipping-zones-v4-controller-tests.php
+++ b/plugins/woocommerce/tests/php/includes/rest-api/Controllers/Version4/ShippingZones/class-wc-rest-shipping-zones-v4-controller-tests.php
@@ -100,7 +100,6 @@ class WC_REST_Shipping_Zones_V4_Controller_Tests extends WC_Unit_Test_Case {
$this->zones = array();
$this->clear_rest_server();
- remove_filter( 'woocommerce_admin_features', array( self::class, 'enable_rest_api_v4_feature' ) );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/includes/tracks/class-wc-tracks-client-test.php b/plugins/woocommerce/tests/php/includes/tracks/class-wc-tracks-client-test.php
index ad0691162be..580f1be63d5 100644
--- a/plugins/woocommerce/tests/php/includes/tracks/class-wc-tracks-client-test.php
+++ b/plugins/woocommerce/tests/php/includes/tracks/class-wc-tracks-client-test.php
@@ -41,7 +41,6 @@ class WC_Tracks_Client_Test extends \WC_Unit_Test_Case {
*/
public function tearDown(): void {
$this->reset_batch_state();
- remove_filter( 'pre_http_request', array( $this, 'intercept_http_requests' ), 10 );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/includes/tracks/events/class-wc-extensions-tracking-test.php b/plugins/woocommerce/tests/php/includes/tracks/events/class-wc-extensions-tracking-test.php
index f299640b438..d57134633dd 100644
--- a/plugins/woocommerce/tests/php/includes/tracks/events/class-wc-extensions-tracking-test.php
+++ b/plugins/woocommerce/tests/php/includes/tracks/events/class-wc-extensions-tracking-test.php
@@ -62,7 +62,6 @@ class WC_Extensions_Tracking_Test extends \WC_Unit_Test_Case {
*/
public function tearDown(): void {
$_GET = $this->original_get;
- remove_filter( 'woocommerce_tracks_event_properties', array( $this, 'record_event_properties' ), 10 );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/includes/wc-attribute-functions-test.php b/plugins/woocommerce/tests/php/includes/wc-attribute-functions-test.php
index fb2eec7f13e..db9459460b6 100644
--- a/plugins/woocommerce/tests/php/includes/wc-attribute-functions-test.php
+++ b/plugins/woocommerce/tests/php/includes/wc-attribute-functions-test.php
@@ -42,16 +42,6 @@ class WC_Attribute_Functions_Test extends \WC_Unit_Test_Case {
add_filter( 'sanitize_taxonomy_name', $filter_mock );
}
- /**
- * Tear down.
- */
- public function tearDown(): void {
- remove_all_filters( 'woocommerce_attribute_taxonomies' );
- remove_all_filters( 'sanitize_taxonomy_name' );
-
- parent::tearDown();
- }
-
/**
* @testdox wc_is_attribute_in_product_name() should only search the attributes area of a variation name when the variation is given.
*/
diff --git a/plugins/woocommerce/tests/php/includes/wc-user-functions-test.php b/plugins/woocommerce/tests/php/includes/wc-user-functions-test.php
index dea9cba1b09..7a20949bf88 100644
--- a/plugins/woocommerce/tests/php/includes/wc-user-functions-test.php
+++ b/plugins/woocommerce/tests/php/includes/wc-user-functions-test.php
@@ -53,7 +53,6 @@ class WC_User_Functions_Tests extends WC_Unit_Test_Case {
public function tearDown(): void {
$this->clean_up_cot_setup();
$this->toggle_cot_feature_and_usage( $this->previous_hpos_state );
- remove_filter( 'wc_allow_changing_orders_storage_while_sync_is_pending', '__return_true' );
parent::tearDown();
// In case `wc_update_user_last_active` test fail, clean the global state.
diff --git a/plugins/woocommerce/tests/php/src/Admin/API/MarketingRecommendationsTest.php b/plugins/woocommerce/tests/php/src/Admin/API/MarketingRecommendationsTest.php
index af1fd2da128..87888dffdd9 100644
--- a/plugins/woocommerce/tests/php/src/Admin/API/MarketingRecommendationsTest.php
+++ b/plugins/woocommerce/tests/php/src/Admin/API/MarketingRecommendationsTest.php
@@ -113,15 +113,6 @@ class MarketingRecommendationsTest extends WC_REST_Unit_Test_Case {
add_filter( 'pre_http_request', $this->response_mock_ref, 10, 3 );
}
- /**
- * Tear down.
- */
- public function tearDown(): void {
- remove_filter( 'pre_http_request', $this->response_mock_ref );
-
- parent::tearDown();
- }
-
/**
* Tests that the marketing channel recommendations are returned by the endpoint.
*/
diff --git a/plugins/woocommerce/tests/php/src/Admin/API/Reports/Revenue/Stats/ControllerTest.php b/plugins/woocommerce/tests/php/src/Admin/API/Reports/Revenue/Stats/ControllerTest.php
index 3d901cbb3a2..d26c7444528 100644
--- a/plugins/woocommerce/tests/php/src/Admin/API/Reports/Revenue/Stats/ControllerTest.php
+++ b/plugins/woocommerce/tests/php/src/Admin/API/Reports/Revenue/Stats/ControllerTest.php
@@ -26,15 +26,6 @@ class ControllerTest extends WC_Unit_Test_Case {
$this->sut = new Controller();
}
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- parent::tearDown();
- remove_all_filters( 'woocommerce_report_revenue_stats_export_columns' );
- remove_all_filters( 'woocommerce_report_revenue_stats_prepare_export_item' );
- }
-
/**
* @testdox get_export_columns returns the default column set.
*/
diff --git a/plugins/woocommerce/tests/php/src/Admin/API/Reports/Taxes/ControllerTest.php b/plugins/woocommerce/tests/php/src/Admin/API/Reports/Taxes/ControllerTest.php
index 2edf7b8fa0d..1d7279e6939 100644
--- a/plugins/woocommerce/tests/php/src/Admin/API/Reports/Taxes/ControllerTest.php
+++ b/plugins/woocommerce/tests/php/src/Admin/API/Reports/Taxes/ControllerTest.php
@@ -26,15 +26,6 @@ class ControllerTest extends WC_Unit_Test_Case {
$this->sut = new Controller();
}
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- parent::tearDown();
- remove_all_filters( 'woocommerce_report_taxes_export_columns' );
- remove_all_filters( 'woocommerce_report_taxes_prepare_export_item' );
- }
-
/**
* @testdox get_export_columns returns the default column set.
*/
diff --git a/plugins/woocommerce/tests/php/src/Admin/API/Reports/Variations/ControllerTest.php b/plugins/woocommerce/tests/php/src/Admin/API/Reports/Variations/ControllerTest.php
index 3db6f492447..426c4bc79de 100644
--- a/plugins/woocommerce/tests/php/src/Admin/API/Reports/Variations/ControllerTest.php
+++ b/plugins/woocommerce/tests/php/src/Admin/API/Reports/Variations/ControllerTest.php
@@ -26,15 +26,6 @@ class ControllerTest extends WC_Unit_Test_Case {
$this->sut = new Controller();
}
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- parent::tearDown();
- remove_all_filters( 'woocommerce_report_variations_export_columns' );
- remove_all_filters( 'woocommerce_report_variations_prepare_export_item' );
- }
-
/**
* @testdox get_export_columns returns the default column set.
*/
diff --git a/plugins/woocommerce/tests/php/src/Admin/Features/Analytics/FeatureEnabledTest.php b/plugins/woocommerce/tests/php/src/Admin/Features/Analytics/FeatureEnabledTest.php
index b8cab2e9b8c..263527e7e75 100644
--- a/plugins/woocommerce/tests/php/src/Admin/Features/Analytics/FeatureEnabledTest.php
+++ b/plugins/woocommerce/tests/php/src/Admin/Features/Analytics/FeatureEnabledTest.php
@@ -18,11 +18,6 @@ class FeatureEnabledTest extends WC_Unit_Test_Case {
public function tearDown(): void {
delete_option( 'woocommerce_analytics_enabled' );
delete_option( RemoteInboxNotifications::TOGGLE_OPTION_NAME );
- remove_filter( 'woocommerce_admin_features', array( $this, 'enable_analytics_feature' ) );
- remove_filter( 'woocommerce_admin_features', array( $this, 'disable_analytics_feature' ), PHP_INT_MAX );
- remove_filter( 'woocommerce_admin_disabled', '__return_true', PHP_INT_MAX );
- remove_filter( 'woocommerce_admin_features', array( $this, 'disable_launch_your_store_feature' ) );
- remove_filter( 'woocommerce_admin_features', array( $this, 'disable_customize_store_feature' ) );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Admin/Features/Blueprint/RestApiTest.php b/plugins/woocommerce/tests/php/src/Admin/Features/Blueprint/RestApiTest.php
index 97348cc09c0..0abc0468f96 100644
--- a/plugins/woocommerce/tests/php/src/Admin/Features/Blueprint/RestApiTest.php
+++ b/plugins/woocommerce/tests/php/src/Admin/Features/Blueprint/RestApiTest.php
@@ -86,8 +86,6 @@ class RestApiTest extends WC_Unit_Test_Case {
if ( file_exists( $this->temp_file ) ) {
wp_delete_file( $this->temp_file );
}
-
- remove_all_filters( 'pre_option_woocommerce_coming_soon' );
}
/**
diff --git a/plugins/woocommerce/tests/php/src/Admin/Features/OnboardingTasks/Tasks/ExperimentalShippingRecommendationTest.php b/plugins/woocommerce/tests/php/src/Admin/Features/OnboardingTasks/Tasks/ExperimentalShippingRecommendationTest.php
index ef535a10261..c77d9491776 100644
--- a/plugins/woocommerce/tests/php/src/Admin/Features/OnboardingTasks/Tasks/ExperimentalShippingRecommendationTest.php
+++ b/plugins/woocommerce/tests/php/src/Admin/Features/OnboardingTasks/Tasks/ExperimentalShippingRecommendationTest.php
@@ -47,7 +47,6 @@ class ExperimentalShippingRecommendationTest extends WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
- remove_filter( 'pre_option_active_plugins', array( $this, 'get_mocked_active_plugins' ), self::HOOK_PRIORITY );
wp_cache_delete( 'plugins', 'plugins' );
}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/BlockTypes/OrderConfirmation/DownloadsWrapper.php b/plugins/woocommerce/tests/php/src/Blocks/BlockTypes/OrderConfirmation/DownloadsWrapper.php
index f44bb093cbb..611e957beb6 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/BlockTypes/OrderConfirmation/DownloadsWrapper.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/BlockTypes/OrderConfirmation/DownloadsWrapper.php
@@ -40,7 +40,6 @@ final class DownloadsWrapper extends \WP_UnitTestCase {
public function tear_down() {
delete_option( 'woocommerce_product_lookup_table_is_generating' );
wp_cache_delete( 'woocommerce_has_downloadable_products', 'woocommerce' );
- remove_filter( 'pre_option_woocommerce_attribute_lookup_direct_updates', array( self::class, 'enable_direct_attribute_lookup_updates' ) );
parent::tear_down();
}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/Domain/Services/DeleteDraftOrders.php b/plugins/woocommerce/tests/php/src/Blocks/Domain/Services/DeleteDraftOrders.php
index d50f6510afd..96634dcae1e 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/Domain/Services/DeleteDraftOrders.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/Domain/Services/DeleteDraftOrders.php
@@ -125,7 +125,6 @@ class DeleteDraftOrders extends \WC_Unit_Test_Case {
//restore original logging destination
ini_set('error_log', $this->original_logging_destination);
OrderHelper::toggle_cot_feature_and_usage( $this->previous_hpos_state );
- remove_filter( 'wc_allow_changing_orders_storage_while_sync_is_pending', '__return_true' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Routes/CartItems.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Routes/CartItems.php
index 578105e6990..d0f21f7d903 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Routes/CartItems.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Routes/CartItems.php
@@ -170,14 +170,6 @@ class CartItems extends ControllerTestCase {
add_filter( 'woocommerce_logging_class', array( $this, 'override_wc_logger' ) );
}
- /**
- * Tear down.
- */
- protected function tearDown(): void {
- parent::tearDown();
- remove_filter( 'woocommerce_logging_class', array( $this, 'override_wc_logger' ) );
- }
-
/**
* Test getting cart.
*/
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Routes/Patterns.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Routes/Patterns.php
index 73cda1d2f7c..3157cd45557 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Routes/Patterns.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Routes/Patterns.php
@@ -30,14 +30,6 @@ class Patterns extends ControllerTestCase {
add_filter( 'pre_http_request', array( $this, 'mock_ptk_response' ), 10, 3 );
}
- /**
- * Remove the scoped PTK HTTP mock.
- */
- public function tearDown(): void {
- remove_filter( 'pre_http_request', array( $this, 'mock_ptk_response' ), 10 );
- parent::tearDown();
- }
-
/**
* Test the post endpoint when tracking is not allowed.
*
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/ShopperListSchemaTest.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/ShopperListSchemaTest.php
index d878769c999..1b884b2cc22 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/ShopperListSchemaTest.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/ShopperListSchemaTest.php
@@ -60,7 +60,6 @@ class ShopperListSchemaTest extends WC_Unit_Test_Case {
public function tearDown(): void {
wp_delete_user( $this->user_id );
$this->sut = null;
- remove_filter( 'pre_option_woocommerce_cart_save_for_later_enabled', array( $this, 'filter_save_for_later_enabled' ) );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/V1/AbstractAddressSchemaTest.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/V1/AbstractAddressSchemaTest.php
index 176042152b5..e2ed9349b80 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/V1/AbstractAddressSchemaTest.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/V1/AbstractAddressSchemaTest.php
@@ -70,7 +70,6 @@ class AbstractAddressSchemaTest extends WC_Unit_Test_Case {
*/
public function tearDown(): void {
__internal_woocommerce_blocks_deregister_checkout_field( $this->field_id );
- remove_filter( 'doing_it_wrong_trigger_error', '__return_false' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/V1/CheckoutSchemaTest.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/V1/CheckoutSchemaTest.php
index 6f1c63baff4..246917d679a 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/V1/CheckoutSchemaTest.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Schemas/V1/CheckoutSchemaTest.php
@@ -73,7 +73,6 @@ class CheckoutSchemaTest extends WC_Unit_Test_Case {
*/
public function tearDown(): void {
__internal_woocommerce_blocks_deregister_checkout_field( $this->field_id );
- remove_filter( 'doing_it_wrong_trigger_error', '__return_false' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/QuantityLimitsTests.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/QuantityLimitsTests.php
index fea004d5063..3925280c7f1 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/QuantityLimitsTests.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/QuantityLimitsTests.php
@@ -10,14 +10,6 @@ use Automattic\WooCommerce\StoreApi\Utilities\QuantityLimits;
* QuantityLimitsTests class.
*/
class QuantityLimitsTests extends \WC_Unit_Test_Case {
- /**
- * Clean up filters registered by tests, even when a test fails mid-way.
- */
- public function tearDown(): void {
- remove_all_filters( 'woocommerce_store_api_cart_item_quantity_validation' );
- parent::tearDown();
- }
-
/**
* Enable float support for tests.
*/
diff --git a/plugins/woocommerce/tests/php/src/Caching/ObjectCacheTest.php b/plugins/woocommerce/tests/php/src/Caching/ObjectCacheTest.php
index 5630b8d3372..406d0ad59cf 100644
--- a/plugins/woocommerce/tests/php/src/Caching/ObjectCacheTest.php
+++ b/plugins/woocommerce/tests/php/src/Caching/ObjectCacheTest.php
@@ -61,10 +61,6 @@ class ObjectCacheTest extends \WC_Unit_Test_Case {
*/
public function tearDown(): void {
delete_option( 'wp_object_cache_key_prefix_the_type' );
- remove_all_filters( 'wc_object_cache_get_engine' );
- remove_all_filters( 'woocommerce_after_serializing_the_type_for_caching' );
- remove_all_actions( 'woocommerce_after_removing_the_type_from_cache' );
- remove_all_actions( 'woocommerce_after_flushing_the_type_cache' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Database/Migrations/CustomOrderTable/PostsToOrdersMigrationControllerTest.php b/plugins/woocommerce/tests/php/src/Database/Migrations/CustomOrderTable/PostsToOrdersMigrationControllerTest.php
index bbfc038375d..73ea47da6ae 100644
--- a/plugins/woocommerce/tests/php/src/Database/Migrations/CustomOrderTable/PostsToOrdersMigrationControllerTest.php
+++ b/plugins/woocommerce/tests/php/src/Database/Migrations/CustomOrderTable/PostsToOrdersMigrationControllerTest.php
@@ -93,7 +93,6 @@ class PostsToOrdersMigrationControllerTest extends \WC_Unit_Test_Case {
$this->restore_option( CustomOrdersTableController::DB_TRANSACTIONS_ISOLATION_LEVEL_OPTION, $this->previous_isolation_option );
OrderHelper::toggle_cot_feature_and_usage( $this->previous_hpos_state );
self::disable_direct_product_attribute_lookup_updates();
- remove_filter( 'wc_allow_changing_orders_storage_while_sync_is_pending', '__return_true' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Gateways/PayPal/ButtonsTest.php b/plugins/woocommerce/tests/php/src/Gateways/PayPal/ButtonsTest.php
index fa2a4e2af64..2031cce2aee 100644
--- a/plugins/woocommerce/tests/php/src/Gateways/PayPal/ButtonsTest.php
+++ b/plugins/woocommerce/tests/php/src/Gateways/PayPal/ButtonsTest.php
@@ -84,8 +84,6 @@ class ButtonsTest extends \WC_Unit_Test_Case {
// Remove any filters that might have been added.
remove_all_filters( 'woocommerce_is_checkout' );
- remove_all_filters( 'woocommerce_is_cart' );
- remove_all_filters( 'woocommerce_is_product' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Gateways/PayPal/RequestTest.php b/plugins/woocommerce/tests/php/src/Gateways/PayPal/RequestTest.php
index 6cd66dc448f..836fe97e79d 100644
--- a/plugins/woocommerce/tests/php/src/Gateways/PayPal/RequestTest.php
+++ b/plugins/woocommerce/tests/php/src/Gateways/PayPal/RequestTest.php
@@ -37,19 +37,6 @@ class RequestTest extends \WC_Unit_Test_Case {
add_filter( 'pre_option_jetpack_private_options', array( $this, 'return_blog_token' ) );
}
- /**
- * Tear down the test environment.
- *
- * @return void
- */
- public function tearDown(): void {
- remove_filter( 'pre_option_jetpack_options', array( $this, 'return_valid_site_id' ) );
- remove_filter( 'pre_option_jetpack_private_options', array( $this, 'return_blog_token' ) );
- remove_all_filters( 'pre_http_request' );
-
- parent::tearDown();
- }
-
/**
* Test create_paypal_order when API returns error.
*
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/FeaturePluginTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/FeaturePluginTest.php
index ca7ef17659f..d97e4edf501 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/FeaturePluginTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/FeaturePluginTest.php
@@ -18,8 +18,6 @@ class FeaturePluginTest extends WC_Unit_Test_Case {
*/
public function tearDown(): void {
delete_option( Analytics::TOGGLE_OPTION_NAME );
- remove_filter( 'woocommerce_admin_disabled', '__return_true', PHP_INT_MAX );
- remove_filter( 'woocommerce_admin_features', array( $this, 'disable_analytics_feature' ), PHP_INT_MAX );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/MarketingRecommendations/InitTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/MarketingRecommendations/InitTest.php
index 6ba09798c8f..463539565fb 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/MarketingRecommendations/InitTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/MarketingRecommendations/InitTest.php
@@ -36,7 +36,6 @@ class InitTest extends WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
MarketingRecommendationsDataSourcePoller::get_instance()->delete_specs_transient();
- remove_all_filters( 'transient_woocommerce_admin_' . MarketingRecommendationsDataSourcePoller::ID . '_specs' );
}
/**
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/COTRedirectionControllerTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/COTRedirectionControllerTest.php
index 492f8df6f61..a18f16d1845 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/COTRedirectionControllerTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/COTRedirectionControllerTest.php
@@ -36,16 +36,6 @@ class COTRedirectionControllerTest extends \WC_Unit_Test_Case {
add_filter( 'wp_redirect', array( $this, 'watch_and_annul_redirects' ) );
}
- /**
- * Remove our redirect listener.
- *
- * @return void
- */
- public function tearDown(): void {
- parent::tearDown();
- remove_filter( 'wp_redirect', array( $this, 'watch_and_annul_redirects' ) );
- }
-
/**
* Captures the attempted redirect location, and stops the redirect from taking place.
*
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/EditLockTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/EditLockTest.php
index 9a695a7b390..63716ac51a5 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/EditLockTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/EditLockTest.php
@@ -71,7 +71,6 @@ class EditLockTest extends \WC_Unit_Test_Case {
*/
public function tearDown(): void {
$this->clean_up_cot_setup();
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/MetaBoxes/CustomerHistoryTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/MetaBoxes/CustomerHistoryTest.php
index f3648fe2105..f8c983c4c06 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/MetaBoxes/CustomerHistoryTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/MetaBoxes/CustomerHistoryTest.php
@@ -116,8 +116,6 @@ class CustomerHistoryTest extends WC_Unit_Test_Case {
unset( $GLOBALS['wp_post_statuses']['wc-competition-completed'] );
$this->add_long_status_callback = null;
}
- remove_filter( 'woocommerce_order_class', array( AdminOrder::class, 'order_class_name' ) );
- remove_filter( 'wc_allow_changing_orders_storage_while_sync_is_pending', '__return_true' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/Reports/ProductSearchQueryTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/Reports/ProductSearchQueryTest.php
index 9a4ffd9b11c..50a39166e20 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/Reports/ProductSearchQueryTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/Reports/ProductSearchQueryTest.php
@@ -12,14 +12,6 @@ use WC_Unit_Test_Case;
*/
class ProductSearchQueryTest extends WC_Unit_Test_Case {
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- remove_filter( 'wc_product_sku_enabled', '__return_false' );
- parent::tearDown();
- }
-
/**
* @testdox Should split a comma separated string into terms.
*/
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/ShippingPartnerSuggestions/ShippingPartnerSuggestionsTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/ShippingPartnerSuggestions/ShippingPartnerSuggestionsTest.php
index f8e06793eef..2c801af4def 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/ShippingPartnerSuggestions/ShippingPartnerSuggestionsTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/ShippingPartnerSuggestions/ShippingPartnerSuggestionsTest.php
@@ -82,7 +82,6 @@ class ShippingPartnerSuggestionsTest extends WC_Unit_Test_Case {
ShippingPartnerSuggestionsDataSourcePoller::get_instance()->delete_specs_transient();
remove_all_filters( 'transient_woocommerce_admin_' . ShippingPartnerSuggestionsDataSourcePoller::ID . '_specs' );
update_option( 'woocommerce_default_country', 'US' );
- remove_filter( 'woocommerce_logging_class', array( $this, 'override_wc_logger' ) );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/AgentReadiness/ReadinessRestControllerTest.php b/plugins/woocommerce/tests/php/src/Internal/AgentReadiness/ReadinessRestControllerTest.php
index ffd2e57bc2a..a82cc1867a6 100644
--- a/plugins/woocommerce/tests/php/src/Internal/AgentReadiness/ReadinessRestControllerTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/AgentReadiness/ReadinessRestControllerTest.php
@@ -34,7 +34,6 @@ class ReadinessRestControllerTest extends WC_REST_Unit_Test_Case {
public function tearDown(): void {
as_unschedule_all_actions( ScheduledScan::RUN_NOW_HOOK );
as_unschedule_all_actions( ScheduledScan::DAILY_HOOK );
- remove_filter( 'pre_option_woocommerce_feature_agent_readiness_enabled', array( __CLASS__, 'enabled' ) );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/AgentReadiness/ScheduledScanTest.php b/plugins/woocommerce/tests/php/src/Internal/AgentReadiness/ScheduledScanTest.php
index 45885391e29..d22b93da003 100644
--- a/plugins/woocommerce/tests/php/src/Internal/AgentReadiness/ScheduledScanTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/AgentReadiness/ScheduledScanTest.php
@@ -43,7 +43,6 @@ class ScheduledScanTest extends WC_Unit_Test_Case {
as_unschedule_all_actions( ScheduledScan::RUN_NOW_HOOK );
as_unschedule_all_actions( ScheduledScan::DAILY_HOOK );
delete_option( 'woocommerce_feature_agent_readiness_enabled' );
- remove_filter( 'woocommerce_agent_readiness_rules_url', '__return_empty_string' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Core/PlatformRegistryTest.php b/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Core/PlatformRegistryTest.php
index b23dceabad8..b427082e4b4 100644
--- a/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Core/PlatformRegistryTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Core/PlatformRegistryTest.php
@@ -16,14 +16,6 @@ use Automattic\WooCommerce\Internal\CLI\Migrator\Core\PlatformRegistry;
*/
class PlatformRegistryTest extends \WC_Unit_Test_Case {
- /**
- * Clean up filters after each test.
- */
- public function tearDown(): void {
- parent::tearDown();
- remove_all_filters( 'woocommerce_migrator_platforms' );
- }
-
/**
* Test platform registration and retrieval.
*/
diff --git a/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Platforms/Webflow/WebflowPlatformTest.php b/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Platforms/Webflow/WebflowPlatformTest.php
index f78fbb70087..7f3a6c97980 100644
--- a/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Platforms/Webflow/WebflowPlatformTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Platforms/Webflow/WebflowPlatformTest.php
@@ -28,14 +28,6 @@ class WebflowPlatformTest extends \WC_Unit_Test_Case {
WebflowPlatform::init();
}
- /**
- * Clean up after each test.
- */
- public function tearDown(): void {
- remove_all_filters( 'woocommerce_migrator_platforms' );
- parent::tearDown();
- }
-
/**
* Test that the Webflow platform is registered with the expected configuration.
*/
diff --git a/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Platforms/Webflow/WebflowRealDataIntegrationTest.php b/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Platforms/Webflow/WebflowRealDataIntegrationTest.php
index bf4c438bcf2..44ba312f6f9 100644
--- a/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Platforms/Webflow/WebflowRealDataIntegrationTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/CLI/Migrator/Platforms/Webflow/WebflowRealDataIntegrationTest.php
@@ -40,14 +40,6 @@ class WebflowRealDataIntegrationTest extends WC_Unit_Test_Case {
add_filter( 'pre_http_request', array( $this, 'handle_request' ), 10, 3 );
}
- /**
- * Tear down the HTTP stub.
- */
- public function tearDown(): void {
- remove_filter( 'pre_http_request', array( $this, 'handle_request' ), 10 );
- parent::tearDown();
- }
-
/**
* Route Webflow API requests to canned responses. The fetcher hits three endpoints:
* - /sites/{id}/products → product list
diff --git a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/HposOrderCapabilityHelperTest.php b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/HposOrderCapabilityHelperTest.php
index 1eb702e096e..ab5ee8dc4ce 100644
--- a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/HposOrderCapabilityHelperTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/HposOrderCapabilityHelperTest.php
@@ -60,7 +60,6 @@ class HposOrderCapabilityHelperTest extends WC_Unit_Test_Case {
remove_filter( 'wc_order_statuses', array( $this, 'add_private_test_order_status' ) );
$this->unregister_private_test_order_status();
$this->clean_up_cot_setup();
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/LegacyDataHandlerTests.php b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/LegacyDataHandlerTests.php
index e1bfadbfff5..3c4c81559fc 100644
--- a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/LegacyDataHandlerTests.php
+++ b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/LegacyDataHandlerTests.php
@@ -56,7 +56,6 @@ class LegacyDataHandlerTests extends \WC_Unit_Test_Case {
public function tearDown(): void {
$this->clean_up_cot_setup();
OrderHelper::toggle_cot_feature_and_usage( $this->previous_hpos_state );
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreCacheCrossBleedTest.php b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreCacheCrossBleedTest.php
index aebb6d05f34..4bc245901a6 100644
--- a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreCacheCrossBleedTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreCacheCrossBleedTest.php
@@ -73,8 +73,6 @@ class OrdersTableDataStoreCacheCrossBleedTest extends \HposTestCase {
$this->clean_up_cot_setup();
delete_option( CustomOrdersTableController::HPOS_DATASTORE_CACHING_ENABLED_OPTION );
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
- remove_all_filters( 'woocommerce_logging_class' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreCachePrimingTest.php b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreCachePrimingTest.php
index 4e1f38019a6..25b1d1179a8 100644
--- a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreCachePrimingTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreCachePrimingTest.php
@@ -64,7 +64,6 @@ class OrdersTableDataStoreCachePrimingTest extends \HposTestCase {
$this->toggle_cot_feature_and_usage( $this->cot_state );
$this->clean_up_cot_setup();
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreRestOrdersControllerTests.php b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreRestOrdersControllerTests.php
index 081bda2ba29..419830bcc59 100644
--- a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreRestOrdersControllerTests.php
+++ b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreRestOrdersControllerTests.php
@@ -57,8 +57,6 @@ class OrdersTableDataStoreRestOrdersControllerTests extends \WC_REST_Orders_Cont
add_filter( 'query', array( $this, '_drop_temporary_tables' ) );
parent::tearDown();
-
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
}
/**
diff --git a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreTests.php b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreTests.php
index 20a7e97a50b..318c308c0da 100644
--- a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreTests.php
+++ b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableDataStoreTests.php
@@ -116,9 +116,6 @@ class OrdersTableDataStoreTests extends \HposTestCase {
$this->clean_up_cot_setup();
$this->disable_cogs_feature();
- remove_all_filters( 'wc_allow_changing_orders_storage_while_sync_is_pending' );
- remove_all_filters( 'woocommerce_load_order_cogs_value' );
- remove_all_filters( 'woocommerce_save_order_cogs_value' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableQueryTests.php b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableQueryTests.php
index 752b49eacbb..c308face491 100644
--- a/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableQueryTests.php
+++ b/plugins/woocommerce/tests/php/src/Internal/DataStores/Orders/OrdersTableQueryTests.php
@@ -93,7 +93,6 @@ class OrdersTableQueryTests extends \WC_Unit_Test_Case {
*/
public function tearDown(): void {
$this->toggle_cot_feature_and_usage( $this->cot_state );
- remove_filter( 'wc_allow_changing_orders_storage_while_sync_is_pending', '__return_true' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/Email/EmailLoggerTest.php b/plugins/woocommerce/tests/php/src/Internal/Email/EmailLoggerTest.php
index 33084fa7503..8403e42b9f5 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Email/EmailLoggerTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Email/EmailLoggerTest.php
@@ -34,21 +34,6 @@ class EmailLoggerTest extends WC_Unit_Test_Case {
$this->sut = new EmailLogger();
}
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- remove_all_filters( 'woocommerce_email_log_enabled' );
- remove_all_filters( 'woocommerce_email_log_context' );
- remove_all_filters( 'woocommerce_email_log_add_order_note' );
- remove_all_filters( 'woocommerce_mail_callback' );
- remove_all_actions( 'woocommerce_email_disabled' );
- remove_all_actions( 'woocommerce_email_skipped' );
- remove_action( 'woocommerce_email_sent', array( $this->sut, 'handle_woocommerce_email_sent' ) );
- remove_action( 'wp_mail_failed', array( $this->sut, 'capture_mail_error' ) );
- parent::tearDown();
- }
-
/**
* @testdox Register method adds hooks for woocommerce_email_sent, wp_mail_failed, woocommerce_email_disabled, and woocommerce_email_skipped.
*/
diff --git a/plugins/woocommerce/tests/php/src/Internal/EmailEditor/BlockEmailRendererTest.php b/plugins/woocommerce/tests/php/src/Internal/EmailEditor/BlockEmailRendererTest.php
index f48a361099a..12c3b6926be 100644
--- a/plugins/woocommerce/tests/php/src/Internal/EmailEditor/BlockEmailRendererTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/EmailEditor/BlockEmailRendererTest.php
@@ -418,7 +418,6 @@ class BlockEmailRendererTest extends \WC_Unit_Test_Case {
*/
public function tearDown(): void {
parent::tearDown();
- remove_filter( 'woocommerce_transactional_emails_for_block_editor', array( $this, 'register_fake_block_editor_email_ids' ) );
WCTransactionalEmailPostsManager::get_instance()->clear_caches();
update_option( 'woocommerce_feature_block_email_editor_enabled', 'no' );
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/EmailEditor/EmailApiControllerTest.php b/plugins/woocommerce/tests/php/src/Internal/EmailEditor/EmailApiControllerTest.php
index 88071a5e53b..d6c72e5364b 100644
--- a/plugins/woocommerce/tests/php/src/Internal/EmailEditor/EmailApiControllerTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/EmailEditor/EmailApiControllerTest.php
@@ -67,7 +67,6 @@ class EmailApiControllerTest extends \WC_Unit_Test_Case {
parent::tearDown();
update_option( 'woocommerce_feature_block_email_editor_enabled', 'no' );
delete_option( 'woocommerce_' . $this->email_type . '_settings' );
- remove_all_filters( 'woocommerce_transactional_emails_for_block_editor' );
WCEmailTemplateSyncRegistry::reset_cache();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/OrderReviews/StarRatingTest.php b/plugins/woocommerce/tests/php/src/Internal/OrderReviews/StarRatingTest.php
index 8217c4c4b57..972f923f18d 100644
--- a/plugins/woocommerce/tests/php/src/Internal/OrderReviews/StarRatingTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/OrderReviews/StarRatingTest.php
@@ -11,14 +11,6 @@ use WC_Unit_Test_Case;
*/
class StarRatingTest extends WC_Unit_Test_Case {
- /**
- * Reset filter state between tests.
- */
- public function tearDown(): void {
- remove_all_filters( 'woocommerce_review_order_rating_labels' );
- parent::tearDown();
- }
-
/**
* @testdox render() emits a radiogroup, five radios, and a caption span.
*/
diff --git a/plugins/woocommerce/tests/php/src/Internal/OrderReviews/SubmissionHandlerRoutingTest.php b/plugins/woocommerce/tests/php/src/Internal/OrderReviews/SubmissionHandlerRoutingTest.php
index 38795555e5c..a59b83fb7eb 100644
--- a/plugins/woocommerce/tests/php/src/Internal/OrderReviews/SubmissionHandlerRoutingTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/OrderReviews/SubmissionHandlerRoutingTest.php
@@ -46,9 +46,6 @@ class SubmissionHandlerRoutingTest extends WC_Unit_Test_Case {
delete_option( 'comment_moderation' );
$_POST = array();
ItemEligibility::reset_cache();
- remove_all_filters( 'wp_die_ajax_handler' );
- remove_all_filters( 'wp_doing_ajax' );
- remove_all_filters( 'wp_send_json_handler' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/Orders/UnpaidOrderTimeoutMarkerTest.php b/plugins/woocommerce/tests/php/src/Internal/Orders/UnpaidOrderTimeoutMarkerTest.php
index 12d8a0c8828..e61c32caef0 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Orders/UnpaidOrderTimeoutMarkerTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Orders/UnpaidOrderTimeoutMarkerTest.php
@@ -33,14 +33,6 @@ class UnpaidOrderTimeoutMarkerTest extends WC_Unit_Test_Case {
$this->sut->register();
}
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- remove_all_actions( 'woocommerce_order_status_cancelled' );
- parent::tearDown();
- }
-
/**
* Load an order from the database, bypassing the order cache.
*
diff --git a/plugins/woocommerce/tests/php/src/Internal/ProductAttributesLookup/DataRegeneratorTest.php b/plugins/woocommerce/tests/php/src/Internal/ProductAttributesLookup/DataRegeneratorTest.php
index 7226c51b3b8..3a5169d3efd 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ProductAttributesLookup/DataRegeneratorTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ProductAttributesLookup/DataRegeneratorTest.php
@@ -265,8 +265,6 @@ class DataRegeneratorTest extends \WC_Unit_Test_Case {
do_action( 'woocommerce_run_product_attribute_lookup_regeneration_callback' );
- remove_all_filters( ' woocommerce_attribute_lookup_regeneration_step_size' );
-
$expected_limit_value = $set_filter ? $filtered_size : DataRegenerator::PRODUCTS_PER_GENERATION_STEP;
$this->assertEquals( array( $expected_limit_value ), $requested_step_sizes );
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Feed/ProductWalkerTest.php b/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Feed/ProductWalkerTest.php
index 42fdfaf04d4..42e4c18f05d 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Feed/ProductWalkerTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Feed/ProductWalkerTest.php
@@ -38,7 +38,6 @@ class ProductWalkerTest extends \WC_Unit_Test_Case {
*/
public function tearDown(): void {
parent::tearDown();
- remove_all_filters( 'woocommerce_product_feed_args' );
$this->test_container->reset_all_replacements();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Integrations/POSCatalog/AsyncGeneratorTest.php b/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Integrations/POSCatalog/AsyncGeneratorTest.php
index 9695bfe1063..dea232af658 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Integrations/POSCatalog/AsyncGeneratorTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Integrations/POSCatalog/AsyncGeneratorTest.php
@@ -67,8 +67,6 @@ class AsyncGeneratorTest extends \WC_Unit_Test_Case {
delete_option( self::OPTION_KEY );
delete_option( self::OPTION_KEY . '_chunk_size' );
- remove_all_filters( 'woocommerce_product_feed_chunk_size' );
- remove_all_filters( 'woocommerce_product_feed_batch_size' );
// Always clear the timeout filter here so a failed assertion in a test that registers it cannot
// leak it into later tests in the same process (tearDown runs even when a test fails).
remove_all_filters( 'woocommerce_product_feed_in_progress_timeout' );
diff --git a/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Storage/JsonFileFeedTest.php b/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Storage/JsonFileFeedTest.php
index 74d942aab7b..75b50ed8165 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Storage/JsonFileFeedTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ProductFeed/Storage/JsonFileFeedTest.php
@@ -27,7 +27,6 @@ class JsonFileFeedTest extends \WC_Unit_Test_Case {
public function tearDown(): void {
parent::tearDown();
$this->get_and_delete_dir();
- remove_all_filters( 'woocommerce_product_feed_time' );
}
/**
diff --git a/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Dispatchers/InternalNotificationDispatcherTest.php b/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Dispatchers/InternalNotificationDispatcherTest.php
index 8fbe88503e7..68b40ebc521 100644
--- a/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Dispatchers/InternalNotificationDispatcherTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Dispatchers/InternalNotificationDispatcherTest.php
@@ -59,14 +59,6 @@ class InternalNotificationDispatcherTest extends WC_Unit_Test_Case {
add_filter( 'pre_http_request', array( $this, 'intercept_http_request' ), 10, 3 );
}
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- remove_filter( 'pre_http_request', array( $this, 'intercept_http_request' ), 10 );
- parent::tearDown();
- }
-
/**
* Intercepts wp_remote_post calls and captures request data.
*
diff --git a/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Dispatchers/WpcomNotificationDispatcherTest.php b/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Dispatchers/WpcomNotificationDispatcherTest.php
index 604a57a4ff2..91cc8288d74 100644
--- a/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Dispatchers/WpcomNotificationDispatcherTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Dispatchers/WpcomNotificationDispatcherTest.php
@@ -70,16 +70,6 @@ class WpcomNotificationDispatcherTest extends WC_Unit_Test_Case {
add_filter( 'pre_http_request', array( $this, 'intercept_http_request' ), 10, 3 );
}
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- remove_filter( 'pre_option_jetpack_options', array( $this, 'filter_jetpack_options' ) );
- remove_filter( 'pre_option_jetpack_private_options', array( $this, 'filter_jetpack_private_options' ) );
- remove_filter( 'pre_http_request', array( $this, 'intercept_http_request' ), 10 );
- parent::tearDown();
- }
-
/**
* Returns fake Jetpack options with a site ID.
*
diff --git a/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Services/PendingNotificationStoreTest.php b/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Services/PendingNotificationStoreTest.php
index 35aa8863b10..51861050e59 100644
--- a/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Services/PendingNotificationStoreTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/PushNotifications/Services/PendingNotificationStoreTest.php
@@ -42,14 +42,6 @@ class PendingNotificationStoreTest extends WC_Unit_Test_Case {
$this->store->register();
}
- /**
- * Tear down test fixtures.
- */
- public function tearDown(): void {
- remove_action( 'shutdown', array( $this->store, 'dispatch_all' ) );
- parent::tearDown();
- }
-
/**
* @testdox Should add a notification to the store.
*/
diff --git a/plugins/woocommerce/tests/php/src/Internal/StockNotifications/AsyncTasks/NotificationsProcessorTests.php b/plugins/woocommerce/tests/php/src/Internal/StockNotifications/AsyncTasks/NotificationsProcessorTests.php
index d1001c78e28..44343d6dacf 100644
--- a/plugins/woocommerce/tests/php/src/Internal/StockNotifications/AsyncTasks/NotificationsProcessorTests.php
+++ b/plugins/woocommerce/tests/php/src/Internal/StockNotifications/AsyncTasks/NotificationsProcessorTests.php
@@ -77,7 +77,6 @@ class NotificationsProcessorTests extends \WC_Unit_Test_Case {
$wpdb->query( "DELETE FROM {$wpdb->prefix}wc_stock_notificationmeta" );
$wpdb->query( "DELETE FROM {$wpdb->prefix}wc_stock_notifications" );
$this->restore_stock_notifications_feature_option();
- remove_all_filters( 'woocommerce_customer_stock_notifications_product_is_valid' );
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/Traits/RestApiCacheTest.php b/plugins/woocommerce/tests/php/src/Internal/Traits/RestApiCacheTest.php
index e8a66c74708..7ec984d95b4 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Traits/RestApiCacheTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Traits/RestApiCacheTest.php
@@ -90,8 +90,6 @@ class RestApiCacheTest extends WC_Unit_Test_Case {
public function tearDown(): void {
$this->clear_rest_server();
- remove_all_filters( 'woocommerce_rest_api_not_modified_response' );
-
parent::tearDown();
}
diff --git a/plugins/woocommerce/tests/php/src/Internal/VariationGallery/LegacyVariationGalleryCompatibilityTest.php b/plugins/woocommerce/tests/php/src/Internal/VariationGallery/LegacyVariationGalleryCompatibilityTest.php
index 6343a49f5ed..92ccea1a7d6 100644
--- a/plugins/woocommerce/tests/php/src/Internal/VariationGallery/LegacyVariationGalleryCompatibilityTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/VariationGallery/LegacyVariationGalleryCompatibilityTest.php
@@ -27,18 +27,6 @@ class LegacyVariationGalleryCompatibilityTest extends \WC_Unit_Test_Case {
$this->sut = new LegacyVariationGalleryCompatibility();
}
- /**
- * Remove registered filters after each test.
- */
- public function tearDown(): void {
- remove_filter(
- 'woocommerce_product_variation_get_gallery_image_ids',
- array( $this->sut, 'maybe_read_legacy_gallery_image_ids' ),
- 10
- );
- parent::tearDown();
- }
-
/**
* @testdox Legacy variation gallery meta is ignored when the core gallery already has values.
*/
diff --git a/plugins/woocommerce/tests/php/src/Utilities/RestApiUtilTest.php b/plugins/woocommerce/tests/php/src/Utilities/RestApiUtilTest.php
index 484bd9a61ff..9f4716168f0 100644
--- a/plugins/woocommerce/tests/php/src/Utilities/RestApiUtilTest.php
+++ b/plugins/woocommerce/tests/php/src/Utilities/RestApiUtilTest.php
@@ -35,7 +35,6 @@ class RestApiUtilTest extends \WC_Unit_Test_Case {
public function tearDown(): void {
// Clear any filters that may have been added during tests.
remove_all_filters( 'woocommerce_rest_should_lazy_load_namespace' );
- remove_all_filters( 'rest_pre_dispatch' );
// Clear global wp query vars.
if ( isset( $GLOBALS['wp'] ) ) {