Skip to content

fix: Resolve sonar issues (#21329)#21345

Open
rmch91 wants to merge 2 commits intorelease/221121.10.xfrom
cherry-pick/839365b-sonar-fixes
Open

fix: Resolve sonar issues (#21329)#21345
rmch91 wants to merge 2 commits intorelease/221121.10.xfrom
cherry-pick/839365b-sonar-fixes

Conversation

@rmch91
Copy link
Copy Markdown
Contributor

@rmch91 rmch91 commented Apr 8, 2026

No description provided.

@rmch91 rmch91 requested a review from a team as a code owner April 8, 2026 10:31
@github-actions github-actions bot marked this pull request as draft April 8, 2026 10:31
@rmch91 rmch91 marked this pull request as ready for review April 8, 2026 10:31
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

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"`

@cypress
Copy link
Copy Markdown

cypress bot commented Apr 8, 2026

spartacus    Run #52665

Run Properties:  status check failed Failed #52665  •  git commit a0fe71cee3 ℹ️: Merge 4d1410e20107a1f1586f3d4c4f99b27e4ef2e1ab into 4535dc1eba56a34c2df7f19df9d8...
Project spartacus
Branch Review cherry-pick/839365b-sonar-fixes
Run status status check failed Failed #52665
Run duration 05m 00s
Commit git commit a0fe71cee3 ℹ️: Merge 4d1410e20107a1f1586f3d4c4f99b27e4ef2e1ab into 4535dc1eba56a34c2df7f19df9d8...
Committer Roman
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 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 100
View all changes introduced in this branch ↗︎

Tests for review

Failed  cypress/e2e/regression/site-context/currency/currency.core-e2e.cy.ts • 1 failed test • B2C

View Output Video

Test Artifacts
Currency change > on the product page > should display the chosen currency Test Replay 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 Video
SSR > should render PLP Test Replay Video
SSR > should render PDP Test Replay Video

@github-actions github-actions bot marked this pull request as draft April 8, 2026 13:35
@rmch91 rmch91 marked this pull request as ready for review April 8, 2026 13:35
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

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"`

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.

1 participant