Temporary fix for a unit test.
The "Advanced" section in the product settings page exists only if the product attributes lookup table exists. The table will eventually exist for everybody, so a temporary check has been added.
This commit is contained in:
parent
fd81e477d2
commit
79f2936ae9
|
@ -7,6 +7,8 @@
|
||||||
|
|
||||||
// phpcs:ignore Squiz.Commenting.FileComment.Missing
|
// phpcs:ignore Squiz.Commenting.FileComment.Missing
|
||||||
|
|
||||||
|
use Automattic\WooCommerce\Internal\ProductAttributesLookup\LookupDataStore;
|
||||||
|
|
||||||
require_once __DIR__ . '/class-wc-settings-unit-test-case.php';
|
require_once __DIR__ . '/class-wc-settings-unit-test-case.php';
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -28,6 +30,11 @@ class WC_Settings_Products_Test extends WC_Settings_Unit_Test_Case {
|
||||||
'downloadable',
|
'downloadable',
|
||||||
);
|
);
|
||||||
|
|
||||||
|
// TODO: Once the lookup table is created in a migration, remove the check and just include 'advanced' in $expected.
|
||||||
|
if ( wc_get_container()->get( LookupDataStore::class )->check_lookup_table_exists() ) {
|
||||||
|
array_push( $expected, 'advanced' );
|
||||||
|
}
|
||||||
|
|
||||||
$this->assertEquals( $expected, $section_names );
|
$this->assertEquals( $expected, $section_names );
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -134,12 +141,12 @@ class WC_Settings_Products_Test extends WC_Settings_Unit_Test_Case {
|
||||||
$settings_ids_and_types = $this->get_ids_and_types( $settings );
|
$settings_ids_and_types = $this->get_ids_and_types( $settings );
|
||||||
|
|
||||||
$expected = array(
|
$expected = array(
|
||||||
'digital_download_options' => array( 'title', 'sectionend' ),
|
'digital_download_options' => array( 'title', 'sectionend' ),
|
||||||
'woocommerce_file_download_method' => 'select',
|
'woocommerce_file_download_method' => 'select',
|
||||||
'woocommerce_downloads_redirect_fallback_allowed' => 'checkbox',
|
'woocommerce_downloads_redirect_fallback_allowed' => 'checkbox',
|
||||||
'woocommerce_downloads_require_login' => 'checkbox',
|
'woocommerce_downloads_require_login' => 'checkbox',
|
||||||
'woocommerce_downloads_grant_access_after_payment' => 'checkbox',
|
'woocommerce_downloads_grant_access_after_payment' => 'checkbox',
|
||||||
'woocommerce_downloads_add_hash_to_filename' => 'checkbox',
|
'woocommerce_downloads_add_hash_to_filename' => 'checkbox',
|
||||||
);
|
);
|
||||||
|
|
||||||
$this->assertEquals( $expected, $settings_ids_and_types );
|
$this->assertEquals( $expected, $settings_ids_and_types );
|
||||||
|
|
Loading…
Reference in New Issue