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-82408 Add unit test #740

Closed
wants to merge 5 commits into from
Closed

Conversation

pedroqueiroz
Copy link

@pedroqueiroz pedroqueiroz commented Jun 20, 2018

No description provided.

@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.

@pedroqueiroz
Copy link
Author

ci:test:sf

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 1 minute 22 seconds 124 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 46c9ef5ff3732f9ecef957d4f513028af3fdd7d7

Sender Branch:

Branch Name: LPS-82408
Branch GIT ID: 1ea7619fc2d11b20639997db039f21da0928d82e

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

@rafaprax
Copy link
Owner

ci:test:relevant

@rafaprax
Copy link
Owner

ci:test:relevant

Repository owner deleted a comment from liferay-continuous-integration Jun 22, 2018
Repository owner deleted a comment from liferay-continuous-integration Jun 22, 2018
Repository owner deleted a comment from liferay-continuous-integration Jun 22, 2018
@rafaprax
Copy link
Owner

ci:test:relevant

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:relevant - 28 out of 31 jobs passed in 1 hour 35 minutes 21 seconds 902 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: d4c1ddb40e1e0d549dc197493995a5a8b67ebe09

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: 293d52d53b17ef6e378a0f91e62a33d1613da1d9

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

Failures unique to this pull:


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

    601 Tests Passed.
    2 Tests Failed.

    1. AXIS_VARIABLE=0,label_exp=!master #213794
      1. PortalLogAssertorTest.testScanXMLLog
        junit.framework.AssertionFailedError: 
        [com.liferay.portal.workflow.kaleo.runtime.internal.activator.KaleoActivator(811)] The activate method has thrown an exception 
        com.liferay.portal.kernel.exception.ModelListenerException: com.liferay.portal.kernel.exception.NoSuchResourceActionException: There are no actions associated with the resource com.liferay.portal.workflow.kaleo.model.KaleoDefinitionVersion
        	at com.liferay.portal.workflow.kaleo.designer.web.internal.messaging.KaleoDefinitionVersionModelListener.onAfterCreate(KaleoDefinitionVersionModelListener.java:48)
        	at com.liferay.portal.workflow.kaleo.designer.web.internal.messaging.KaleoDefinitionVersionModelListener.onAfterCreate(KaleoDefinitionVersionModelListener.java:33)
        	at com.liferay.portal.kernel.service.persistence.impl.BasePersistenceImpl.update(BasePersistenceImpl.java:361)
        	at com.liferay.portal.workflow.kaleo.service.impl.KaleoDefinitionVersionLocalServiceImpl.addKaleoDefinitionVersion(KaleoDefinitionVersionLocalServiceImpl.java:93)
        	at com.liferay.portal.spring.aop.ServiceBeanMethodInvocation.proceed(ServiceBeanMethodInvocation.java:158)
        	at com.liferay.portal.spring.transaction.DefaultTransactionExecutor.execute(DefaultTransactionExecutor.java:54)
        	at com.liferay.portal.spring.transaction.TransactionInterceptor.invoke(TransactionInterceptor.java:58)
        	at com.liferay.portal.spring.aop.ServiceBeanMethodInvocation.proceed(ServiceBeanMethodInvocation.java:135)
        	at com.liferay.portal.spring.aop.RetryAdvice.invok...
      2. PortalLogAssertorTest.testScanOSGiLog
        junit.framework.AssertionFailedError: 
        Portal log assert failure, OSGi log found: /opt/dev/projects/github/liferay-portal/bundles/osgi/state/1529697009436.log:
        !SESSION 2018-06-22 19:51:50.088 -----------------------------------------------
        eclipse.buildId=unknown
        java.version=1.8.0_121
        java.vendor=Oracle Corporation
        BootLoader constants: OS=linux, ARCH=x86_64, WS=gtk, NL=en_US
        

        !ENTRY com.liferay.portal.workflow.kaleo.runtime.impl 4 0 2018-06-22 19:51:50.088
        !MESSAGE [com.liferay.portal.workflow.kaleo.runtime.internal.activator.KaleoActivator(811)] The activate method has thrown an exception
        !STACK 0
        com.liferay.portal.kernel.exception.ModelListenerException: com.liferay.portal.kernel.exception.NoSuchResourceActionException: There are no actions associated with the resource com.liferay.portal.workflow.kaleo.model.KaleoDefinitionVersion
        at com.liferay.portal.workflow.kaleo.designer.web.internal.messaging.KaleoDefinitionVersionModelListener.onAfterCreate(KaleoDefinitionVersionModelListener.java:48)
        at com.liferay.portal.workflow.kaleo.designer.web.internal.messaging.KaleoDefinitionVersionModelListener.onAfterCreate(KaleoDefinitionVersionModelListener.java:33)
        at com.liferay.portal.kernel.service.persistence.impl.BasePersistenceImpl.update(BasePersistenceImpl.java:361)
        at com.liferay.portal.workflow.kaleo.service.impl.KaleoDefinitionVersionLocalServiceImpl.addKaleoDefinitionVersion(KaleoDefinitionVersionLocalServiceImpl.java:93)
        at com.liferay.portal.spring.aop.ServiceB...

Test bundle downloads:

@rafaprax
Copy link
Owner

Pull request forwarded to brianchandotcom#60683. See changes here.

:octocat: Sent from GH.

@rafaprax rafaprax closed this Jun 22, 2018
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.

3 participants