Steve-Dee-Designs
/
woocommerce
mirror of https://github.com/woocommerce/woocommerce.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

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.
This commit is contained in:
Corey McKrill 2023-06-14 10:49:50 -07:00
parent 9db19e4848
commit 1804d44be1
No known key found for this signature in database
GPG Key ID: 84BBFE669C4D97B8
1 changed files with 0 additions and 100 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

100
plugins/woocommerce/tests/php/includes/class-wc-tracker-test.php
View File

@ -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.
*/