Automattic / wc-calypso-bridge

20 stars 4 forks source link

[Test] Configure Repo with CI and Create Test for ecomm/store #531

Closed timmyc closed 4 years ago

timmyc commented 4 years ago

In order to improve the quality of the experience in the ecommerce plan, we need to begin implementing test coverage for the functionality implemented within the Calypso Bridge plugin. There is a test suite now with only example assertions, so for this issue, let's create a test that asserts the proper set of logic is loaded for both scenarios:

Lastly we need to setup CI on this repository to run the test suite against each branch.

timmyc commented 4 years ago

I spent some time trying to get the test suite to run locally for me, and hit the following:

Error: Looks like you're using PHPUnit 8.0.6. WordPress is currently only compatible with PHPUnit up to 7.x. Please use the latest PHPUnit version from the 7.x branch.

So thinking we need to just get the test suite up to the current Woo Core standards so we can continue to use PHPUnit 8.x as we are in wc-admin.

Also the README instructions on the test suite here will need some <3 too.

timmyc commented 4 years ago

Some e2e test ideas:

timmyc commented 4 years ago

Since CI is setup now, I'm going to close this out. We can open up some e2e specific issues likely when we integrate the nav proejct.