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

LPD-48005 Fix EditSiteURLMVCActionCommandTest #158461

Conversation

liferay-continuous-integration
Copy link
Collaborator

Forwarded from: liferay-site-management#1777 (Took 1 ci:forward attempt in 41 seconds)
Console

@jonathanmccann
@liferay-site-management

Original pull request comment:

What is this trying to solve?

https://liferay.atlassian.net/browse/LPD-48005

How am I fixing it?

The redirect returns a URL with a friendly URL converted to lower case. The test has been updated to match.

How can you verify that it works?

In site-test run gw testIntegration --tests *.EditSiteURLMVCActionCommandTest

@liferay-continuous-integration
Copy link
Collaborator Author

✔️ ci:test:sf - 1 out of 1 jobs passed in 5 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: db2576e4761a100b32b94733294292860a5a978d

Sender Branch:

Branch Name: LPD-48005
Branch GIT ID: ec05b9b6acac05748b81205e60e8c2c708d1a768

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator Author

✔️ ci:test:stable - 24 out of 24 jobs passed

✔️ ci:test:relevant - 29 out of 29 jobs passed in 1 hour 23 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: db2576e4761a100b32b94733294292860a5a978d

Upstream Comparison:

Branch GIT ID: 1ff5ca3783a255eeae04ac0b27324c410d212771
Jenkins Build URL: EE Development Acceptance (master) - 1165 - 2025-02-03[04:36:51]

ci:test:stable - 24 out of 24 jobs PASSED
24 Successful Jobs:
ci:test:relevant - 29 out of 29 jobs PASSED
29 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator Author

To conserve resources, the PR Tester does not automatically run for forwarded pull requests.

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 277bd90...30b3d9f

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.

3 participants