Un-skip product import test, delete all products in the setup (https://github.com/woocommerce/woocommerce-admin/pull/6905)

* un-skip test, delete all products in the setup

* add to readme

* change fix to dev
This commit is contained in:
Bec Scott 2021-04-30 11:38:26 +10:00 committed by GitHub
parent b84799d470
commit 7a0b00ec0c
2 changed files with 8 additions and 1 deletions

View File

@ -88,6 +88,7 @@ Release and roadmap notes are available on the [WooCommerce Developers Blog](htt
- Dev: Update E2E jest config, so it correctly creates screenshots on failure. #6858 - Dev: Update E2E jest config, so it correctly creates screenshots on failure. #6858
- Dev: Fixed storybook build script #6875 - Dev: Fixed storybook build script #6875
- Dev: Removed allowed keys list for adding woocommerce_meta data. #6889 🎉 @xristos3490 - Dev: Removed allowed keys list for adding woocommerce_meta data. #6889 🎉 @xristos3490
- Dev: Delete all products when running product import tests, unskip previously skipped test. #6905
- Feature: Add recommended payment methods in payment settings. #6760 - Feature: Add recommended payment methods in payment settings. #6760
- Fix: Disable the continue btn on OBW when requested are being made #6838 - Fix: Disable the continue btn on OBW when requested are being made #6838
- Fix: Event tracking for merchant email notes #6616 - Fix: Event tracking for merchant email notes #6616

View File

@ -30,6 +30,13 @@ class WC_Tests_API_Onboarding_Tasks extends WC_REST_Unit_Test_Case {
'role' => 'administrator', 'role' => 'administrator',
) )
); );
// Empty the db of any products.
$query = new \WC_Product_Query();
$products = $query->get_products();
foreach ( $products as $product ) {
$product->delete( true );
}
} }
/** /**
@ -57,7 +64,6 @@ class WC_Tests_API_Onboarding_Tasks extends WC_REST_Unit_Test_Case {
* Test that sample product data is imported. * Test that sample product data is imported.
*/ */
public function test_import_sample_products() { public function test_import_sample_products() {
$this->markTestSkipped( 'Skipped as test randomly fails on line 77.' );
wp_set_current_user( $this->user ); wp_set_current_user( $this->user );
$this->remove_color_or_logo_attribute_taxonomy(); $this->remove_color_or_logo_attribute_taxonomy();