Merge pull request #634 from lucatume/fix-issue-627

fix(WPTestCase) exclude WooCommerce unserializables from backup
lucatume · Sep 18, 2023 · 68166b4 · 68166b4
2 parents 45eac99 + 8c052ce
commit 68166b4
Show file tree

Hide file tree

Showing 18 changed files with 106 additions and 5,346 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,10 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [unreleased] Unreleased
 
+### Fixed
+
+- Exclude some WooCommerce global values and static attributes from backup to avoid fatals.
+
 ## [4.0.1] 2023-09-15;
 
 ### Added

diff --git a/src/TestCase/WPTestCase.php b/src/TestCase/WPTestCase.php
@@ -35,12 +35,22 @@ class WPTestCase extends Unit
         'wp_current_filter',
         'wp_filter',
         'wp_object_cache',
-        'wp_meta_keys'
+        'wp_meta_keys',
+        // WooCommerce.
+        'woocommerce',
     ];
 
     // Backup, and reset, static class attributes between tests.
     protected $backupStaticAttributes = true;
 
+    // A list of static attributes that should not be backed up as they are wired to explode when doing so.
+    protected $backupStaticAttributesExcludeList = [
+        // WooCommerce.
+        'WooCommerce' => ['_instance'],
+        'Automattic\WooCommerce\Internal\Admin\FeaturePlugin' => ['instance'],
+        'Automattic\WooCommerce\RestApi\Server' => ['instance']
+    ];
+
     /**
      * @var array<string,WP_UnitTestCase>
      */

diff --git a/tests/_support/CommandTester.php b/tests/_support/CommandTester.php
diff --git a/tests/_support/FeaturesTester.php b/tests/_support/FeaturesTester.php
diff --git a/tests/_support/Helper/Muloader.php b/tests/_support/Helper/Muloader.php
diff --git a/tests/_support/MuloaderTester.php b/tests/_support/MuloaderTester.php
diff --git a/tests/_support/ScenariosTester.php b/tests/_support/ScenariosTester.php
diff --git a/tests/_support/TestCase/WooCommerceLoadTestCase.php b/tests/_support/TestCase/WooCommerceLoadTestCase.php
@@ -0,0 +1,23 @@
+<?php
+
+namespace lucatume\WPBrowser\Tests\TestCase;
+
+use lucatume\WPBrowser\TestCase\WPTestCase;
+
+class WooCommerceLoadTestCase extends WPTestCase
+{
+    public function testWordPressLoadedCorrectly(): void
+    {
+        $this->assertTrue(function_exists('do_action'));
+    }
+
+    public function testWooCommerceIsActivated(): void
+    {
+        $this->assertTrue(is_plugin_active('woocommerce/woocommerce.php'));
+    }
+
+    public function testWooCommerceFunctionsExist(): void
+    {
+        $this->assertTrue(function_exists('wc_get_product'));
+    }
+}
diff --git a/tests/_support/UnitHelper.php b/tests/_support/UnitHelper.php
diff --git a/tests/_support/WpunitTester.php b/tests/_support/WpunitTester.php