Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement/E2E: Added some additional tests including tests related to subscriptions and pre-orders. #152

Merged
merged 41 commits into from
Aug 2, 2024

Conversation

iamdharmesh
Copy link
Member

@iamdharmesh iamdharmesh commented Jun 12, 2024

All Submissions:

  • Does your code follow the WooCommerce Sniffs variant of WordPress coding standards?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?
  • Will this change require new documentation or changes to existing documentation?

Changes proposed in this Pull Request:

The PR adds some E2E tests from the critical flow to increase test coverage and also adds E2E tests for the subscriptions and pre-orders products.

Closes #118

Steps to test the changes in this Pull Request:

  1. Verify E2E tests passing in GH action or in local

Changelog entry

Dev - Additional E2E tests to increase Critical Flows test coverage.

@iamdharmesh iamdharmesh self-assigned this Jun 12, 2024
@vikrampm1 vikrampm1 added this to the Future Release milestone Jun 12, 2024
@iamdharmesh iamdharmesh mentioned this pull request Jun 17, 2024
@peterwilsoncc peterwilsoncc force-pushed the enhancement/e2e branch 2 times, most recently from e25660f to 5ad4812 Compare June 18, 2024 02:56
@peterwilsoncc peterwilsoncc force-pushed the enhancement/e2e branch 3 times, most recently from 7011d77 to 957afd5 Compare June 18, 2024 04:36
@iamdharmesh iamdharmesh changed the title [WIP] Enhancement/E2E Enhancement/E2E: Added some additional tests including tests related to subscriptions and pre-orders. Jun 20, 2024
@iamdharmesh iamdharmesh requested a review from dkotter June 20, 2024 14:10
@jeffpaul jeffpaul added the changelog: none Skip changelog entry for this PR label Jun 24, 2024
@dkotter
Copy link
Contributor

dkotter commented Jul 22, 2024

@iamdharmesh There are some merge conflicts here that need fixed. Can you take a look at those?

@iamdharmesh
Copy link
Member Author

@dkotter Done. Some newly added tests are failing due to NUX changes, I will update it to fix it.

@iamdharmesh
Copy link
Member Author

@dkotter Done.

@qasumitbagthariya
Copy link
Contributor

Regression / Smoke Test Report ✅

Tested with Archive File created via "php woorelease.phar build repo_URL" (Composer version 2.5.5, npm version 8.19.4, node version 16.20.0)

Status- The E2E test is passed, Working expected with Plugin Archive/Zip file same as fix specific branch.

Next Step- Ready to Merge 🚀

@vikrampm1 vikrampm1 marked this pull request as ready for review August 2, 2024 14:40
@vikrampm1 vikrampm1 modified the milestones: Future Release, 4.7.3 Aug 2, 2024
@vikrampm1 vikrampm1 merged commit 8aa91fb into trunk Aug 2, 2024
6 checks passed
@vikrampm1 vikrampm1 deleted the enhancement/e2e branch August 2, 2024 14:42
@vikrampm1 vikrampm1 mentioned this pull request Aug 2, 2024
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increase E2E test coverage
6 participants