Skip to content

chore: (CXSPA-10166) Continuum E2Es - B2B Reorder #20358

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

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

StanislavSukhanov
Copy link
Contributor

(CXSPA-10166) Continuum E2Es - B2B Reorder
closes: https://jira.tools.sap/browse/CXSPA-10166

@StanislavSukhanov StanislavSukhanov self-assigned this May 22, 2025
@github-actions github-actions bot marked this pull request as draft May 22, 2025 09:29
Copy link
Contributor

🚨 PR Title Validation Failed 🚨

Your pull request title does not follow the required format. Please update it to match the expected pattern:

Expected format:
<type>: <subject>

Allowed Types

  • docs: Changes to documentation only
  • feat: New feature work
  • fix: Bug fixes
  • perf: Code improvements for performance
  • refactor: Code changes that are not bug fixes or features
  • style: Code style changes (e.g., whitespace, formatting)
  • test: Adding or updating tests
  • chore: Build, CI, scripts, configs, etc.

Example of a valid PR title

feat: Add user authentication
fix: Resolve checkout bug
docs: Update API documentation

Merge is blocked until the PR title is corrected.

1 similar comment
Copy link
Contributor

🚨 PR Title Validation Failed 🚨

Your pull request title does not follow the required format. Please update it to match the expected pattern:

Expected format:
<type>: <subject>

Allowed Types

  • docs: Changes to documentation only
  • feat: New feature work
  • fix: Bug fixes
  • perf: Code improvements for performance
  • refactor: Code changes that are not bug fixes or features
  • style: Code style changes (e.g., whitespace, formatting)
  • test: Adding or updating tests
  • chore: Build, CI, scripts, configs, etc.

Example of a valid PR title

feat: Add user authentication
fix: Resolve checkout bug
docs: Update API documentation

Merge is blocked until the PR title is corrected.

Copy link
Contributor

🚨 PR Title Validation Failed 🚨

Your pull request title does not follow the required format. Please update it to match the expected pattern:

Expected format:
<type>: <subject>

Allowed Types

  • docs: Changes to documentation only
  • feat: New feature work
  • fix: Bug fixes
  • perf: Code improvements for performance
  • refactor: Code changes that are not bug fixes or features
  • style: Code style changes (e.g., whitespace, formatting)
  • test: Adding or updating tests
  • chore: Build, CI, scripts, configs, etc.

Example of a valid PR title

feat: Add user authentication
fix: Resolve checkout bug
docs: Update API documentation

Merge is blocked until the PR title is corrected.

@StanislavSukhanov StanislavSukhanov changed the title (CXSPA-10166) Continuum E2Es - B2B Reorder chore: (CXSPA-10166) Continuum E2Es - B2B Reorder May 22, 2025
@StanislavSukhanov StanislavSukhanov marked this pull request as ready for review May 22, 2025 10:56
@github-actions github-actions bot marked this pull request as draft May 22, 2025 10:56
@StanislavSukhanov StanislavSukhanov marked this pull request as ready for review May 22, 2025 11:02
Copy link
Contributor

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

1 similar comment
Copy link
Contributor

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

Copy link

cypress bot commented May 22, 2025

spartacus    Run #48390

Run Properties:  status check failed Failed #48390  •  git commit d3d3eec4f4 ℹ️: Merge 1ce02cbd89ea1edd3a520bd98b7bfade94177ed9 into da0e035364973bc97dd2ae3d3a9b...
Project spartacus
Branch Review feature/CXSPA-10166-Continuum-E2Es---B2B-Reorder
Run status status check failed Failed #48390
Run duration 05m 51s
Commit git commit d3d3eec4f4 ℹ️: Merge 1ce02cbd89ea1edd3a520bd98b7bfade94177ed9 into da0e035364973bc97dd2ae3d3a9b...
Committer Stanislav Sukhanov
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 3
Tests that did not run due to a developer annotating a test with .skip  Pending 2
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 237
View all changes introduced in this branch ↗︎

Tests for review

Failed  cypress/e2e/accessibility/reorder.a11y-e2e.cy.ts • 1 failed test • A11Y

View Output Video

Test Artifacts
Reorder accessibility > Reorder Test Replay Screenshots Video
Flakiness  cypress/e2e/ssr/pages.core-e2e.cy.ts • 3 flaky tests • SSR

View Output Video

Test Artifacts
SSR > should render homepage Test Replay Screenshots Video
SSR > should render PLP Test Replay Screenshots Video
SSR > should render PDP Test Replay Screenshots Video

Copy link
Contributor

@Pio-Bar Pio-Bar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, let's wait for the B2B setup. :)

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