Ecommerce version bump 1.3.33

[aratidgr8]

Proposed changes

• PRESS0-1325 : Post Migration scripts by @krsomayagi in Post Migration scripts newfold-labs/wp-module-ecommerce#307
• PRESS0-1159 : Update Store Home Page by @aratidgr8 @manikantakailasa in Press0 1159 newfold-labs/wp-module-ecommerce#308
• PRESS0-1448: Remove Unused Thumbnail Service Code by @arunshenoy99 in Remove Unused Thumbnail Service Code newfold-labs/wp-module-ecommerce#315
• Ecommerce version bump 1.3.32 by @aratidgr8 in Ecommerce version bump 1.3.32 newfold-labs/wp-module-ecommerce#326
• PRESS0-1497 | Fix sitecapability and store page test failures in plugin workflow by @sangeetha-nayak in PRESS4-168 Pass Marketplace component to eCommerce module #322
• eCommerce version bump 1.3.33 by @ramyakrishnai in bump onboarding to 1.1.2 #327

Type of Change

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation Update (if none of the other choices apply)

Checklist

• I have read the CONTRIBUTING doc
• Linting and tests pass locally with my changes
• I have added tests that prove my fix is effective or that my feature works
• I have added necessary documentation (if appropriate)

Further comments

[cypress]

6 failed tests on run #8743 ↗︎

6

384

59

0

0

Details:

Merge 4c57fb0 into 6e8d8bb...
Project: Bluehost Brand Plugin	Commit: 7a0e6e1352 ℹ️
Status: Failed	Duration: 25:10 💡
Started: Jun 19, 2024 5:40 AM	Ended: Jun 19, 2024 6:05 AM

  vendor/newfold-labs/wp-module-coming-soon/tests/cypress/integration/coming-soon.cy.js • 1 failed test

View Output Video

Test		Artifacts
Coming Soon > Launching launches site		Test Replay Screenshots Video

  vendor/newfold-labs/wp-module-deactivation/tests/cypress/integration/deactivation-survey.cy.js • 2 failed tests

View Output Video

Test		Artifacts
Plugin Deactivation Survey > Continue button exists and advances to survey, skip button functions		Test Replay Screenshots Video
Plugin Deactivation Survey > Survey successfully deactivates plugin		Test Replay Screenshots Video

  vendor/newfold-labs/wp-module-marketplace/tests/cypress/integration/product-page.cy.js • 3 failed tests

View Output Video

Test		Artifacts
Product Page > Show loading state while fetching		Test Replay Screenshots Video
Product Page > Product page content is visible		Test Replay Screenshots Video
Product Page > Show error state if fetching fails		Test Replay Screenshots Video

Review all test suite changes for PR #1117 ↗︎

[circlecube]

Looks like these latest e-commerce module releases (1.3.32 and 1.3.33) do not have passing cypress tests. I see there are more tests being skipped and still failing tests not being skipped. Ideally we do not skip tests but can update them to pass. This is the only way to ensure code integrity and functional products for users.

Passing tests is a requirement for tagging a module release.

[aratidgr8]

@sangeetha-nayak Can you please take a look at failed cypress tests?

[ramyakrishnai]

Hi @circlecube , the latest ecommerce version 1.3.33 has been released to fix all the ecommerce related tests, In the current PR, the tests that are failing are from other modules, coming soon and deactivation module. We skipped a test that is wondercart related as that is breaking because of console errors coming from features module. So, we skipped it for now until it is fixed.

[circlecube]

Branches should follow the naming convention so that workflows will trigger tests to run. See https://newfold-labs.github.io/how-we-work/9-version-control.html#branch-naming and https://github.com/bluehost/bluehost-wordpress-plugin/blob/main/.github/workflows/cypress.yml#L6-L15

[circlecube]

Since we have a conflict and branch name didn't trigger tests, I've set a new PR up at #1128. Closing this one.