-
Notifications
You must be signed in to change notification settings - Fork 1
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
[TS] LPS-155811 Incorrect assets shown for collections when combined segments enabled and user belongs to segments without personalized variations #2532
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. |
Hi @liferay-tango, Please let me know if you need further information to move this pr along. Thanks! |
ci:test:sf |
ci:test:relevant |
ci:test:tango |
✔️ 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-155811 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#1043 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-tango#2532 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#3318 |
Jenkins Build:test-portal-acceptance-pullrequest(master)#3653 Jenkins Report:jenkins-report.html Jenkins Suite:tango Pull Request:liferay-tango#2532 Testray Routine:EE Pull Request Testray Build:[master] ci:test:tango - ricardocousolr > liferay-tango - PR#2532 - 2022-06-15[02:02:10] Testray Importer:publish-testray-report#9463 |
✔️ ci:test:stable - 26 out of 27 jobs passed✔️ ci:test:relevant - 123 out of 130 jobs passed in 3 hours 21 minutesClick here for more details.Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: d8ad49e8b7f162a4882b5054b9ea4ddcae3289db ci:test:stable - 26 out of 27 jobs PASSED1 Failed Jobs:
26 Successful Jobs:
ci:test:relevant - 122 out of 130 jobs PASSED8 Failed Jobs:
122 Successful Jobs:
For more details click here.This pull contains no unique failures.Failures in common with acceptance upstream results at d8ad49e:
|
Jenkins Build:test-portal-acceptance-pullrequest(master)#4334 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-tango#2532 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - ricardocousolr > liferay-tango - PR#2532 - 2022-06-15[02:02:11] Testray Importer:publish-testray-report#12130 |
Hi @liferay-tango, Is it necessary to launch the tango tests again since some have failed? |
Hey @ricardocousolr !! The errors look unrelated, I am going to forward this PR. Thanks!! |
ci:forward |
CI is automatically triggering the following test suites:
The pull request will automatically be forwarded to the user
|
Skipping previously passed test suites: |
All required test suite(s) passed. |
Pull request has been successfully forwarded to brianchandotcom#118981 |
Hi @liferay-tango!
This is a solution to LPS-155811 as agreed in PTR-3103.
In
master
the only issue to fix is making sure that we return the array[0]
when none of the segments the user belongs to have an associated personalized variation.(For older versions it will be necessary to backport the filtering by
Objects::nonNull
to discard particular segments without personalized variations.)As explained in the LPS-155811, this issue can only be tested (in
master
) for manual collections, because for dynamic collections it's masked by another problem. Nevertheless, the fix in this pr should work for both collection types.Please let me know if you have any questions.
Thanks!