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
LPS-142436 | master #6474
LPS-142436 | master #6474
Conversation
To conserve resources, the PR Tester does not automatically run for every pull. If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed. If your pull was never tested, comment "ci:test" to run the PR Tester for this pull. |
...tional/com/liferay/portalweb/tests/enduser/wem/layout/widgetpage/WidgetPagesUseCase.testcase
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ozysouza, could you update according to comments? Btw, please resolve rebase conflict. Thanks
portal-web/test/functional/com/liferay/portalweb/paths/portlet/sitepages/SitePages.path
Outdated
Show resolved
Hide resolved
portal-web/test/functional/com/liferay/portalweb/macros/ManagementBar.macro
Outdated
Show resolved
Hide resolved
portal-web/test/functional/com/liferay/portalweb/macros/layout/sitepage/Page.macro
Outdated
Show resolved
Hide resolved
portal-web/test/functional/com/liferay/portalweb/paths/portlet/sitepages/SitePages.path
Outdated
Show resolved
Hide resolved
portal-web/test/functional/com/liferay/portalweb/macros/layout/pageeditor/PageEditor.macro
Outdated
Show resolved
Hide resolved
...est/functional/com/liferay/portalweb/tests/enduser/wem/layout/pagesadmin/PagesAdmin.testcase
Outdated
Show resolved
Hide resolved
...est/functional/com/liferay/portalweb/tests/enduser/wem/layout/pagesadmin/PagesAdmin.testcase
Outdated
Show resolved
Hide resolved
...est/functional/com/liferay/portalweb/tests/enduser/wem/layout/pagesadmin/PagesAdmin.testcase
Outdated
Show resolved
Hide resolved
...est/functional/com/liferay/portalweb/tests/enduser/wem/layout/pagesadmin/PagesAdmin.testcase
Outdated
Show resolved
Hide resolved
...est/functional/com/liferay/portalweb/tests/enduser/wem/layout/pagesadmin/PagesAdmin.testcase
Outdated
Show resolved
Hide resolved
…and modify effected cases
Hi @Tim-Cao, PR updated following your comments and conflicts solved. Would you mind run the SF locally to see if everything is okay? I don't know why but my source format is not working today, I've let a wrong formatation on porpuse to check and the build got successeful even though it should fail then I did the SF manually and I maybe missed something. Additionally I added PagesAdminWithStaging in the PR. Thank you |
ci:test:sf |
✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: LPS-142436 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#1005 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-echo#6474 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#3455 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ozysouza, could you update according to new comments? Thanks!
...l/com/liferay/portalweb/macros/sitenavigation/sitenavigationmenu/NavigationMenusWidget.macro
Outdated
Show resolved
Hide resolved
...st/functional/com/liferay/portalweb/tests/enduser/wem/layout/pagesadmin/PagesFinder.testcase
Show resolved
Hide resolved
...st/functional/com/liferay/portalweb/tests/enduser/wem/layout/pagesadmin/PagesFinder.testcase
Show resolved
Hide resolved
...nal/com/liferay/portalweb/tests/enduser/wem/layout/pagesadmin/PagesAdminWithStaging.testcase
Outdated
Show resolved
Hide resolved
Hi @ozysouza, see changes 8e1cd27...56b59f5 |
ci:close |
https://issues.liferay.com/browse/LPS-142436
cc @david-gutierrez-mesa
cc @Tim-Cao