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

LPS-116693 baseline #2282

Closed
wants to merge 200 commits into from
Closed

LPS-116693 baseline #2282

wants to merge 200 commits into from

Conversation

ealonso
Copy link
Collaborator

@ealonso ealonso commented Sep 10, 2020

forward

brianchandotcom and others added 30 commits September 10, 2020 11:01
Igor Beslic and others added 8 commits September 10, 2020 12:30
				<aui:input name="commerceInventoryWarehouseId" type="hidden" />
				<aui:input name="commerceInventoryWarehouseItemId" type="hidden" />
				<aui:input name="sku" type="hidden" value="<%= cpInstance.getSku() %>" />
				<aui:input name="quantity" type="hidden" />
				<aui:input name="mvccVersion" type="hidden" />
…e I can administer for asset publisher and doesn't include recent sites and my sites
@ealonso
Copy link
Collaborator Author

ealonso commented Sep 10, 2020

ci:forward

:octocat: Sent from GH.

@liferay-continuous-integration
Copy link
Collaborator

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.

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering the following test suites:

  •     ci:test:relevant
  •     ci:test:sf

The pull request will automatically be forwarded to the user brianchandotcom if the following test suites pass:

  •     ci:test:relevant
  •     ci:test:sf
  •     ci:test:stable

@liferay-continuous-integration
Copy link
Collaborator

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

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: a6542fe6c2e05052e83df0ba1aabd5e4b8aa1e36

Sender Branch:

Branch Name: LPS-106461
Branch GIT ID: a71bc3448ea4904e5624a24a7e4133dcb72221f0

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

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

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

❌ ci:test:relevant - 89 out of 92 jobs passed in 2 hours 21 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 049adaa8060bfec3681b787a5186107b630d7cef

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: 35186087dfbc07539501385ca053c52db3343bb7

ci:test:stable - 21 out of 21 jobs PASSED
21 Successful Jobs:
ci:test:relevant - 89 out of 92 jobs PASSED
89 Successful Jobs:
For more details click here.

Failures unique to this pull:


Failures in common with acceptance upstream results at a6542fe:
  1. test-portal-acceptance-pullrequest-batch(master)/modules-integration-mysql57-jdk8
    Job Results:

    2705 Tests Passed.
    1 Test Failed.

    1. AXIS_VARIABLE=4,label_exp=!master #437554
      1. DLFileEntryTypeServiceTest.testUpdateFileEntryTypeWithEmptyDDMForm
        com.liferay.dynamic.data.mapping.validator.DDMFormValidationException$MustSetValidType: Invalid type set for field type Text
        	at com.liferay.dynamic.data.mapping.validator.internal.DDMFormValidatorImpl.validateDDMFormFieldType(DDMFormValidatorImpl.java:299)
        	at com.liferay.dynamic.data.mapping.validator.internal.DDMFormValidatorImpl.validateDDMFormFields(DDMFormValidatorImpl.java:258)
        	at com.liferay.dynamic.data.mapping.validator.internal.DDMFormValidatorImpl.validate(DDMFormValidatorImpl.java:97)
        	at com.liferay.dynamic.data.mapping.service.impl.DDMStructureLocalServiceImpl.validate(DDMStructureLocalServiceImpl.java:1913)
        	at com.liferay.dynamic.data.mapping.service.impl.DDMStructureLocalServiceImpl.validate(DDMStructureLocalServiceImpl.java:1963)
        	at com.liferay.dynamic.data.mapping.service.impl.DDMStructureLocalServiceImpl.validate(DDMStructureLocalServiceImpl.java:1951)
        	at com.liferay.dynamic.data.mapping.service.impl.DDMStructureLocalServiceImpl.addStructure(DDMStructureLocalServiceImpl.java:169)
        	at com.liferay.dynamic.data.mapping.service.impl.DDMStructureLocalServiceImpl.addStructure(DDMStructureLocalServiceImpl.java:314)
        	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        	at java.lang.reflect.Method.invoke(Method.java:498)
        	at com.liferay.portal.spring.aop.AopMethodInvocationImpl....

@liferay-continuous-integration
Copy link
Collaborator

@Tim-Cao
Copy link

Tim-Cao commented Sep 11, 2020

Just start reviewing

@Tim-Cao
Copy link

Tim-Cao commented Sep 11, 2020

Hi @ealonso, your changes cause the following regression issue
Add a widget page
Add an Asset Publisher to page
Click the ellipsis button on the topper of Asset Publisher
Configuration > Setup > Asset Selection
Expand Scope
Click Select
Other Site or Asset Library...

Expected Results:
Expected

Actual Results:
Actual

@ealonso
Copy link
Collaborator Author

ealonso commented Sep 11, 2020

Hey @Tim-Cao

The second screenshot is the expected behaviour, since for some reason the Lima Team change the behaviour in our component. I made some changes to keep the behaviour that we have in 7.2, and it is the second screenshot.

Could you please take a look and fix the poshi tests?

Regards,
Eudaldo.

@Tim-Cao
Copy link

Tim-Cao commented Sep 11, 2020

ci:close

@Tim-Cao
Copy link

Tim-Cao commented Sep 11, 2020

See #2289

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
9 participants