Skip to content

Conversation

@niehuayang
Copy link
Contributor

No description provided.

@github-actions github-actions bot marked this pull request as draft September 29, 2025 07:30
@github-actions
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.

@niehuayang niehuayang changed the title CXSPA-11031: refine e2e test test: refine e2e test Sep 29, 2025
@niehuayang niehuayang marked this pull request as ready for review September 29, 2025 07:47
@cypress
Copy link

cypress bot commented Sep 29, 2025

spartacus    Run #50322

Run Properties:  status check passed Passed #50322  •  git commit da5aab387a ℹ️: Merge 6db68609cfa9b82e3df84a2a38c534dfb9de17f6 into 9ffd5d0444381d087be187b27889...
Project spartacus
Branch Review fix/CXSPA-11031
Run status status check passed Passed #50322
Run duration 03m 37s
Commit git commit da5aab387a ℹ️: Merge 6db68609cfa9b82e3df84a2a38c534dfb9de17f6 into 9ffd5d0444381d087be187b27889...
Committer Niehua Yang
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
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 101
View all changes introduced in this branch ↗︎

@github-actions github-actions bot marked this pull request as draft October 10, 2025 03:02
@niehuayang niehuayang marked this pull request as ready for review October 10, 2025 05:41
@github-actions
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"`

@github-actions github-actions bot marked this pull request as draft October 10, 2025 07:44
@niehuayang niehuayang marked this pull request as ready for review October 10, 2025 08:16
).then((customerId) => {
cy.visit('/assisted-service/emulate?customerId=' + customerId);

agentLoginForJDK21(b2cAgent.userName, b2cAgent.password);
Copy link
Contributor

Choose a reason for hiding this comment

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

I am a little confused, don't we need it any more? how does the agent login without this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

agent already logged in the above 60th line, so no need to log in again

Copy link
Contributor

@Melody-zhou-512 Melody-zhou-512 left a comment

Choose a reason for hiding this comment

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

LGTM.

@github-actions github-actions bot marked this pull request as draft October 13, 2025 02:50
@niehuayang niehuayang marked this pull request as ready for review October 14, 2025 02:21
@LindaSAP LindaSAP merged commit cdd798e into develop Oct 14, 2025
39 checks passed
@LindaSAP LindaSAP deleted the fix/CXSPA-11031 branch October 14, 2025 02:47
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.

5 participants