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

Bug Fixes Solutions module #1346

Merged
merged 2 commits into from
Oct 10, 2024
Merged

Bug Fixes Solutions module #1346

merged 2 commits into from
Oct 10, 2024

Conversation

aratidgr8
Copy link
Contributor

Proposed changes

Solutions module bug fixes:

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

Copy link

cypress bot commented Oct 10, 2024

Copy link

cypress bot commented Oct 10, 2024

Bluehost Brand Plugin    Run #10991

Run Properties:  status check failed Failed #10991  •  git commit eb28fe2df4 ℹ️: Merge 312527f11fec03b76cd6ce1a2679e2c04b84b343 into ed407fed778ae1d4bb36525fc196...
Project Bluehost Brand Plugin
Run status status check failed Failed #10991
Run duration 32m 46s
Commit git commit eb28fe2df4 ℹ️: Merge 312527f11fec03b76cd6ce1a2679e2c04b84b343 into ed407fed778ae1d4bb36525fc196...
Committer Arati Bhandare
View all properties for this run ↗︎

Test results
Tests that failed  Failures 8
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 36
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 408

Tests for review

Failed  tests/cypress/integration/home.cy.js • 5 failed tests

View Output Video

Test Artifacts
Home Page > Site Info Exists Test Replay Screenshots Video
Home Page > Is Accessible Test Replay Screenshots Video
Home Page > Welcome Section Exists Test Replay Screenshots Video
Home Page > Account Section Exists Test Replay Screenshots Video
Home Page > Help Section Exists Test Replay Screenshots Video
Failed  vendor/newfold-labs/wp-module-ecommerce/tests/cypress/integration/Home/ecommerce-next-steps.cy.js • 2 failed tests

View Output Video

Test Artifacts
e-commerce Home Page- Next Steps > Verify Next steps for your site when woocommerce is not active Test Replay Screenshots Video
e-commerce Home Page- Next Steps > Verify Next steps when experience level is novice Test Replay Screenshots Video
Failed  vendor/newfold-labs/wp-module-ecommerce/tests/cypress/integration/Home/migration.cy.js • 1 failed test

View Output Video

Test Artifacts
Home page - post migration events with help center > Verify when continue with store setup clicked Test Replay Screenshots Video

@circlecube circlecube merged commit 38af4c0 into solutions Oct 10, 2024
2 of 4 checks passed
@circlecube circlecube deleted the fix/solutions-bug-fixes branch October 10, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants