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

Delete all products before running tests (https://github.com/woocommerce/woocommerce-admin/pull/6178) 

* Delete all products before running tests

* Delete products in tear down instead

* Only remove the color attribute, as other attitributes are necessary in other tests

* Make assertion more lenient, and remove all products deletion

* Fix lint errors

* Changed to check if imported is at-least 2 items
This commit is contained in:
louwie17 2021-01-27 13:57:58 -04:00 committed by GitHub
parent c0d6ec7bfd
commit b0126c36bf
1 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
plugins/woocommerce-admin/tests/api/onboarding-tasks.php
View File

@ -32,14 +32,22 @@ class WC_Tests_API_Onboarding_Tasks extends WC_REST_Unit_Test_Case {
); );
} }
/**
* Tear down.
*/
public function tearDown() {
parent::tearDown();
$this->remove_color_or_logo_attribute_taxonomy();
}
/** /**
* Remove product attributes that where created in previous tests. * Remove product attributes that where created in previous tests.
*/ */
public function clear_product_attribute_taxonomies() { public function remove_color_or_logo_attribute_taxonomy() {
$taxonomies = get_taxonomies(); $taxonomies = get_taxonomies();
foreach ( (array) $taxonomies as $taxonomy ) { foreach ( (array) $taxonomies as $taxonomy ) {
// pa - product attribute. // pa - product attribute.
if ( 'pa_' === substr( $taxonomy, 0, 3 ) ) { if ( 'pa_color' === $taxonomy || 'pa_logo' === $taxonomy ) {
unregister_taxonomy( $taxonomy ); unregister_taxonomy( $taxonomy );
} }
} }
@ -51,7 +59,7 @@ class WC_Tests_API_Onboarding_Tasks extends WC_REST_Unit_Test_Case {
public function test_import_sample_products() { public function test_import_sample_products() {
wp_set_current_user( $this->user ); wp_set_current_user( $this->user );
$this->clear_product_attribute_taxonomies(); $this->remove_color_or_logo_attribute_taxonomy();
$request = new WP_REST_Request( 'POST', $this->endpoint . '/import_sample_products' ); $request = new WP_REST_Request( 'POST', $this->endpoint . '/import_sample_products' );
$response = $this->server->dispatch( $request ); $response = $this->server->dispatch( $request );
@ -65,7 +73,7 @@ class WC_Tests_API_Onboarding_Tasks extends WC_REST_Unit_Test_Case {
$this->assertArrayHasKey( 'skipped', $data ); $this->assertArrayHasKey( 'skipped', $data );
// There might be previous products present. // There might be previous products present.
if ( 0 === count( $data['skipped'] ) ) { if ( 0 === count( $data['skipped'] ) ) {
$this->assertGreaterThan( 10, count( $data['imported'] ) ); $this->assertGreaterThan( 1, count( $data['imported'] ) );
} }
$this->assertArrayHasKey( 'updated', $data ); $this->assertArrayHasKey( 'updated', $data );
$this->assertEquals( 0, count( $data['updated'] ) ); $this->assertEquals( 0, count( $data['updated'] ) );
@ -76,7 +84,6 @@ class WC_Tests_API_Onboarding_Tasks extends WC_REST_Unit_Test_Case {
*/ */
public function test_create_product_from_template() { public function test_create_product_from_template() {
wp_set_current_user( $this->user ); wp_set_current_user( $this->user );
$this->clear_product_attribute_taxonomies();
$request = new WP_REST_Request( 'POST', $this->endpoint . '/create_product_from_template' ); $request = new WP_REST_Request( 'POST', $this->endpoint . '/create_product_from_template' );
$request->set_param( 'template_name', 'physical' ); $request->set_param( 'template_name', 'physical' );