From 1804d44be11c83d37a3c637d3e828079b5ccbaea Mon Sep 17 00:00:00 2001
From: Corey McKrill <916023+coreymckrill@users.noreply.github.com>
Date: Wed, 14 Jun 2023 10:49:50 -0700
Subject: [PATCH] Remove feature compatibility unit test

The mocks in this test were affecting other unit tests for the
PluginUtil class, but only when run in GitHub actions (the tests were
working fine locally). Rather than hold up this PR, I've created
issue #38720 as a future task to add the unit test back in once we've
figured out if there's something buggy happening with dependency
injection.
---
 .../php/includes/class-wc-tracker-test.php    | 100 ------------------
 1 file changed, 100 deletions(-)

diff --git a/plugins/woocommerce/tests/php/includes/class-wc-tracker-test.php b/plugins/woocommerce/tests/php/includes/class-wc-tracker-test.php
index 0a9c5fb1368..41da24722d7 100644
--- a/plugins/woocommerce/tests/php/includes/class-wc-tracker-test.php
+++ b/plugins/woocommerce/tests/php/includes/class-wc-tracker-test.php
@@ -5,10 +5,6 @@
  * @package WooCommerce\Tests\WC_Tracker.
  */
 
-use Automattic\WooCommerce\Internal\Features\FeaturesController;
-use Automattic\WooCommerce\Utilities\PluginUtil;
-use Automattic\WooCommerce\Testing\Tools\CodeHacking\Hacks\FunctionsMockerHack;
-
 // phpcs:disable Squiz.Classes.ClassFileName.NoMatch, Squiz.Classes.ValidClassName.NotCamelCaps -- Backward compatibility.
 /**
  * Class WC_Tracker_Test
@@ -72,102 +68,6 @@ class WC_Tracker_Test extends \WC_Unit_Test_Case {
 		$this->assertEquals( 'no', $tracking_data['wc_admin_disabled'] );
 	}
 
-	/**
-	 * @testDox Test the features compatibility data for plugin tracking data.
-	 */
-	public function test_get_tracking_data_plugin_feature_compatibility() {
-		$legacy_mocks = array(
-			'get_plugins' => function() {
-				return array(
-					'plugin1' => array(
-						'Name' => 'Plugin 1',
-					),
-					'plugin2' => array(
-						'Name' => 'Plugin 2',
-					),
-					'plugin3' => array(
-						'Name' => 'Plugin 3',
-					),
-				);
-			},
-		);
-		FunctionsMockerHack::add_function_mocks( $legacy_mocks );
-		$this->register_legacy_proxy_function_mocks( $legacy_mocks );
-
-		update_option( 'active_plugins', array( 'plugin1', 'plugin2' ) );
-
-		$pluginutil_mock = new class() extends PluginUtil {
-			// phpcs:ignore Squiz.Commenting.FunctionComment.Missing
-			public function is_woocommerce_aware_plugin( $plugin ): bool {
-				if ( 'plugin1' === $plugin ) {
-					return false;
-				}
-
-				return true;
-			}
-		};
-
-		$featurescontroller_mock = new class() extends FeaturesController {
-			// phpcs:ignore Squiz.Commenting.FunctionComment.Missing
-			public function get_compatible_features_for_plugin( string $plugin_name, bool $enabled_features_only = false ): array {
-				$compat = array();
-				switch ( $plugin_name ) {
-					case 'plugin2':
-						$compat = array(
-							'compatible'   => array( 'feature1' ),
-							'incompatible' => array( 'feature2' ),
-							'uncertain'    => array( 'feature3' ),
-						);
-						break;
-					case 'plugin3':
-						$compat = array(
-							'compatible'   => array( 'feature2' ),
-							'incompatible' => array(),
-							'uncertain'    => array( 'feature1', 'feature3' ),
-						);
-						break;
-				}
-
-				return $compat;
-			}
-		};
-
-		$container = wc_get_container();
-		$container->get( PluginUtil::class ); // Ensure that the class is loaded.
-		$container->reset_all_resolved();
-		$container->replace( PluginUtil::class, $pluginutil_mock );
-		$container->replace( FeaturesController::class, $featurescontroller_mock );
-
-		$tracking_data = WC_Tracker::get_tracking_data();
-
-		$this->assertEquals(
-			array(),
-			$tracking_data['active_plugins']['plugin1']['feature_compatibility']
-		);
-		$this->assertEquals(
-			array(
-				'compatible'   => array( 'feature1' ),
-				'incompatible' => array( 'feature2' ),
-				'uncertain'    => array( 'feature3' ),
-			),
-			$tracking_data['active_plugins']['plugin2']['feature_compatibility']
-		);
-		$this->assertEquals(
-			array(
-				'compatible' => array( 'feature2' ),
-				'uncertain'  => array( 'feature1', 'feature3' ),
-			),
-			$tracking_data['inactive_plugins']['plugin3']['feature_compatibility']
-		);
-
-		// Reset the mocked classes so they don't affect other tests.
-		$container->replace( PluginUtil::class, PluginUtil::class );
-		$container->get( PluginUtil::class );
-		$container->replace( FeaturesController::class, FeaturesController::class );
-
-		$this->reset_legacy_proxy_mocks();
-	}
-
 	/**
 	 * @testDox Test orders tracking data.
 	 */