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-125728 Fix render of empty accordions in Forms #709

Closed
wants to merge 2 commits into from

Conversation

edalgrin
Copy link
Collaborator

This PR fixes the accordion bug in Forms https://issues.liferay.com/browse/LPS-125728

The bug was caused by the empty tab, probably because the responsible script is trying to get an height size.

ezgif com-gif-maker (1)

Since keeping the empty tab it's actually an UX error, I preferred to remove it instead of adjusting the script for this corner case.

ezgif com-gif-maker

@marcoscv-work
Copy link

I agree with: Since keeping the empty tab it's actually an UX error, I preferred to remove it instead of adjusting the script for this corner case.

:-D

@matuzalemsteles
Copy link

Hey @edalgrin and @jbalsas I do not see any problems in proceeding with this change since we are going to remove this Sidebar in the ticket we are working on 🤪 to use the Data Engine MultiPanelSidebar 😁.

@jbalsas
Copy link

jbalsas commented Jan 19, 2021

ci:forward

@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 5 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: a4b3d0cdbc5bdfd9adff1ab24a594c02820dd567

Sender Branch:

Branch Name: LPS-125728
Branch GIT ID: 214d803364fef9f68778016e7ebb64b22523bf45

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 - 8 out of 9 jobs passed

❌ ci:test:relevant - 23 out of 25 jobs passed in 1 hour 32 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 1450345f4b675c215f214c8a680ad2082e80ac39

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: 2b89e6be0ae0f6e12dec390930d3e9525006e687

ci:test:stable - 8 out of 9 jobs PASSED
8 Successful Jobs:
ci:test:relevant - 23 out of 25 jobs PASSED
23 Successful Jobs:
For more details click here.

Failures unique to this pull:

  1. test-portal-acceptance-pullrequest-batch(master)/js-unit-jdk8/0
    Job Results:

    0 Tests Passed.
    1 Test Failed.

    1. AXIS_VARIABLE=0,label_exp=!master #456649
           [exec] * What went wrong:
           [exec] Execution failed for task ':apps:dynamic-data-mapping:dynamic-data-mapping-form-builder:packageRunTest'.
           [exec] > Process 'command '/opt/dev/projects/github/liferay-portal/build/node/bin/node'' finished with non-zero exit value 1
           [exec] 
           [exec] * Try:
           [exec] Run with --info or --debug option to get more log output. Run with --scan to get full insights.
           [exec] 
           [exec] * Exception is:
           [exec] org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':apps:dynamic-data-mapping:dynamic-data-mapping-form-builder:packageRunTest'.
           [exec] 	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.lambda$executeIfValid$1(ExecuteActionsTaskExecuter.java:208)
           [exec] 	at org.gradle.internal.Try$Failure.ifSuccessfulOrElse(Try.java:263)
           [exec] 	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeIfValid(ExecuteActionsTaskExecuter.java:206)
           [exec] 	at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.execute(ExecuteActionsTaskExecuter.java:187)
           [exec] 	at org.gradle.api.internal.tasks.execution.CleanupStaleOutputsExecuter.execute(CleanupStaleOutputsExecuter.java:114)
           [exec] 	at org.gradle.api.internal.tasks.execution.FinalizePropertiesTaskExecuter.execute(FinalizePropertiesTaskExecuter.java:46)
           [exec] 	at org.gradle.api.internal.tasks.execution.ResolveTaskExecutionModeExecuter.execute(ResolveTaskExecutionModeExecuter.java:62)
           [exec] 	at org.gradle.api.internal.tasks.execution.SkipTaskWithNoActionsExecuter.execute(SkipTaskWithNoActionsExecuter.java:57)
           [exec] 	at org.gradle.api.internal.tasks.execution.SkipOnlyIfTaskExecuter.execute(SkipOnlyIfTaskExecuter.java:56)
           [exec] 	at org.gradle.api.internal.tasks.execution.CatchExceptionTaskExecuter.execute(CatchExceptionTaskExecuter.java:36)
           [exec] 	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.executeTask(EventFiringTaskExecuter.java:77)
           [exec] 	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:55)
           [exec] 	at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:52)
           [exec] 	at org.gradle.internal.operations.DefaultBuildOperationExecutor$CallableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:409)

For upstream results, click here.

@liferay-continuous-integration
Copy link
Collaborator

@edalgrin
Copy link
Collaborator Author

ci:test:relevant

@liferay-continuous-integration
Copy link
Collaborator

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

✔️ ci:test:relevant - 22 out of 25 jobs passed in 1 hour 48 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: d87f3c5df11698d064d67445ab4f79097ebf3d80

Upstream Comparison:

Branch GIT ID: 0dddcd3bd491ba7902755f0c4138befa610c3947
Jenkins Build URL: Acceptance Upstream DXP (master) #1496

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

This pull contains no unique failures.


Failures in common with acceptance upstream results at 0dddcd3:
  1. test-portal-acceptance-pullrequest-batch(master)/modules-semantic-versioning-jdk8/0
    Job Results:

    0 Tests Passed.
    1 Test Failed.

    1. AXIS_VARIABLE=0,label_exp=!master #412985

      Please fix semantic versioning on edalgrin/LPS-125728

           [exec] 			-   return     void
           [exec] 		-   method     getFieldType()
           [exec] 			-   return     java.lang.String
           [exec] 		-   method     hashCode()
           [exec] 			-   return     int
           [exec] 		-   method     toJSON(com.liferay.portal.kernel.repository.model.FileEntry)
           [exec] 			-   access     protected
           [exec] 			-   return     java.lang.String
           [exec] 		-   method     toString()
           [exec] 			-   return     java.lang.String
           [exec] 		-   method     transform(com.liferay.dynamic.data.mapping.storage.DDMFormFieldValue)
           [exec] 			-   return     void
           [exec] 	+   class      com.liferay.dynamic.data.mapping.util.SettingsDDMFormFieldsUtil
           [exec] 		+   method     <init>()
           [exec] 			+   return     void
           [exec] 		+   method     clone()
           [exec] 			+   access     protected
           [exec] 			+   return     java.lang.Object
           [exec] 		+   method     equals(java.lang.Object)
           [exec] 			+   return     boolean
           [exec] 		+   method     finalize()
           [exec] 			+   access     protected
           [exec] 			+   return     void
           [exec] 		+   method     getSettingsDDMFormFields(com.liferay.dynamic.data.mapping.form.field.type.DDMFormFieldTypeServicesTracker,java.lang.String)
           [exec] 			+   access     static
           [exec] 			+   return     java.util.Map
           [exec] 		+   method     hashCode()
           [exec] 			+   return     int
           [exec] 		+   method     toString()
           [exec] 			+   return     java.lang.String
           [exec] 	+   interface  com.liferay.dynamic.data.mapping.util.DDMDataDefinitionConverter
           [exec] 		+   access     abstract
           [exec] 		+   method     convert(com.liferay.dynamic.data.mapping.model.DDMForm,java.util.Locale)
           [exec] 			+   access     abstract
           [exec] 			+   return     java.lang.String
           [exec] 		+   method     convert(java.lang.String,java.util.Locale)
           [exec] 			+   access     abstract
           [exec] 			+   return     java.lang.String
           [exec] 	-   version    2.7.0
           [exec] 	+   version    2.8.0
           [exec] [Baseline Report] Mode: diff (persisted)
           [exec] Semantic versioning is incorrect while checking /opt/dev/projects/github/liferay-portal/tools/sdk/dist/com.liferay.dynamic.data.mapping.api-9.0.0.jar against /opt/dev/projects/github/liferay-portal/.gradle/caches/modules-2/files-2.1/com.liferay/com.liferay.dynamic.data.mapping.api/8.11.2/61bc64754fb215cadedf88e952ddfcbcd78c7188/com.liferay.dynamic.data.mapping.api-8.11.2.jar

  2. test-portal-acceptance-pullrequest-batch(master)/modules-unit-jdk8/0
    Job Results:

    805 Tests Passed.
    7 Tests Failed.

    1. AXIS_VARIABLE=0,label_exp=!master #457449
      1. DDMFormUploadValidatorTest.testInvalidFileException
        java.lang.ExceptionInInitializerError
        	at com.liferay.portal.configuration.metatype.bnd.util.ConfigurableUtil._overrideDictionary(ConfigurableUtil.java:217)
        	at com.liferay.portal.configuration.metatype.bnd.util.ConfigurableUtil.createConfigurable(ConfigurableUtil.java:51)
        	at com.liferay.dynamic.data.mapping.form.web.internal.upload.DDMFormUploadValidatorTest.setUpDDMFormWebConfigurationActivator(DDMFormUploadValidatorTest.java:96)
        	at com.liferay.dynamic.data.mapping.form.web.internal.upload.DDMFormUploadValidatorTest.setUp(DDMFormUploadValidatorTest.java:44)
        	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 org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
        	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
        	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
        	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
        	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
        	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
        	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
        	at org.junit.runners.ParentRunner$3.run(ParentRunner.ja...
      2. DDMFormUploadValidatorTest.testValidFileSize
        java.lang.NoClassDefFoundError: Could not initialize class com.liferay.portal.configuration.persistence.ConfigurationOverridePropertiesUtil
        	at com.liferay.portal.configuration.metatype.bnd.util.ConfigurableUtil._overrideDictionary(ConfigurableUtil.java:217)
        	at com.liferay.portal.configuration.metatype.bnd.util.ConfigurableUtil.createConfigurable(ConfigurableUtil.java:51)
        	at com.liferay.dynamic.data.mapping.form.web.internal.upload.DDMFormUploadValidatorTest.setUpDDMFormWebConfigurationActivator(DDMFormUploadValidatorTest.java:96)
        	at com.liferay.dynamic.data.mapping.form.web.internal.upload.DDMFormUploadValidatorTest.setUp(DDMFormUploadValidatorTest.java:44)
        	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 org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
        	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
        	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
        	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
        	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
        	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
        	at org.junit.runners.BlockJUnit4ClassRu...
      3. DDMFormWebConfigurationModelListenerTest.testNegativeAutosaveIntervalShouldThrowException
        java.lang.Exception: Unexpected exception, expected<com.liferay.portal.configuration.persistence.listener.ConfigurationModelListenerException> but was<java.lang.ExceptionInInitializerError>
        	at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl$PowerMockJUnit44MethodRunner.handleException(PowerMockJUnit44RunnerDelegateImpl.java:364)
        	at org.powermock.modules.junit4.internal.impl.PowerMockJUnit47RunnerDelegateImpl$PowerMockJUnit47MethodRunner.handleException(PowerMockJUnit47RunnerDelegateImpl.java:122)
        	at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl$PowerMockJUnit44MethodRunner.handleInvocationTargetException(PowerMockJUnit44RunnerDelegateImpl.java:337)
        	at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl$PowerMockJUnit44MethodRunner.runTestMethod(PowerMockJUnit44RunnerDelegateImpl.java:315)
        	at org.junit.internal.runners.MethodRoadie$2.run(MethodRoadie.java:89)
        	at org.junit.internal.runners.MethodRoadie.runBeforesThenTestThenAfters(MethodRoadie.java:97)
        	at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl$PowerMockJUnit44MethodRunner.executeTest(PowerMockJUnit44RunnerDelegateImpl.java:294)
        	at org.powermock.modules.junit4.internal.impl.PowerMockJUnit47RunnerDelegateImpl$PowerMockJUnit47MethodRunner.executeTestInSuper(PowerMockJUnit47RunnerDelegateImpl.java:127)
        	at org.powermock.modules.junit4.internal.impl.PowerMockJUnit47RunnerDelegateImpl$PowerMockJUni...
      4. ...

@liferay-continuous-integration
Copy link
Collaborator

@zhangyan0701
Copy link

ci:test:bundle

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:bundle - 1 out of 1 jobs passed in 48 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 8c437d084cef9ec3e4e1a3382be525bf6875254a

ci:test:bundle - 1 out of 1 jobs PASSED
For more details click here.
Test bundle downloads:

@edalgrin
Copy link
Collaborator Author

ci:forward

@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

Skipping previously passed test suites:
ci:test:relevant

@liferay-continuous-integration
Copy link
Collaborator

✔️ 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: d87f3c5df11698d064d67445ab4f79097ebf3d80

Sender Branch:

Branch Name: LPS-125728
Branch GIT ID: cf0ef2c192f31ab31e37ef383918b56dacfe765f

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

@liferay-continuous-integration
Copy link
Collaborator

All required test suite(s) passed.
Forwarding pullrequest to brianchandotcom.

@liferay-continuous-integration
Copy link
Collaborator

Pull request has been successfully forwarded to brianchandotcom#98232

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

@edalgrin edalgrin deleted the LPS-125728 branch February 10, 2021 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants